Skip to content

Set slider to min/max constraint when value exceeds constraint. #770

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

Merged
merged 1 commit into from
May 14, 2012

Conversation

tonysyu
Copy link
Contributor

@tonysyu tonysyu commented Mar 16, 2012

Currently, when a slider's limits are constrained by another slider (using slidermin/slidermax), the slider will simply return unchanged when you move the slider beyond the limits.

Instead, the slider should update to the min/max value when closedmin/closedmax attributes are True. This would match the behavior of the valmin/valmax limits.

Currently, when a slider's limits are constrained by another slider (using `slidermin/slidermax`), the slider will simply return unchanged when you move the slider beyond the limits.

Instead, the slider should update to the min/max value when `closedmin/closedmax` attributes are True. This would match the behavior of the `valmin/valmax` limits.
@WeatherGod
Copy link
Member

This change looks ok to me. Sorry for the delay. Tony, I take it that you haven't encountered any issues with this change?

@tonysyu
Copy link
Contributor Author

tonysyu commented May 14, 2012

I've used it a bit on a simple interactive plot and it worked perfectly for my purposes. And I can't see why there would be any corner cases.

Thanks for reviewing.

@WeatherGod
Copy link
Member

Agreed. Pulling...

WeatherGod added a commit that referenced this pull request May 14, 2012
Set slider to min/max constraint when value exceeds constraint.
@WeatherGod WeatherGod merged commit 91310f8 into matplotlib:master May 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants