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