Skip to content

CSS: Allow to select and copy comments in a card#83

Merged
Rajat-Dabade merged 1 commit into
mattermost:mainfrom
rtfm98:main
Jul 16, 2025
Merged

CSS: Allow to select and copy comments in a card#83
Rajat-Dabade merged 1 commit into
mattermost:mainfrom
rtfm98:main

Conversation

@rtfm98

@rtfm98 rtfm98 commented Mar 2, 2025

Copy link
Copy Markdown
Contributor

Summary

This pull request changes the user-select value for .focalboard-body * from none to text. With this change, users are able to select and copy comments (either whole or parts of it). This is a functionality our Mattermost and Boards users request repeatedly because they have to be able to copy parts of comments for further usage (e.g., phone numbers or ticket/incident numbers; in our case, they are really long).

I couldn't think of any situation in which the current setting would be useful, but maybe I am missing something. Is there a reason that this was set to none?

Ticket Link

-

@iosifpeterfi

Copy link
Copy Markdown

I added a PR that fixes the comments specifically, leaving everything else like it is:

#102

@mattermost-build

Copy link
Copy Markdown
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@marianunez marianunez added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester and removed Lifecycle/1:stale labels Jul 11, 2025

@Rajat-Dabade Rajat-Dabade left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not able to think of any use case where the selection should be disabled. This LGTM 👍

@Rajat-Dabade
Rajat-Dabade merged commit 6e90b8d into mattermost:main Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants