Review Request 126463: Allow applets to set full ToolTip item
Martin Klapetek
martin.klapetek at gmail.com
Mon Jan 4 16:38:33 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126463/
-----------------------------------------------------------
(Updated Jan. 4, 2016, 4:38 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Frameworks and Plasma.
Changes
-------
Submitted with commit 3992593f31681254fb5fa9bafcaa47e18fc19421 by Martin Klapetek to branch master.
Repository: plasma-framework
Description
-------
Adds new property available as Plasmoid.toolTipItem that allows
to set a full custom item as the tooltip.
The main motivation is to have a custom Item tooltip for digital
clock to properly format the list of timezones (version of
https://git.reviewboard.kde.org/r/124047/). Adding a custom
PlasmaCore.ToolTipArea to the compact representation does not
work well because then it's a ToolTipArea nested inside another
ToolTipArea (from CompactApplet.qml), this causes a sliding effect
as it changes visual parents when hovering.
I dunno if this could perhaps be fixed, but seems redundant to have
two ToolTipAreas anyway, so instead this exposes the main applet's
ToolTipArea's mainItem property through Plasmoid.toolTipItem which
the appelts can use to set a custom item.
This has a part2 patch which goes of plasma-desktop:
https://git.reviewboard.kde.org/r/126464/
Diffs
-----
src/scriptengines/qml/plasmoid/appletinterface.h 40016b1
src/scriptengines/qml/plasmoid/appletinterface.cpp 60b7fa5
Diff: https://git.reviewboard.kde.org/r/126463/diff/
Testing
-------
I can have custom tooltip item in digital clock without any artifacts
while all other applets still have correct tooltips.
Thanks,
Martin Klapetek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160104/17d82a66/attachment.html>
More information about the Plasma-devel
mailing list