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
Fix tab disabled issue #568
Conversation
@@ -7,8 +7,9 @@ This project adheres to [Semantic Versioning](http://semver.org/). | |||
- Ability to add tooltips to `Slider` and `RangeSlider`, which can be visible always or on hover. Tooltips also take a position argument. [#564](https://github.com/plotly/dash-core-components/pull/564) | |||
|
|||
### Fixed | |||
- Fixed `min_date_allowed` and `max_date_allowed` bug in `DatePickerRange` [#551]https://github.com/plotly/dash-core-components/issues/551) | |||
- Fixed `min_date_allowed` and `max_date_allowed` bug in `DatePickerRange` [#551](https://github.com/plotly/dash-core-components/issues/551) |
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.
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.
Yeah, I thought I fixed it, must be.
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.
With a test even, thanks @zoncrd ! 💃
Co-Authored-By: alexcjohnson <johnson.alex.c@gmail.com>
Hmm, not sure why percy isn't picking up this job. Would it need write access to the source repo? Merging anyway. |
Fix an issue occurring in the Tab component when disabled and styled with custom CSS classes.
The
tab--disabled
CSS class was appended to the component className without an initial space and was concatenated to the last custom CSS class if provided, consequently breaking the styling.I had also a simple test to check if the disabled Tab component is findable by the
tab--disabled
CSS class.It's my first PR, let me know if it's okay or if I have to do something more.