Bug 458014 - Effective module width changes size for different slider values
Summary: Effective module width changes size for different slider values
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_feedback (show other bugs)
Version: 5.25.4
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Nate Graham
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-17 23:12 UTC by Jack Hill
Modified: 2022-08-23 13:32 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.26
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jack Hill 2022-08-17 23:12:44 UTC
SUMMARY
The effective module width seems to be calculated based on the length of the text under the slider. Since some slider values have longer text than others, this causes the width to change between these values. This means:
1. The slider changes length, and therefore the slider grooves actually change place while the user is moving the handle
2. The bullet points move left/right

Both of these are quite jarring, but issue 1 is the worst. When moving the slider between grooves 2 & 3 (assuming left=0 and right=4) the page flickers between the two values.

STEPS TO REPRODUCE
1. Move the slider through all different values
2. 
3. 

OBSERVED RESULT
The effective page width changes size

EXPECTED RESULT
The page width should be calculated based on slider value with the longest width

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 36
KDE Plasma Version: 5.25.4
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Kernel Version: 5.18.17-200.fc36.x86_64 (64-bit)
Graphics Platform: Wayland

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2022-08-23 05:54:20 UTC
Can confirm.
Comment 2 Nate Graham 2022-08-23 13:32:53 UTC
Git commit 83029afbc22d8a8ed520c3854c3da491e8c456e7 by Nate Graham.
Committed on 23/08/2022 at 13:31.
Pushed by ngraham into branch 'master'.

Increase minimum header text width to prevent UI jumpiness

And increase the maximum widths of other things to keep them in sync.
FIXED-IN: 5.26

M  +4    -3    kcms/feedback/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-workspace/commit/83029afbc22d8a8ed520c3854c3da491e8c456e7