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.

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

feat: Enhance add link modal and disabling message items while typing #809

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

thepiyush-303
Copy link
Contributor

@thepiyush-303 thepiyush-303 commented Jan 5, 2025

Brief Title

Enhance add link modal and disabling message items while typing

Acceptance Criteria fulfillment

  • Improving Add Link Modal - adding link icon, giving label
  • Disabling Audio/video/file options in toolbar while typing
  • change position of tooltip of message toolbar facing above because for the bottom most messages the titles are being cut off.
  • pressing outside of the modal or clicking 'esc' should close the modal.
  • change the labels to what RC follows

Fixes #808

Video/Screenshots

Screencast.from.2025-01-16.23-16-15.mp4

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-809 after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@dhairyashiil
Copy link
Contributor

Please attach a video/screenshot of the Add Link modal as well.

@thepiyush-303
Copy link
Contributor Author

Please attach a video/screenshot of the Add Link modal as well.

It's already shown in the screencast attached above

@dhairyashiil
Copy link
Contributor

Please attach a video/screenshot of the Add Link modal as well.

It's already shown in the screencast attached above

Got it. Can we remove the placeholders? Since you added labels, users will know what to type inside them.

Regarding the labels, since it's a Link Window and we're only interacting with the link (not any other functionality), users will understand that the 'Text' and 'URL' refer to the link. There's no need to specifically label them as 'Link Text' and 'Link URL'.

Also, I think it would be nice to stay consistent with Rocket.Chat channel terminology, using only 'Text' and 'URL'.

@thepiyush-303
Copy link
Contributor Author

Got it!
Will commit soon.

@Spiral-Memory
Copy link
Collaborator

Resolve conflicts

@thepiyush-303
Copy link
Contributor Author

@Spiral-Memory

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhance Add Link Modal and disable audio/video/file messages on typing
3 participants