ToolTipTelegate is quite complex, so load it only the first time it's used and don't if tooltips are disabled altogether:
Also move populating the tooltip data to aboutToShow, this way we don't repeatedly update it just because the mouse moved over Task Manager without actually resting long enough to trigger the tooltip.
Details
Details
- All the bindings still work and the highlight effect also excludes the tooltip just fine.
- Verified tooltip delegate is only created once
- Verified disabling tooltip destroy delegate, and it is then recreated once when re-enabled as needed
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
destroy it when no longer needed
When I disable the "show tooltips" setting or when I close the tooltip? I don't want to re-create the tooltip item everytime, though.
Comment Actions
- Rebase ontop of master
- Populate in aboutToShow (cf. D10381)
- Destroy toolTipDelegate when tooltips are disabled
Comment Actions
Sometimes I wish we'd implement lazy loading in the backends instead (e.g. why is heavy when it's never been visible). The more we optimize our QML the more we go procedural/non-declarative and hard to read sadly.
Comment Actions
It seems to cause a lot of trouble with grouping and things, the tooltip is just super wonky with this. Needs to be revisited from scratch again