Skip to content

Conversation

chohongm
Copy link
Contributor

@chohongm chohongm commented May 30, 2024

Fixes: AC-2521

Changelogs

  • Fixed a bug where horizontal suggested reply items contents are not wrapping to multiple lines.

After

Screenshot 2024-05-30 at 1 16 39 PM
Screenshot 2024-05-30 at 1 08 47 PM

Figma

https://www.figma.com/design/SVbXU00FhjztekD8AiVukK/UIKit_Work-file_React?node-id=2393-13688&t=cyEn8F21R1sNrdNx-0

@chohongm chohongm requested review from AhyoungRyu, bang9 and HoonBaek May 30, 2024 04:20
@chohongm chohongm self-assigned this May 30, 2024
Copy link

netlify bot commented May 30, 2024

Deploy Preview for sendbird-uikit-react ready!

Name Link
🔨 Latest commit d3bbb64
🔍 Latest deploy log https://app.netlify.com/sites/sendbird-uikit-react/deploys/66580092591a120008f58bda
😎 Deploy Preview https://deploy-preview-1115--sendbird-uikit-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@chohongm chohongm requested review from AhyoungRyu and bang9 May 30, 2024 04:29
@chohongm chohongm added this pull request to the merge queue May 30, 2024
white-space: pre-wrap;
font-size: 12px;
padding: 0 14px;
padding: 8px 14px;
Copy link
Contributor

Choose a reason for hiding this comment

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

There could be extra padding(or margin) in Widget which used to be covering this. If so, please remove it :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no widget now uses UIKit SR so no problem

Merged via the queue into main with commit 59a6f0d May 30, 2024
@AhyoungRyu AhyoungRyu deleted the fix/AC-2521-fix-multiline-suggested-reply branch May 30, 2024 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants