-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[Tabs] Fix the types of the color props #20595
Conversation
sirajalam049
commented
Apr 16, 2020
•
edited by oliviertassinari
Loading
edited by oliviertassinari
- I have followed (at least) the PR section of the contributing guide.
Details of bundle changes.Comparing: fe687bf...f59edc8 Details of page changes
|
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.
Please add a test that explains what you're trying to fix.
I just updated the .d.ts file. It is also required to write the test? @eps1lon |
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 believe the types have been wrong since day one: #7874. While we aim to add the support for | string
to all the components, we are not here yet.
We aim to add the support for | string to all the components, so that the user may pass the color string along with the theme colors (primary, secondary etc.)? |
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 had the diff backwards, sorry. Yeah those types didn't make much sense. They were probably too wide to work around type widening.
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.
We need to run yarn proptypes
, yarn docs:api
and commit the changes though.
…efined and used in the Tabs.js
08442b8
to
f59edc8
Compare
@sirajalam049 It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
I'm glad. Thanks. Deleting branch |