-
-
Notifications
You must be signed in to change notification settings - Fork 482
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
Add explicit size option to ToggleIcon #6092
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6092 +/- ##
=======================================
Coverage 84.66% 84.67%
=======================================
Files 296 296
Lines 44057 44073 +16
=======================================
+ Hits 37302 37318 +16
Misses 6755 6755
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
"cell_type": "markdown", | ||
"metadata": {}, | ||
"source": [ | ||
"The icon will automatically adapt to the specified `width`/`height` but you may also provide an explicit `size`:" |
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.
"The icon will automatically adapt to the specified `width`/`height` but you may also provide an explicit `size`:" | |
"The icon will automatically adapt to the specified `width`/`height` but you may also provide an explicit `size` which takes precedence:" |
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.
Tested and looks good to me! Thanks for cleaning up the notebook too.
* Add explicit size option to ToggleIcon * Add tests * Remove line-height around icons
ToggleIcon
now allows eitherwidth
/height
or an explicit size to be defined.