Fix handling of style property in Tabs component#139
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #139 +/- ##
=======================================
Coverage 92.21% 92.22%
=======================================
Files 86 86
Lines 1503 1504 +1
Branches 231 230 -1
=======================================
+ Hits 1386 1387 +1
Misses 98 98
Partials 19 19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
clarasb
requested review from
b-yogesh and
forman
and removed request for
b-yogesh
b-yogesh
left a comment
b-yogesh
left a comment
There was a problem hiding this comment.
Thanks. Looks good. Please have a look at my comments.
| type: "Tab"; | ||
| label?: string; | ||
| icon?: string; | ||
| iconPosition?: "bottom" | "end" | "start" | "top" | undefined; |
There was a problem hiding this comment.
why do we have undefined here?
Also, in the API docs https://mui.com/material-ui/api/tab/, the default is set to top. If you change that to undefined, does it affect it in any way?
| ## Version 0.1.8 (in development) | ||
|
|
||
| * Fixed handling of `style` prop in `Tabs` component and added prop | ||
| `iconPosition`. (#135) |
There was a problem hiding this comment.
This PR mentioned two issues, please mention them both here.
| @@ -1,5 +1,7 @@ | |||
| ## Version 0.1.8 (in development) | |||
|
|
|||
| * Added `iconPosition` to `Tabs` Component. (#135) | |||
| return ( | ||
| <MuiTab | ||
| key={index} | ||
| style={tabState?.style} |
There was a problem hiding this comment.
we apply the same style to both the Tab button and body, was this a design decision? or does it needs discussion?
2 participants
This PR:
styleprop inTabscomponent.iconPositiontoTabscomponentExample Panel H (chartlets.py/demo/my_extension/my_panel_8.py):
Closes #135 and #136.