-
Notifications
You must be signed in to change notification settings - Fork 296
Rework on messages view top bars #5401
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
No code change.
…d to MessagesViewTopBar.
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #5401 +/- ##
===========================================
- Coverage 80.37% 80.37% -0.01%
===========================================
Files 2282 2284 +2
Lines 63069 63066 -3
Branches 7887 7887
===========================================
- Hits 50692 50689 -3
Misses 9464 9464
Partials 2913 2913 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Content
Extract code for the messages view top bar to reduce file size. Create dedicated Preview for the top bar and reduce the number of Preview for MessagesView, for a faster rendering in the IDE.
Motivation and context
Smaller file size
Better performance on rendering previews of MessagesView.
Complement for #5374
Screenshots / GIFs
The recorded one should be OK, and the code coverage should not decrease.
Tests
Tested devices
Checklist