D22860: introduce FallbackTapHandler

2019-08-01 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R296:e634bda6331c: introduce FallbackTapHandler (authored by mart). REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22860?vs=62891&id=62894 REVISION DETAIL https:

D22860: introduce FallbackTapHandler

2019-08-01 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R296 KDeclarative BRANCH phab/fallbacktaphandler REVISION DETAIL https://phabricator.kde.org/D22860 To: mart, #plasma, davidedmundson Cc: davidedmundson, kde-frameworks-devel, LeGast00n, sbe

D22860: introduce FallbackTapHandler

2019-08-01 Thread Marco Martin
mart updated this revision to Diff 62891. mart added a comment. - get completely rid of acceptedMouse§Buttons REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22860?vs=62890&id=62891 BRANCH phab/fallbacktaphandler REVISION DETAIL https://phabricato

D22860: introduce FallbackTapHandler

2019-08-01 Thread David Edmundson
davidedmundson added inline comments. INLINE COMMENTS > fallbacktaphandler.cpp:50 > +{ > +setAcceptedMouseButtons(buttons); > +emit acceptedButtonsChanged(); m_buttons = buttons or change the getter for acceptedButtons to call QQuickItem::acceptedMouseButtons REPOSITORY R296 KDeclar

D22860: introduce FallbackTapHandler

2019-08-01 Thread Marco Martin
mart marked 3 inline comments as done. REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D22860 To: mart, #plasma, davidedmundson Cc: davidedmundson, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns

D22860: introduce FallbackTapHandler

2019-08-01 Thread Marco Martin
mart updated this revision to Diff 62890. mart added a comment. - adress comments REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D22860?vs=62868&id=62890 BRANCH phab/fallbacktaphandler REVISION DETAIL https://phabricator.kde.org/D22860 AFFECTED F

D22860: introduce FallbackTapHandler

2019-08-01 Thread David Edmundson
davidedmundson requested changes to this revision. davidedmundson added inline comments. This revision now requires changes to proceed. INLINE COMMENTS > fallbacktaphandler.cpp:71 > +m_mouseDownItem.clear(); > +m_mouseDownPos = QPointF(); > + You're clearing before doing the tap

D22860: introduce FallbackTapHandler

2019-07-31 Thread Marco Martin
mart created this revision. mart added a reviewer: Plasma. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mart requested review of this revision. REVISION SUMMARY This element is intended to be used as "click on empty areas to dismiss" use case. a MouseA