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] Support SVG Icons color #7091

Merged
merged 2 commits into from Jun 10, 2017
Merged

[Tab] Support SVG Icons color #7091

merged 2 commits into from Jun 10, 2017

Conversation

RejinR
Copy link

@RejinR RejinR commented Jun 9, 2017

The SVG icons color style is not being used in the tab even though the comment before the line says it takes the color style if it's SVG icon.

@oliviertassinari
Copy link
Member

@RejinR Isn't that a breaking change? Following our ROADMAP, I'm not sure we want to go forward with that PR. We focus on bug fixes for the master branch as we migrated our effort to the next branch.

@oliviertassinari oliviertassinari added the component: tabs This is the name of the generic UI component, not the React module! label Jun 9, 2017
@oliviertassinari oliviertassinari changed the title SVG Icons will take the color from the style props of the icon elemen… [Tab] SVG Icons will take the color from the style props of the icon elemen… Jun 9, 2017
@RejinR
Copy link
Author

RejinR commented Jun 9, 2017

I've added conditions to avoid breaking now..

@oliviertassinari oliviertassinari merged commit 547964a into mui:master Jun 10, 2017
@oliviertassinari
Copy link
Member

@RejinR Thanks!

@oliviertassinari oliviertassinari changed the title [Tab] SVG Icons will take the color from the style props of the icon elemen… [Tab] Support SVG Icons color Jun 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work 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.

None yet

2 participants