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:File attachement UI #953

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

Conversation

AyushKumar123456789
Copy link
Contributor

@AyushKumar123456789 AyushKumar123456789 commented Feb 3, 2025

Fix UI issues for file and audio attachments

Acceptance Criteria fulfillment

  • File attachments now display multi-line descriptions correctly.
  • A collapse/expand button has been added for file attachments.
  • Audio attachments now render multi-line messages without overlapping.

Fixes #952

Video/Screenshots

image
image
Previously :
image
image

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-953 after approval.

@SinghaAnirban005
Copy link
Contributor

Hey @AyushKumar123456789 could you not implement "A collapse/expand button has been added for file attachments." because i had already raised a PR specific to it #836 and it has been approved
Thank You !

@Spiral-Memory
Copy link
Collaborator

Let me know once the @SinghaAnirban005 feedback is implemented

@AyushKumar123456789
Copy link
Contributor Author

AyushKumar123456789 commented Feb 9, 2025

Sir, I have made the necessary changes as pointed out by @SinghaAnirban005; also, currently, I am a little busy with my college exams; pardon me for not being able to reply to your messages quickly.

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.

Fix : File_Attachement_UI_Not_Correct
3 participants