D15786: share common values for both Breeze and Breeze-dark GTK themes

mat gr noreply at phabricator.kde.org
Wed Sep 26 23:34:59 BST 2018


grmat created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
grmat requested review of this revision.

REVISION SUMMARY
  As described in bug #396091, the Breeze-dark theme is often neglected in updates. This patch fixes #396091 and some additional, related inconsistencies, enables sharing the basic stuff like shape, size, style, etc. of components in a common single css file while keeping the colour definitions separated from that in different files for both the light and dark theme.
  This allows easier changes, easier maintenance because of less LOC, less duplicate code as well as easier extensibility for potential additional colour schemes like light <https://hig.kde.org/style/color/light.html> or high-contrast <https://hig.kde.org/style/color/high.html>.
  
  I only did this for GTK 3.20 for now. If you like the effort, it could (and maybe should) get extended to other versions.
  
  Further potential steps in the same direction of saving code would be going back to SASS, which is used by many other popular GTK themes and was used by Breeze-gtk as well in the past. The only downside of that is that the SASS source files would have to be "compiled" to CSS prior to packaging.

TEST PLAN
  I don't know of any automated test suites comparing UI components. I have already posted screenshots in the original bug report and can post additional ones of the new state for comparison. Apart from that, my manual sparse testing makes me feel the dark GTK theme looks and feels much more consistent to Breeze now.

REPOSITORY
  R98 Breeze for Gtk

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

AFFECTED FILES
  Breeze-dark-gtk/gtk-3.20/gtk.css
  Breeze-gtk/gtk-3.20/common.css
  Breeze-gtk/gtk-3.20/gtk.css

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


More information about the Plasma-devel mailing list