Review Request 125226: move EventForge from the desktop containment

[David Edmundson]

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



src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.cpp (line 257)
<https://git.reviewboard.kde.org/r/125226/#comment59064>

    This adds the same item multiple times.
    
    If you have an item with a child: the first itteration will add it and the child, then we recurse and add the child again.
    
    I think you can kill the
    
    itemList.append(parentItem->childItems()) 
    
    ?


- David Edmundson


On Sept. 15, 2015, 5:32 p.m., Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125226/
> -----------------------------------------------------------
> 
> (Updated Sept. 15, 2015, 5:32 p.m.)
> 
> 
> Review request for KDE Frameworks, Plasma and Eike Hein.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> -------
> 
> EventForge can cancel the half-managed events of child items
> in order to implement the move of them with press and hold,
> not having the parent and the children battling for the same
> press-move-release event sequence.
> 
> API-wise I think it's fine, apart perhaps the EventForge class name
> (perhaps MouseEventGrabber is less technically correct, but still more clear?)
> 
> 
> Diffs
> -----
> 
>   src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.h a62efb7 
>   src/qmlcontrols/kquickcontrolsaddons/mouseeventlistener.cpp d3321a3 
> 
> Diff: https://git.reviewboard.kde.org/r/125226/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco Martin
> 
>

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