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