Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add "View Comment" context action to channels #6525

Open
dexgs opened this issue Aug 4, 2024 · 3 comments · May be fixed by #6526
Open

Add "View Comment" context action to channels #6525

dexgs opened this issue Aug 4, 2024 · 3 comments · May be fixed by #6526
Labels
accessibility Everything related to accessibility-features (like screen-readers) client feature-request This issue or PR deals with a new feature good first issue Good for first-time contributors ui

Comments

@dexgs
Copy link
Contributor

dexgs commented Aug 4, 2024

Context

Currently, the only way to view channel comments is to hover the cursor over the channel, or click the comment indicator. Either way, the channel comment is only viewable as a pop-up, not a proper window which allows scrolling/copy-pasting.

Currently, there is no way to copy text from a channel comment or scroll through a channel comment that is too big to fit on screen at once without having "write ACL" permission on the channel and using the comment editor.

Description

There is currently a "View Comment" context action on users which lets you view their comment in a window which allows text selection and scrolling. This "View Comment" action should also be available on channels.

Mumble component

Client

OS-specific?

No

Additional information

No response
@dexgs dexgs added feature-request This issue or PR deals with a new feature triage This issue is waiting to be triaged by one of the project members labels Aug 4, 2024
@Hartmnt Hartmnt added client ui good first issue Good for first-time contributors accessibility Everything related to accessibility-features (like screen-readers) and removed triage This issue is waiting to be triaged by one of the project members labels Aug 4, 2024
@dexgs dexgs linked a pull request Aug 4, 2024 that will close this issue
Open
1 task
@Ali-Bytes
Copy link

Hi, I'm interested in contributing!

@Krzmbrzl
Copy link
Member

Krzmbrzl commented Oct 28, 2024 •
edited
Loading

@Ali-Hon thanks for your interest. However, this particular issue has already been addressed by @dexgs in the linked pull request (we just did not get around reviewing it).

There are lots of other open issues though!

@Ali-Bytes
Copy link

Oki :) I will keep looking for one, defect or feature but feel free to recommend one or point me towards an important one that has not been picked up yet :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accessibility Everything related to accessibility-features (like screen-readers) client feature-request This issue or PR deals with a new feature good first issue Good for first-time contributors ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FEAT(client): Add "View Description" context action to channels dexgs/mumble
4 participants
@Hartmnt @Krzmbrzl @Ali-Bytes @dexgs