Bug 472899 - Play button is misplaced
Summary: Play button is misplaced
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_soundtheme (other bugs)
Version First Reported In: master
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-08-01 22:54 UTC by Patrick Silva
Modified: 2024-02-14 15:34 UTC (History)
2 users (show)

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


Attachments
screenshot (219.64 KB, image/png)
2023-08-01 22:54 UTC, Patrick Silva
Details
new screenshot (151.56 KB, image/png)
2024-02-14 10:35 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2023-08-01 22:54:30 UTC
Created attachment 160672 [details]
screenshot

STEPS TO REPRODUCE
1. oen System Sounds KCM
2. observe the play button
3. 

OBSERVED RESULT
as we can see in the attached screenshot, the play button is misplaced

EXPECTED RESULT
play button in the correct place

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.27.80
KDE Frameworks Version: 5.240.0
Qt Version: 6.6.0
Graphics Platform: Wayland
Comment 1 Nate Graham 2023-08-02 01:15:37 UTC
Are you using a fractional point size with a non-default font?
Comment 2 Patrick Silva 2023-08-02 01:22:05 UTC
I use an integer size with a non-default font.
Comment 3 Nate Graham 2023-08-02 15:51:50 UTC
What font?

Does it happen with 10pt Noto Sans?
Comment 4 Patrick Silva 2023-08-02 18:17:29 UTC
SF Pro Display font.

Cannot reproduce with 10pt Noto Sans.
Comment 5 Bug Janitor Service 2023-08-02 18:31:14 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3149
Comment 6 Ismael Asensio 2023-08-02 21:25:19 UTC
Git commit a61ab30ad0f6f341b805c2e5ad3bd471b53f8c5e by Ismael Asensio.
Committed on 02/08/2023 at 22:51.
Pushed by iasensio into branch 'master'.

kcms/soundtheme: Better compact mode calculation

Slightly adjust the width calculation to position the demo
play button on top and avoid edge cases (ex Noto Sans 11pt).

This mode is however a small workaround as we cannot set a
minimum size for the cards. Ideally, it can be dropped if
we adopt non-resizable cards for the KCMs.
FIXED-IN: 6

M  +2    -2    kcms/soundtheme/ui/main.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/a61ab30ad0f6f341b805c2e5ad3bd471b53f8c5e
Comment 7 Patrick Silva 2024-02-14 10:35:53 UTC
Created attachment 165815 [details]
new screenshot

Can reproduce again when using SF Pro Display font.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 6.0.80
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.1
Graphics Platform: Wayland
Comment 8 Nate Graham 2024-02-14 15:34:17 UTC
Fonts with the name "Display" in them are not intended for use on a screen; they're intended for use on physical signs and billboards. As such they have odd metrics that can provoke issues like this on your screen. I recommend not using Display fonts for the UI on your machine. If you insist on using it, don't submit bug reports for issues that it causes. :) 

I should add a warning on the Fonts KCM about that.