> On March 22, 2015, 9:28 p.m., David Edmundson wrote: > > src/declarativeimports/core/tooltip.cpp, line 63 > > <https://git.reviewboard.kde.org/r/122939/diff/1/?file=354909#file354909line63> > > > > I'm not sure about this, doesn't this mean if I'm hovering over the > > clock and kmail closes my tooltip will vanish?
yeah, this should happen only when the tooltip verifies it's the temporary owner of the dialog - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122939/#review77931 ----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
