Skip to content

Conversation

@DataTriny
Copy link
Contributor

Closes #9932

@ogoffart ogoffart added the a:widgets Implementation of widgets (from std-widgets.slint) and their styles (mF,bS) label Nov 10, 2025
@ogoffart
Copy link
Member

Thanks for this PR.

I had a quick try, but the interaction with the mouse is now reversed.

@ogoffart
Copy link
Member

Ah, it's only the Qt style that has it reverse, all the other styles are behaving properly.

@ogoffart
Copy link
Member

So the bug with the Qt slider is that this PR doesn't actually invert it.
The same as this should be done: https://github.com/qt/qtbase/blob/6667d4998546367e0bac972617231a1ca06ee4db/src/widgets/widgets/qslider.cpp#L114

That is, in initQSliderOptions , we should set option.upsideDown = vertical;

@DataTriny DataTriny force-pushed the vertical-slider-direction branch from 8ed8921 to f838a2a Compare November 12, 2025 16:20
@DataTriny
Copy link
Contributor Author

I have applied the requested changes and removed changes to Home/End keys that were actually wrong.

However I have manually tested the Qt style and it doesn't seem to make any difference. On my Windows system, dragging the mouse towards the top increases the value and dragging it towards the bottom decreases it, as for the other styles. I tried running the test cases but I can't get the Rust driver to run the tests for Qt even though my Qt installation is working.

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

Labels

a:widgets Implementation of widgets (from std-widgets.slint) and their styles (mF,bS)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vertical Slider is upside down

2 participants