Bug 472872

Summary: Tooltips have inconsistent height and font size
Product: [Frameworks and Libraries] libplasma Reporter: Patrick Silva <bugseforuns>
Component: tooltipsAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: kde, me, nate, notmart
Priority: NOR Keywords: qt6
Version: unspecified   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 6.0
Sentry Crash Report:
Attachments: screen recording

Description Patrick Silva 2023-08-01 11:25:42 UTC
Created attachment 160665 [details]
screen recording

STEPS TO REPRODUCE
1. open Kickoff
2. hover over icons and buttons repeatedly
3. 

OBSERVED RESULT
tooltips with inconsistent height and font size. Please watch the attached screen recording.

EXPECTED RESULT
tooltips with consistent height and font size


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 Patrick Silva 2023-08-01 11:27:37 UTC
Can reproduce with tooltips from Plasma Styles KCM too.
Comment 2 Nate Graham 2023-08-01 19:41:31 UTC
Can reproduce. When it happens, the following it printed to the journal log:

Aug 01 13:35:53 Liberator plasmashell[270504]: <Unknown File>:1:26: QML ToolTip: Binding loop detected for property "implicitHeight"

I can reproduce this with Kickoff (which uses a PlasmaComponents3.Tooltip) and Elisa (which uses a QQC2.ToolTip). So either our styling code is broken in the same way in both places, or this is a Qt bug. Commenting out our implicitHeight overrides from both doesn't fix the problem, so I'm leaning towards a Qt bug.
Comment 4 Nate Graham 2023-08-02 17:20:54 UTC
Git commit f76173ada72fa8c5011f2f7f3e0d4740e9bf920c by Nate Graham.
Committed on 02/08/2023 at 19:18.
Pushed by ngraham into branch 'master'.

ToolTip: do not rely on Layout

This is Fabian Kosmale's fix from
https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/268,
but applied to Plasma tooltips as well.

Co-authored-by: Fabian Kosmale <fabian.kosmale@qt.io>

M  +16   -5    src/declarativeimports/plasmacomponents3/ToolTip.qml

https://invent.kde.org/frameworks/plasma-framework/-/commit/f76173ada72fa8c5011f2f7f3e0d4740e9bf920c