-
Notifications
You must be signed in to change notification settings - Fork 226
Timeline header: ensure that the decoration is clickable #4495
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
Conversation
tests/uitests/src/test/snapshots/images/features.messages.impl_MessagesView_Day_13_en.png
Show resolved
Hide resolved
tests/uitests/src/test/snapshots/images/features.messages.impl_MessagesView_Day_12_en.png
Show resolved
Hide resolved
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Note sure why there are so many diff in the sc recording. It's probably coming from #4484 being merged. Edit: I have created a branch to record the screenshots currently on develop to confirm this. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #4495 +/- ##
===========================================
- Coverage 80.10% 80.10% -0.01%
===========================================
Files 2080 2080
Lines 55680 55679 -1
Branches 6812 6812
===========================================
- Hits 44605 44604 -1
Misses 8737 8737
Partials 2338 2338 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…y fix the icon rendering.
a3a6b4d
to
6fb8424
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Content
Timeline header: ensure that the decoration is clickable and properly fix the icon rendering (when the room name is too long).
Motivation and context
Whole clickable room title / nice UI
Screenshots / GIFs
Tests
Tested devices
Checklist