-
Couldn't load subscription status.
- Fork 290
fix: inconsistency in quoted message and regular message white space… #2601
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
base: master
Are you sure you want to change the base?
Conversation
|
Size Change: +52 B (0%) Total Size: 1.19 MB
ℹ️ View Unchanged
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2601 +/- ##
==========================================
+ Coverage 81.12% 81.14% +0.02%
==========================================
Files 448 448
Lines 9464 9464
Branches 2312 2312
==========================================
+ Hits 7678 7680 +2
+ Misses 1423 1421 -2
Partials 363 363 ☔ View full report in Codecov by Sentry. |
4a90078 to
f9159c1
Compare
f9159c1 to
5c30405
Compare
…ges (#322) ### 🎯 Goal - White spaces were treated differently in quoted messages - Adds clamping to quoted message See screenshot for details Merge this if you want the fix for React as well: GetStream/stream-chat-react#2601 ### 🛠 Implementation details _Provide a description of the implementation_ ### 🎨 UI Changes **White space incosistencies** If we quoted a message with line-breaks before...  ...the line breaks were not properly preserved in the quote preview:  **Clamping** Without message clamping quoted messages can take up a lot of space:  With message clamping: 
Submit a pull request
🎯 Goal
Details: GetStream/stream-chat-css#322
Draft because we have to update stream-chat-css before merging!!!!
🛠 Implementation details
Provide a description of the implementation
🎨 UI Changes
Add relevant screenshots