Skip to content

Feature/tab inputs#1801

Merged
shaurya947 merged 5 commits intomui:masterfrom
Yepstr:feature/tabInputs
Oct 5, 2015
Merged

Feature/tab inputs#1801
shaurya947 merged 5 commits intomui:masterfrom
Yepstr:feature/tabInputs

Conversation

@mullwaden
Copy link

Fixed issue where tab only accepts a string for its label. Added support for string and object in the PropTypes

src/tabs/tab.jsx Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if React.PropTypes.node would be better here.
According to the doc of facebook

Anything that can be rendered: numbers, strings, elements or an array

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so too. @mullwaden what do you think?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree

@mullwaden mullwaden mentioned this pull request Oct 5, 2015
shaurya947 added a commit that referenced this pull request Oct 5, 2015
@shaurya947 shaurya947 merged commit 2f8d461 into mui:master Oct 5, 2015
@shaurya947
Copy link
Contributor

Cool stuff, thanks @mullwaden

@zannager zannager added the scope: tabs Changes related to the tabs. label Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: tabs Changes related to the tabs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants