-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Better overflow and wrapping in Status area #7850
base: master
Are you sure you want to change the base?
Conversation
- Added `headCanWrap` stored property, and toggle button. - CSS mods, cleanup.
- New glyphs for `icon-multiline` and `icon-singleline`. - Refinements to button labels and titles for clarity.
- Add code and CSS for toggling single/multiline display of indicators. - Add code to detect overflow state of indicators.
- WIP, but is working! - TODOs: - Make sure this is a good way to do things. - Cleanup code. - Tests.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7850 +/- ##
==========================================
- Coverage 59.18% 58.76% -0.42%
==========================================
Files 675 675
Lines 27284 27306 +22
Branches 2671 2675 +4
==========================================
- Hits 16148 16047 -101
- Misses 11089 11210 +121
- Partials 47 49 +2
... and 13 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
- Fixed CSS classing on collapse/expand button. - New method added for storing head props in local storage. - TODOs: - Make sure using Update hook is performant. - Tests.
…AppLayout.vue component. - Code cleanup, console.log removals.
Closes #6655
Describe your changes:
TODOs:
All Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist