Merge lp:~desrt/ubuntu-themes/gtk3.8-fixes into lp:ubuntu-themes

Proposed by Allison Karlitskaya
Status: Merged
Approved by: Allison Karlitskaya
Approved revision: 276
Merged at revision: 275
Proposed branch: lp:~desrt/ubuntu-themes/gtk3.8-fixes
Merge into: lp:ubuntu-themes
Diff against target: 35 lines (+6/-6)
2 files modified
Ambiance/gtk-3.0/gtk-widgets.css (+3/-3)
Radiance/gtk-3.0/gtk-widgets.css (+3/-3)
To merge this branch: bzr merge lp:~desrt/ubuntu-themes/gtk3.8-fixes
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+149274@code.launchpad.net

Description of the change

Fix a few issues caused by Gtk 3.8's improved CSS support

We were setting a solid border around menu items (for debugging?) that
wasn't being rendered by old Gtk versions and now is. Similar for
background colour. Also: Gtk used to automatically add 3 to the padding
around menuitems and it no longer does that, so increase our padding
(but turn off the old implicit padding to maintain compatibility with
Gtk 3.6).

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Approve

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Ambiance/gtk-3.0/gtk-widgets.css'
--- Ambiance/gtk-3.0/gtk-widgets.css 2013-02-10 12:59:27 +0000
+++ Ambiance/gtk-3.0/gtk-widgets.css 2013-02-19 14:03:23 +0000
@@ -956,10 +956,10 @@
956}*/956}*/
957957
958.menuitem {958.menuitem {
959 -GtkMenuItem-horizontal-padding: 0;
960 background: transparent;
959 border-radius: 0;961 border-radius: 0;
960 border-style: solid;962 padding: 3px 5px 3px 5px;
961 border-width: 1px;
962 padding: 3px 2px 3px 2px;
963963
964 text-shadow: none;964 text-shadow: none;
965}965}
966966
=== modified file 'Radiance/gtk-3.0/gtk-widgets.css'
--- Radiance/gtk-3.0/gtk-widgets.css 2013-02-10 12:59:27 +0000
+++ Radiance/gtk-3.0/gtk-widgets.css 2013-02-19 14:03:23 +0000
@@ -956,10 +956,10 @@
956}*/956}*/
957957
958.menuitem {958.menuitem {
959 -GtkMenuItem-horizontal-padding: 0;
960 background: transparent;
959 border-radius: 0;961 border-radius: 0;
960 border-style: solid;962 padding: 3px 5px 3px 5px;
961 border-width: 1px;
962 padding: 3px 2px 3px 2px;
963963
964 text-shadow: none;964 text-shadow: none;
965}965}

Subscribers

People subscribed via source and target branches