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: prevent chat window resize on long links #661

Closed

Conversation

haresahani
Copy link

Fix Message Body Word Wrapping and Overflow Handling

  • Ensure long links break properly in the message body without overflowing.
  • Apply word-wrap and overflow-wrap to the message body and anchor tags.
  • Ensure the styles are consistent across light and dark modes.
  • Validate that text and links in the message body wrap normally.

Fixes #92

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@devanshkansagra
Copy link
Contributor

devanshkansagra commented Nov 8, 2024

Hey @haresahani, There are lots of unwanted additions in package-lock.json in every monorepo package, can you please discard the unnecessary ones

Copy link
Contributor

@devanshkansagra devanshkansagra left a comment

Choose a reason for hiding this comment

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

if the file as a dependency in every package is unnecessary then kindly remove all of those
these ones:
"@embeddedchat/api": "file:"
"@embeddedchat/auth": "file:"
"docs": "file:",
"@embeddedchat/htmlembed": "file:",
"@embeddedchat/rc-app": "file:"
"@embeddedchat/react-native": "file:",
"@embeddedchat/ui-elements": "file:",
"@embeddedchat/ui-kit": "file:",

Copy link
Collaborator

@Spiral-Memory Spiral-Memory left a comment

Choose a reason for hiding this comment

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

@haresahani
With 1 lakh lines of changes, something seems off. Please revise this PR carefully. Please respond to it, or i'll have to close this PR to avoid confusion for other collaboraors.

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.

Size getting large on sending long links
4 participants