D12835: Draw borders around side panels by default

[Nathaniel Graham]

ngraham added a subscriber: abetts.
ngraham added a comment.


  I wasn't around for the KDE4/oxygen days, so I wasn't aware this represented a reversion of anything; I just think this patch results in better visual design that the status quo.
  
  Breeze may be a minimalistic theme, but minimalism always rides a fine line. If you take away too much, you can reduce clarity, or muddy the boundaries between things that are different and separate. The status quo here has never sat well with me: the side panel feels like it isn't sure what it wants to be. Is its content a part of the main window, or separate? It feels like neither. And what's with that blue line that doesn't extend all the way to the top of the bottom? The design feels like an unhappy compromise, not something that's bold and proud (apologies for all the subjective touchy-feely language; this is just how it feels to me). At least with this patch there's a clear and confident visual separation of the category list and the content area, and now I realize that's what I'm looking for.
  
  As an alternative, perhaps we should instead move towards making the side panels (and dockable panels) look more like System Settings' sidebar: a white background, but without a distinct frame, and  a one-pixel line separating it from the content area that extends the full height of the window. This would Breezey and modern, and much better than either IMHO. We might even be able to remove the setting entirely...

REPOSITORY
  R31 Breeze

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

To: ngraham
Cc: abetts, rkflx, cfeck, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180512/abf3fbd1/attachment.html>