Skip to content
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

Closed
wants to merge 3 commits into from
Closed

[Tab] Applies muiTheme to Tab component. #889

wants to merge 3 commits into from

Conversation

mustafashabib
Copy link

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.

Mustafa Shabib added 2 commits June 19, 2015 15:37
Merged default tab styles into the light-theme
and set the inline styles for the tab component
based on those theme properties.
@mustafashabib
Copy link
Author

Cool, just pushed a commit to better adhere to the pattern. Let me know what you think.

fontWeight': '500',
whiteSpace': 'initial',
boxSizing': 'border-box'
},
Copy link
Member

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.

Copy link
Author

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?

Copy link
Member

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.

@hai-cea hai-cea changed the title Applies muiTheme to Tab component. [Tab] Applies muiTheme to Tab component. Jul 8, 2015
@hai-cea
Copy link
Member

hai-cea commented Jul 22, 2015

#1244 Allows us to customize the inkbar color through theme vars. Also, #1133 is a discussion on whether we should be putting css styles inside our theme, so I'm going to close this until we have clearer direction on #1133.

@hai-cea hai-cea closed this Jul 22, 2015
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Nov 10, 2020
* 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
@zannager zannager added the component: tabs This is the name of the generic UI component, not the React module! label Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: tabs This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants