D5111: Provide demo/preview for checkable menu items and colour scheme comparison
Hugo Pereira Da Costa
noreply at phabricator.kde.org
Mon Mar 20 12:00:19 UTC 2017
hpereiradacosta added a comment.
Hi,
Thanks for the set of patches.
in general, i am ok with the change but:
- please re-add the screenshot from Review Board. (sorry I was not aware of this review request cause I was not in the list of reviewers, even though official maintainer of oxygen ...)
- right to left layout action <- no. There is already one at the bottom of the window.
- this review should really be several, one per feature: one for the checkboxes/radiobuttons in the mdi window, one for the colorschemechooser. Can you split ?
- finally, there is need for more detail review (once above is done). For instance, in ColorSchemeChooser you use SUPPORT_THEME_SAVING, but this one is set/defined nowhere. So the whole corresponding code should go, right ? Or is it work in progress ? Personally I would disagree with having oxygen-demo being anything other than a demo, and for instance altering configuration. This is not the right place. The right place is the relevant KCM dialog.
Best,
Hugo
REPOSITORY
R113 Oxygen Theme
REVISION DETAIL
https://phabricator.kde.org/D5111
To: rjvbb, jriddell, anthonyfieroni, zhigalin, hpereiradacosta
Cc: kde-mac, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170320/f021636f/attachment.html>
More information about the Kde-frameworks-devel
mailing list