Bug 472872 - Tooltips have inconsistent height and font size
Summary: Tooltips have inconsistent height and font size
Status: RESOLVED FIXED
Alias: None
Product: libplasma
Classification: Frameworks and Libraries
Component: tooltips (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: qt6
Depends on:
Blocks:
 
Reported: 2023-08-01 11:25 UTC by Patrick Silva
Modified: 2023-08-02 17:21 UTC (History)
4 users (show)

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


Attachments
screen recording (1.14 MB, video/webm)
2023-08-01 11:25 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 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