Merge lp://staging/~and471/light-themes/desktop-panels-929665 into lp://staging/~ubuntu-art-pkg/light-themes/trunk

Proposed by Andrew
Status: Merged
Merge reported by: Andrea Cimitan
Merged at revision: not available
Proposed branch: lp://staging/~and471/light-themes/desktop-panels-929665
Merge into: lp://staging/~ubuntu-art-pkg/light-themes/trunk
Diff against target: 139 lines (+28/-14)
4 files modified
Ambiance/gtk-3.0/apps/panel.css (+13/-6)
Ambiance/gtk-3.0/gtk.css (+1/-1)
Radiance/gtk-3.0/apps/panel.css (+13/-6)
Radiance/gtk-3.0/gtk.css (+1/-1)
To merge this branch: bzr merge lp://staging/~and471/light-themes/desktop-panels-929665
Reviewer Review Type Date Requested Status
Andrew (community) Needs Resubmitting
Andrea Cimitan Needs Fixing
Adolfo Jayme Barrientos (community) Approve
Review via email: mp+92327@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Adolfo Jayme Barrientos (fitojb) :
review: Approve
Revision history for this message
Andrea Cimitan (cimi) wrote :

Could you update with latest trunk?

review: Needs Fixing
Revision history for this message
Andrew (and471) wrote :
Download full text (5.3 KiB)

> Could you update with latest trunk?

yes sure, below is the bzr diff (use this as the patch, don't merge with the branch as it hasn't changed)

=== renamed file 'Ambiance/gtk-3.0/apps/gnome-panel.css' => 'Ambiance/gtk-3.0/apps/panel.css'
--- Ambiance/gtk-3.0/apps/gnome-panel.css 2012-04-29 13:37:13 +0000
+++ Ambiance/gtk-3.0/apps/panel.css 2012-04-29 20:03:01 +0000
@@ -1,10 +1,11 @@
-/* Fallback Mode Panel */
+/* Fallback Mode GNOME Panel and other Desktop Panels */
 PanelWidget :backdrop,
 PanelApplet :backdrop,
 PanelToplevel :backdrop,
 PanelWidget,
 PanelApplet,
-PanelToplevel {
+PanelToplevel,
+.desktop-panel {
     background-image: -gtk-gradient (linear, left top, left bottom,
                                      from (shade (@dark_bg_color, 1.5)),
                                      to (shade (@dark_bg_color, 1.05)));
@@ -31,7 +32,9 @@
 PanelApplet > GtkMenuBar.menubar,
 PanelApplet > GtkMenuBar.menubar.menuitem,
 PanelMenuBar.menubar,
-PanelMenuBar.menubar.menuitem {
+PanelMenuBar.menubar.menuitem,
+.desktop-panel .menubar,
+.desktop-panel .menubar .menuitem {
     -PanelMenuBar-icon-visible: true;
     background-image: -gtk-gradient (linear, left top, left bottom,
                                      from (shade (@dark_bg_color, 1.5)),
@@ -47,7 +50,8 @@
 }

 PanelApplet .button:backdrop,
-PanelApplet .button {
+PanelApplet .button,
+.desktop-panel .button {
     -GtkButton-inner-border: 2;

     background-image: -gtk-gradient (linear, left top, left bottom,
@@ -70,7 +74,9 @@
 PanelApplet .button:active:prelight:backdrop,
 PanelApplet .button:active:prelight,
 PanelApplet .button:active:backdrop,
-PanelApplet .button:active {
+PanelApplet .button:active,
+.desktop-panel .button:active:prelight,
+.desktop-panel .button:active {
     background-image: -gtk-gradient (linear, left top, left bottom,
                                      from (shade (@dark_bg_color, 0.85)),
                                      to (shade (@dark_bg_color, 1.0)));
@@ -78,7 +84,8 @@
 }

 PanelApplet .button:prelight:backdrop,
-PanelApplet .button:prelight {
+PanelApplet .button:prelight,
+.desktop-panel .button:prelight {
     background-image: -gtk-gradient (linear, left top, left bottom,
                                      from (shade (@dark_bg_color, 1.7)),
                                      to (shade (@dark_bg_color, 1.25)));

=== modified file 'Ambiance/gtk-3.0/gtk.css'
--- Ambiance/gtk-3.0/gtk.css 2012-03-30 16:29:46 +0000
+++ Ambiance/gtk-3.0/gtk.css 2012-04-29 19:58:43 +0000
@@ -33,7 +33,7 @@
 @import url("gtk-widgets.css");
 @import url("gtk-widgets-backdrop.css");
 @import url("apps/gedit.css");
-@import url("apps/gnome-panel.css");
+@import url("apps/panel.css");
 @import url("apps/gnome-terminal.css");
 @import url("apps/nautilus.css");
 @import url("apps/unity.css");

=== renamed file 'Radiance/gtk-3.0/apps/gnome-panel.css' => 'Radiance/gtk-3.0/apps/panel.css'
--- Radiance/gtk-3.0/apps/gnome-panel.css 2012-04-29 13:37:13 +0000
+++ Radiance/gtk-3.0/apps/panel.css 2012-04-29 20:02:52 +0000
@@ -1,10 +1,11 @@
-/* Fallback Mode Panel */
+/* Fallback Mode GNOME Panel and other Desktop Panels */
 PanelWidget :backdrop,
 PanelApplet :ba...

Read more...

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

push to another branch dude! :-)

Revision history for this message
Andrew (and471) wrote :

Okay I have now pushed to lp:~and471/+junk/desktop-panels but I do wonder why you are making it a little difficult and frustrating to contribute...

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

That's for you Andrew! :-)
Because when you work on separate branches you get proper credits!

Look for example here, branches merged and tracked. https://code.launchpad.net/~unity-team/unity/trunk

Revision history for this message
Andrew (and471) wrote :

Oh okay...I guess I was a bit grumpy this morning :) thanks for the merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches