Merge lp:~dashua/light-themes/fix.lp.818541 into lp:~ubuntu-art-pkg/light-themes/trunk

Proposed by James Schriver
Status: Merged
Merged at revision: 154
Proposed branch: lp:~dashua/light-themes/fix.lp.818541
Merge into: lp:~ubuntu-art-pkg/light-themes/trunk
Diff against target: 53 lines (+24/-0)
3 files modified
Ambiance/gtk-3.0/gtk-widgets.css (+8/-0)
Radiance/gtk-3.0/gtk-widgets.css (+8/-0)
debian/changelog (+8/-0)
To merge this branch: bzr merge lp:~dashua/light-themes/fix.lp.818541
Reviewer Review Type Date Requested Status
Andrea Cimitan Pending
Review via email: mp+71490@code.launchpad.net

Description of the change

@Cimi-This looks quite nice on Radiance as well and globally themes the spinner. This also looks good good in Nautilus in the infobar. Fixes the dark spinner viability on Ambiance in the sound menu.

To post a comment you must log in.
142. By James Schriver

[ James Schriver ]
* Ambiance/Radiance/gtk-3.0/gtk-widgets.css (LP: #818541)
  - Theme spinner to @selected_bg color

Revision history for this message
Andrea Cimitan (cimi) wrote :

I think it should be better to use dark everywhere, except where we really want it

Revision history for this message
James Schriver (dashua) wrote :

The spinner also loads in Gwibber too and a few other places on the dark toolbars. I guess we could specifically theme the spinner for this instances, but I think it is going to show up more on dark backgrounds than light.

Revision history for this message
James Schriver (dashua) wrote :

Empathy too.

Revision history for this message
Andrea Cimitan (cimi) wrote :

So, match it with .menu GtkSpinner, .primary-toolbar GtkSpinner

Revision history for this message
James Schriver (dashua) wrote :

Ok

Revision history for this message
Andrea Cimitan (cimi) wrote :

well, it requires touches to indicator sound, I'll manage it, thx anyway

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Ambiance/gtk-3.0/gtk-widgets.css'
2--- Ambiance/gtk-3.0/gtk-widgets.css 2011-08-10 19:43:39 +0000
3+++ Ambiance/gtk-3.0/gtk-widgets.css 2011-08-14 16:41:23 +0000
4@@ -1393,6 +1393,14 @@
5 color: @fg_color;
6 }
7
8+/***********
9+ * spinner *
10+ ***********/
11+
12+GtkSpinner {
13+ color: @selected_bg_color;
14+}
15+
16 /*************
17 * statusbar *
18 *************/
19
20=== modified file 'Radiance/gtk-3.0/gtk-widgets.css'
21--- Radiance/gtk-3.0/gtk-widgets.css 2011-08-10 19:43:39 +0000
22+++ Radiance/gtk-3.0/gtk-widgets.css 2011-08-14 16:41:23 +0000
23@@ -1408,6 +1408,14 @@
24 color: @fg_color;
25 }
26
27+/***********
28+ * spinner *
29+ ***********/
30+
31+GtkSpinner {
32+ color: @selected_bg_color;
33+}
34+
35 /*************
36 * statusbar *
37 *************/
38
39=== modified file 'debian/changelog'
40--- debian/changelog 2011-08-11 01:45:10 +0000
41+++ debian/changelog 2011-08-14 16:41:23 +0000
42@@ -1,3 +1,11 @@
43+light-themes (0.1.8.22) UNRELEASED; urgency=low
44+
45+ [ James Schriver ]
46+ * Ambiance/Radiance/gtk-3.0/gtk-widgets.css (LP: #818541)
47+ - Theme spinner to @selected_bg color
48+
49+ -- James Schriver <jws141@gmail.com> Sun, 14 Aug 2011 12:28:00 -0400
50+
51 light-themes (0.1.8.21) oneiric; urgency=low
52
53 * New snapshot

Subscribers

People subscribed via source and target branches