Skip to content
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

ctkRangeWidget, ctkDoubleRangeSlider setValue(), setRange() issues #1157

Closed
HarryDC opened this issue Dec 14, 2023 · 1 comment · Fixed by #1158
Closed

ctkRangeWidget, ctkDoubleRangeSlider setValue(), setRange() issues #1157

HarryDC opened this issue Dec 14, 2023 · 1 comment · Fixed by #1158

Comments

@HarryDC
Copy link

HarryDC commented Dec 14, 2023

I don't know if these are related or not but in implementing the Gui wrapping for the ctkDoubleRangeSlider the following issues have come up. ctkDoubleRangeSlider does not expose setRange in python

But both the ctkRangeWidget and the ctkDoubleRangeSlider expose the setValues method this method though doesn't seem to accept negative values, e.g in testing when calling setValue(-1, 10) the range would result in [0,10]. The ctkRangeWidget::setRange() does support ranges on the negative side, but neither setValue() function does

From the documentation it is hard to tell the difference between the setValues and the setRange function

@jcfr
Copy link
Member

jcfr commented Dec 14, 2023

From the documentation it is hard to tell the difference between the setValues and the setRange function

This should help clarify:

Range(minimum)                                          Range(maximum)
  |                                                         |
  |---------------------------------------------------------|
            |                                   |
      min_value (left slider)         max_value (right slider)

A range slider has two sliders ("left" and "right") and the functions minimumValue() and maximumValue() allow to get these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants