Bug 484243 - GtkLevelBar min-height for vertical and min-width for horizontal too big
Summary: GtkLevelBar min-height for vertical and min-width for horizontal too big
Status: CONFIRMED
Alias: None
Product: Breeze
Classification: Plasma
Component: gtk theme (other bugs)
Version First Reported In: 6.0.2
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-03-22 13:03 UTC by infirit
Modified: 2024-04-11 22:37 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:


Attachments
Image comparison 32px vs 1px (6.13 KB, image/png)
2024-03-22 13:03 UTC, infirit
Details

Note You need to log in before you can comment on or make changes to this bug.
Description infirit 2024-03-22 13:03:33 UTC
Created attachment 167604 [details]
Image comparison 32px vs 1px

SUMMARY
Breeze sets  vertical min-height and horizontal min-width to 32px which is a bit excessive and leads to incorrect results showing on the levelbar. For my use-case it results in most of the levelbar being filled even when the value of the bar is low. I'll attach an example of what the default looks like and when I override min-height to 1px.

More reasonable to me would be 1px but breeze might want more to fit it's style (Adwaita set 1px). Breeze isn't the only theme that is broken so I'll keep my override regardless but it would be nice if this can be fixed :-).

STEPS TO REPRODUCE
1. Set breeze as Gtk Application theme
2. Open any application with GtkLevelBar with a low enough value (<10 should make it fairly obvious)

OBSERVED RESULT
With the scale of 1 -100 and the value of the GtkLevelBar set to 10 the bar in the example is pretty much completely filled.

EXPECTED RESULT
Have only 10 percent of the bar filled

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: 6.0.2
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 6.0.2
Qt Version: 6.6.2
Comment 1 Nate Graham 2024-04-11 22:37:50 UTC
Can you try with 1 and see if it looks okay? If it does please feel free to submit a merge request to fix it at https://invent.kde.org/plasma/breeze-gtk/-/merge_requests. Thanks for spotting this!