[Differential] [Commented On] D2173: WIP: New System Settings delegate design

[broulik (Kai Uwe Broulik)]

broulik added a comment.


  Pretty! Having the sub-categories shown by default and being able to navigate to them directly is a significant improvement over the old layout.
  
  The layout layout doesn't handle different font sizes very well, I run at 125 dpi with 1x device pixel ratio which causes the headings to touch the links and when it wraps it overlaps.F189183: Screenshot_20160720_094058.png <https://phabricator.kde.org/F189183> The delegates also have varying padding.F189196: Screenshot_20160720_094651.png <https://phabricator.kde.org/F189196>
  
  It doesn't work //at all// with Right-to-Left languages.F189186: Screenshot_20160720_094143.png <https://phabricator.kde.org/F189186>
  
  Unfortunately it's quite difficult to get the links to underline on hover (I tried) but it's pretty inconvenient that tapping outside of links but still in the QLabel eats the mouse event, so I can essentially only click either the icon or the headnig to enter the main category. Clicking where the mouse is in the screenshot does nothing:F189198: Screenshot_20160720_094916.png <https://phabricator.kde.org/F189198>
  
  Also, what's up with the slightly gray background which doesn't stretch all the way to the corners? F189191: Screenshot_20160720_094510.png <https://phabricator.kde.org/F189191> (Try looking at your screen at an angle to see the difference in color)
  
  Finally, as others have already noted it usually crashes on quit and clicking a sub-category brings up just this category with no way of switching to the its siblings other than going back to the overview.

REPOSITORY
  rSYSTEMSETTINGS System Settings

REVISION DETAIL
  https://phabricator.kde.org/D2173

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: broulik, colomar, jensreuterberg, ivan, mart, plasma-devel, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160720/81efa47d/attachment.html>