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
make sure tooltips aren't visible when parent slider isn't #878
Conversation
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.
@jdamiba This PR will need:
- a changelog.md entry for the bug being fixed
- a test that shows this works -- I think in this case a simple visual test saved in Percy will do the trick. Can be based on the issue's example code. No need for interactivity or anything like that.
- I suggest you add a comment in the code to the effect that if/when we'll update rc-slider / rc-tooltip we will need to revisit this code as the getTooltipContainer function will need to be a prop instead of a nested property
CHANGELOG.md
Outdated
@@ -7,6 +7,8 @@ This project adheres to [Semantic Versioning](http://semver.org/). | |||
- [#871](https://github.com/plotly/dash-core-components/pull/871) Add Julia syntax highlighting support for dcc.Markdown | |||
|
|||
## Updated | |||
- [#878](https://github.com/plotly/dash-core-components/pull/878) | |||
- used the `getTooltipContainter` prop of dcc.Slider to make sure tooltips are positioned correctly when the parent slider is not visible due to it being in a HTML element with an `overflow-x` style, fixing [#751](https://github.com/plotly/dash-core-components/issues/751) |
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 think I missed the wrong level of details of https://github.com/plotly/dash-core-components/blame/dev/CHANGELOG.md#L26 which you are using as your baseline.
Make the entry more focused on what this fixes for the user. Something along the lines of:
Fix a bug causing the dcc.Slider and dcc.RangerSlider tooltip to be visible even if the component isn't visible (e.g. overflow)
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.
🎚️ 🚢
closes #751