Bug 434009

Summary: Checkbox is not fully rendered in "Quit application" dialog depending on used font
Product: [Applications] plasma-systemmonitor Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, plasma-bugs
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 5.21.3
Attachments: screenshot

Description Patrick Silva 2021-03-05 13:21:55 UTC
Created attachment 136396 [details]
screenshot

SUMMARY
This bug does not occur with default font settings.

STEPS TO REPRODUCE
1. set Liberation Sans font in Fonts kcm
2. open System Monitor > Applications view
3. right-click on any app, hover over "Send signal", choose "Kill (KILL)"

OBSERVED RESULT
"Quit applicaton" dialog shows up with partially rendered checkbox on bottom.
Please see the attached screenshot.

EXPECTED RESULT
fully rendered checkbox

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.21.80
KDE Frameworks Version: 5.80.0
Qt Version: 5.15.2
Graphics Platform: X11
Comment 1 Bug Janitor Service 2021-03-12 09:22:17 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/104
Comment 2 David Redondo 2021-03-12 15:42:59 UTC
Git commit 2c4c2c889217c30a19e802974672dbc94a3fa123 by David Redondo.
Committed on 12/03/2021 at 09:15.
Pushed by davidre into branch 'master'.

The checkbox can be bigger than the text

Do not unconditionally bind the implicitHeight to the one of the text.
The default does the correct thing already.
FIXED-IN:5.21.3

M  +0    -1    src/table/KillDialog.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/2c4c2c889217c30a19e802974672dbc94a3fa123
Comment 3 David Redondo 2021-03-12 15:43:49 UTC
Git commit 93b6c1d5d1d73abfb8259b52afd15b7163681fce by David Redondo.
Committed on 12/03/2021 at 15:43.
Pushed by davidre into branch 'Plasma/5.21'.

The checkbox can be bigger than the text

Do not unconditionally bind the implicitHeight to the one of the text.
The default does the correct thing already.
FIXED-IN:5.21.3


(cherry picked from commit 2c4c2c889217c30a19e802974672dbc94a3fa123)

M  +0    -1    src/table/KillDialog.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/93b6c1d5d1d73abfb8259b52afd15b7163681fce