Review Request 129373: Plasma Framework changes for #129372

[René Fürst]

> On Nov. 10, 2016, 9:59 p.m., David Edmundson wrote:
> > src/declarativeimports/core/tooltipdialog.cpp, line 109
> > <https://git.reviewboard.kde.org/r/129373/diff/1/?file=484556#file484556line109>
> >
> >     there's a bug in this logic
> >     
> >      - Display Tooltip Dialog
> >      - Move mouse over the dialog
> >      - Move Tooltip Dialog
> >     
> >     
> >     contains mouse will still be true even though you're not containing the mouse; and you won't get a leave event until you mouse in and out again.

Are tooltips supposed to be moveable by mouse? If yes, do we have one around?


- René


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


On Nov. 10, 2016, 8:47 p.m., René Fürst wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129373/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2016, 8:47 p.m.)
> 
> 
> Review request for Plasma and Eike Hein.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Plasma Framework changes for #129372
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/tooltip.h d38c49b 
>   src/declarativeimports/core/tooltip.cpp ffe1064 
>   src/declarativeimports/core/tooltipdialog.h d4e0ff0 
>   src/declarativeimports/core/tooltipdialog.cpp 28ba9be 
> 
> Diff: https://git.reviewboard.kde.org/r/129373/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> René Fürst
> 
>

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