Review Request 122939: Immediately hide tooltip on ToolTipArea destruction or when it becomes empty

[David Edmundson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122939/#review77931
-----------------------------------------------------------



src/declarativeimports/core/tooltip.cpp (line 63)
<https://git.reviewboard.kde.org/r/122939/#comment53434>

    I'm not sure about this, doesn't this mean if I'm hovering over the clock and kmail closes my tooltip will vanish?


- David Edmundson


On March 14, 2015, 12:40 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122939/
> -----------------------------------------------------------
> 
> (Updated March 14, 2015, 12:40 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> When the tooltip is empty (no text and no subtext and no mainitem) it will not show. However, when it becomes empty while it is shown, it stays there as a small rectangle.
> 
> This patch hides the tooltip immediately (so KWin's fadeout animation still has the proper content rather than the empty tooltip) when the area has been destroyed or its content becomes empty.
> 
> It access s_dialog directly not to unneccessarily create a tooltipInstance just for a call in the destructor or that wouldn't have effect without a tooltip anyway.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/tooltip.h 7a58826 
>   src/declarativeimports/core/tooltip.cpp 40e0af5 
> 
> Diff: https://git.reviewboard.kde.org/r/122939/diff/
> 
> 
> Testing
> -------
> 
> Hovered KMail tray icon, closed KMail using Ctrl+Q while still hovering the icon.
> Previously the tooltip would change into an empty rectangle, staying there until the tooltip times out, now it hides nicely.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150322/3d73d1c4/attachment.html>