-
-
Notifications
You must be signed in to change notification settings - Fork 32.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
[Tab] Applies muiTheme to Tab component. #889
[Tab] Applies muiTheme to Tab component. #889
Conversation
Merged default tab styles into the light-theme and set the inline styles for the tab component based on those theme properties.
Cool, just pushed a commit to better adhere to the pattern. Let me know what you think. |
fontWeight': '500', | ||
whiteSpace': 'initial', | ||
boxSizing': 'border-box' | ||
}, |
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.
Also, I would rather keep this clean and only have styles that are relevant to override.
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.
Which would you like me to remove?
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.
I guess we should just keep color? If a developer needs to override these other items, they can pass in a style prop which should get merged in with default styles.
* Move running tests to the circleci * Format config.yml * Fix incorrect running command for circleci * Add lerna bootstrap to installation process on circleci * Fix incorrect lerna call in running app * Refactor Calednar.tsx * Calculate animation direction on moving with keyboard * Prevent changing slide direction if month wasn't changed
We noticed that the Tab component did not pick up any theme overrides. This should allow you to style them from your custom theme.
Thanks.