[Differential] [Commented On] D3578: [RFC] Implement drag from free space also for QtQuickControls
hpereiradacosta (Hugo Pereira Da Costa)
noreply at phabricator.kde.org
Tue Dec 6 14:25:07 UTC 2016
hpereiradacosta added a comment.
Thanks for the updated patch !
I tested it compiles on kde4
Made a few suggestions above, then it can go.
INLINE COMMENTS
> breezewindowmanager.cpp:467
> +
> + return true;
> + }
should be "return false"
Comments few lines below says we never eat events.
> breezewindowmanager.h:176
> +#if QT_VERSION >= 0x050000
> +#define WINDOW QWindow
> +#else
I'd rather use typedef, or even better
"using Window = QWindow;"
and "using Window = QWidget;"
instead of preprocessors
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D3578
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: drosca, #plasma, hpereiradacosta
Cc: mart, graesslin, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161206/ea5e6868/attachment-0001.html>
More information about the Plasma-devel
mailing list