Elisa's sliders have Layout.fillHeight: true set on them so that their click areas will be taller than normal without having to re-implement the control from primitives. This gets broken by 95046311eb4a8be2696ee57776348941315b0632. The commit description suggests that the app shouldn't be doing this, but I don't see a clean way to do it without re-implementing the control. Do you have any ideas, Ivan, or should we revert that change?
That sounds cursed on Elisa's part. There used to be no guarantees from style regarding how large it actually wanted the control be. Now there are solid guarantees, and they are respected by qqc2-d-s. Stretching a slider in its perpendicular direction is not something most QStyles expect you to do with them. How about increasing padding instead?
Sure, can you give that a try and see if it will work?
Actually, I tried all four bundled QStyles again (breeze, O2, Fusion and M$ 9x), and it seems to be a bug in O2/Breeze family. I'd rather prefer centering them in Breeze, than adding more UI hacks to Elisa — because I found no clean way to extend padding/insets without making assumptions about particular style in use.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/breeze/-/merge_requests/348
Git commit 4d40895b8e11bef73b6bf8961856becc95358ed0 by ivan tkachenko. Committed on 11/09/2023 at 20:47. Pushed by ratijas into branch 'master'. Slider: Rework sizing and positioning to draw them centered If you stretch in its orthogonal direction (vertically for horizontal and vice-versa), a Slider with Breeze style will now act similar to Fusion style: it will center itself within given rectangle. But unlike Fusion, it would also account for Breeze specific tick marks which are so "small" from QStyle point of view that we have to subtract 2px back. M +77 -26 kstyle/breezestyle.cpp M +4 -0 kstyle/breezestyle.h https://invent.kde.org/plasma/breeze/-/commit/4d40895b8e11bef73b6bf8961856becc95358ed0