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

fix: Code Block Overflow and Add Backdrop for Announcement Modal #940

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

Conversation

thepiyush-303
Copy link
Contributor

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

Brief Title

include text wrapping in code-block for very large strings, Also added a backdrop and close button for Announcement modal.

Acceptance Criteria fulfillment

  • Wrapping long text inside code block.
  • Adding a backdrop for announcement modal.
  • adding a button to close the modal.

Fixes #939

Video/Screenshots

After:

Screencast.from.2025-01-29.22-27-16.mp4

Before:

Screencast.from.2025-01-29.22-26-23.mp4

PR Test Details

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

@Spiral-Memory
Copy link
Collaborator

Hi @thepiyush-303
Can we also make announcement modal scrollable ? Make sure the height is fixed, as other modals and keep it scrollable if text exceeds the size

@thepiyush-303
Copy link
Contributor Author

Screencast.from.2025-02-02.15-18-25.mp4

Done.
@Spiral-Memory

@Spiral-Memory
Copy link
Collaborator

Good work @thepiyush-303
LGTM 🚀

@Spiral-Memory Spiral-Memory added testing chore a subtle fix labels Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore a subtle fix testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code block is not wrapping text and no backdrop for announcement modal.
2 participants