D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-18 Thread Steffen Hartleib
steffenh updated this revision to Diff 58241. steffenh added a comment. change name to KTouchGesture add a private class for every exported class remove unused code REPOSITORY R236 KWidgetsAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D21028?vs=57584&id=58241 BRANCH

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-08 Thread Nathaniel Graham
ngraham added a comment. Since these are touch-only, maybe `KTouchGesture` or `KTouchHandler`? REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frameworks Cc: ngraham, apol, nicolasfella, cfeck, kde-frameworks-devel, michaelh, bruns

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-08 Thread Steffen Hartleib
steffenh added a comment. Hi @ngraham, thanks for the link, I see KGesture is already used. I hope KGestureHandler is not too similar REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frameworks Cc: ngraham, apol, nicolasfella, cfeck,

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-08 Thread Nathaniel Graham
ngraham added a comment. Here's a tip: you can use https://lxr.kde.org to search for existing class names and identifiers. https://lxr.kde.org/search?_filestring=kgesture&_string= REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frame

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-08 Thread Steffen Hartleib
steffenh added a comment. Hi @nicolasfella, Ok, I need a new name, perhaps KGesture ? REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frameworks Cc: apol, nicolasfella, cfeck, kde-frameworks-devel, michaelh, ngraham, bruns

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-08 Thread Steffen Hartleib
steffenh added a comment. Hi @apol, Thanks for your time to look at my code. I know, I'am not good at this, so if you can point me at my errors I will try to correct this. INLINE COMMENTS > apol wrote in koneandtwofingerswipe.h:36 > Why is this commented out? Oops this should not be he

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-05 Thread Aleix Pol Gonzalez
apol added a comment. It could make sense to run clazy on these, I'm pretty sure we'd find interesting stuff. INLINE COMMENTS > koneandtwofingerswipe.h:36 > +{ > +//Q_PROPERTY(bool left READ left WRITE left) > +//Q_PROPERTY(bool right READ right WRITE right) Why is this commented ou

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-05 Thread Nicolas Fella
nicolasfella added a comment. The name "KTouch" is already used for https://kde.org/applications/education/ktouch/ REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frameworks Cc: nicolasfella, cfeck, kde-frameworks-devel, michaelh, ngraham

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-05 Thread Nathaniel Graham
ngraham added a reviewer: Frameworks. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh, #frameworks Cc: cfeck, kde-frameworks-devel, michaelh, ngraham, bruns

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-05 Thread Christoph Feck
cfeck added a comment. Did you try to submit the additional gesture types to Qt? We should only add them here if they rejected the idea to add new types in Qt. REPOSITORY R236 KWidgetsAddons REVISION DETAIL https://phabricator.kde.org/D21028 To: steffenh Cc: cfeck, kde-frameworks-devel,

D21028: add multiple gestures and a handler class to KWidgetsAddons

2019-05-04 Thread Steffen Hartleib
steffenh created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. steffenh requested review of this revision. REVISION SUMMARY This patch adds the gestures and handler class from gwenview to KWidgetsAddons, so another program you can use this.