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

Desktop GUI needs double-/range-sliders #20

Closed
GPMueller opened this issue Oct 18, 2016 · 2 comments · Fixed by #583
Closed

Desktop GUI needs double-/range-sliders #20

GPMueller opened this issue Oct 18, 2016 · 2 comments · Fixed by #583

Comments

@GPMueller
Copy link
Member

For z-range sliders, double sliders would be nicer than two coupled sliders.

Here is an example implementation: http://stackoverflow.com/a/26025796

@IngoMeyer441
Copy link
Contributor

I've tested the implementation provided on stack overflow and got this result:

screen shot 2016-10-20 at 16 21 39

In my opinion, this range slider is not optimal. The blue interval should be drawn between both handles and the second handle should be drawn with the Qt api directly and not using a png file.

The problem is that this approach tries to customize a normal QSlider. I think, it is necessary to create a custom widget to provide a good solution.

@GPMueller
Copy link
Member Author

GPMueller commented Jan 3, 2021

Implemented range sliders for ui-imgui on the feature-imgui branch with a62dd7d:
grafik

Note, range sliders are not yet implemented in ImGui itself, but for my implementation I took inspiration from comments on ocornut/imgui#76.

@GPMueller GPMueller linked a pull request Jan 12, 2021 that will close this issue
12 tasks
@GPMueller GPMueller changed the title UI-QT needs double sliders GUI needs double-/range-sliders May 30, 2021
@GPMueller GPMueller changed the title GUI needs double-/range-sliders Desktop GUI needs double-/range-sliders May 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants