-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
chore: update dataset count badge and tash icon sizing #15608
chore: update dataset count badge and tash icon sizing #15608
Conversation
… text inside via `line-height`
…ext inside via `line-height`, alignedItems inside div w/flex
Codecov Report
@@ Coverage Diff @@
## master #15608 +/- ##
==========================================
- Coverage 76.96% 76.88% -0.08%
==========================================
Files 976 976
Lines 51318 51324 +6
Branches 6907 6907
==========================================
- Hits 39496 39462 -34
- Misses 11603 11643 +40
Partials 219 219
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
This looks great!
* update trash icon size * changed badge icon `height` to `16px` (using `gridUnits` and centered text inside via `line-height` * changed badge icon `height` to `16px` (using `gridUnits`), centered text inside via `line-height`, alignedItems inside div w/flex
* update trash icon size * changed badge icon `height` to `16px` (using `gridUnits` and centered text inside via `line-height` * changed badge icon `height` to `16px` (using `gridUnits`), centered text inside via `line-height`, alignedItems inside div w/flex
* update trash icon size * changed badge icon `height` to `16px` (using `gridUnits` and centered text inside via `line-height` * changed badge icon `height` to `16px` (using `gridUnits`), centered text inside via `line-height`, alignedItems inside div w/flex
SUMMARY
Created a
styledComponent
to set hight and center numbers on thecount-badge
ondataset-tabs
. Also added inlinecss={{...align-items:center}}
to parent div - centers badge and tab-title.Removed inline
css
fromEditDatasetModals
'sIcon.Trash
. This Icon now matches the sizing of the otherTrashIcon
on the datasets homepage(see screenshot).BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
CountBadge:
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION