Skip to content

Conversation

@selvagsz
Copy link
Contributor

@selvagsz selvagsz commented Jun 16, 2016

This will be very useful, mainly when we work with CSS frameworks like bootstrap

Closes #65

@selvagsz selvagsz changed the title Feature/add custom active and disabled class [WIP] Feature/add custom active and disabled class Jun 16, 2016
@selvagsz selvagsz changed the title [WIP] Feature/add custom active and disabled class Feature/add custom active and disabled class Jun 16, 2016
'ReactTabs__Tab--selected': selected,
'ReactTabs__Tab--disabled': disabled,
[`${activeTabClassName}`]: selected,
[`${disabledTabClassName}`]: disabled,
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we could just do [activeTabClassName]: selected without template string?

@danez
Copy link
Collaborator

danez commented Jun 20, 2016

Thanks for contributing. If you could fix the two small issues I can merge it.

@selvagsz selvagsz force-pushed the feature/add_custom_active_and_disabled_class branch from 789ee5b to d0b5186 Compare June 21, 2016 04:34
@selvagsz
Copy link
Contributor Author

@danez done 👍

@selvagsz
Copy link
Contributor Author

selvagsz commented Jul 1, 2016

@danez Anything else needs to be done ?

@danez danez merged commit d420159 into reactjs:master Jul 4, 2016
@selvagsz selvagsz deleted the feature/add_custom_active_and_disabled_class branch July 5, 2016 03:39
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants