[KCM GridDelegate] Use non-attached ToolTip

Authored by broulik on Nov 22 2019, 12:42 PM.


Group Reviewers

Then it can just follow the style's delay and timeout.

Test Plan
  • Hovered grid delegate which had a tooltip, showed after a short delay, stated for like 10 seconds
  • Hovered grid delegate without one, didn't get any tooltip

Diff Detail

R296 KDeclarative
Lint Skipped
Unit Tests Skipped
broulik created this revision.Nov 22 2019, 12:42 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptNov 22 2019, 12:42 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Nov 22 2019, 12:42 PM

This works for you? I still see that the tooltip for KCM grid delegates disappears almost immediately with this patch.

davidedmundson requested changes to this revision.Dec 2 2019, 11:17 AM
davidedmundson added a subscriber: davidedmundson.

The attached version is a light QObject that calls a single tooltip.
The non-attached version contains /the entire tooltip instance/

Which on every delegate is too much of a problem.

You're right that we have to duplicate the delay logic to follow the theme, but at least in KDeclarative we're only doing that in one place and hiding it from the user code. If that needs to use C++ code, (or maybe kirigami theme should export this?) that might work

This revision now requires changes to proceed.Dec 2 2019, 11:17 AM

Kirigami has a toolTipDelay in Units. The other bit is hardcoded to some heuristic:

broulik abandoned this revision.May 21 2020, 9:18 AM