Ubuntu One Control Panel

Merge lp:~dobey/ubuntuone-control-panel/the-darkness into lp:ubuntuone-control-panel

  1. the-darkness
  2. Merge into trunk
Proposed by dobey
Status: Rejected
Rejected by: Natalia Bidart
Proposed branch: lp:~dobey/ubuntuone-control-panel/the-darkness
Merge into: lp:ubuntuone-control-panel
Diff against target: 10 lines (+1/-1)
1 file modified
data/qt/ubuntuone.qss (+1/-1)
To merge this branch: bzr merge lp:~dobey/ubuntuone-control-panel/the-darkness
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Disapprove
Review via email: mp+95239@code.launchpad.net

Commit message

Change the background color of main window to match Ubuntu title bars.

To post a comment you must log in.
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Can we please use the Ubuntu dark grey instead of the one proposed?

Thanks!

review: Needs Fixing
lp:~dobey/ubuntuone-control-panel/the-darkness updated
267. By dobey

Change to "cool grey" which isn't very cool, or grey, from Ubuntu palette

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

So, latest executive decision is to maintain the light gray. Marking this as disapprove.

Sorry :-/

review: Disapprove

Unmerged revisions

267. By dobey

Change to "cool grey" which isn't very cool, or grey, from Ubuntu palette

266. By dobey

Change the background color of main window to match Ubuntu title bars.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/qt/ubuntuone.qss'
2--- data/qt/ubuntuone.qss 2011-11-23 17:22:33 +0000
3+++ data/qt/ubuntuone.qss 2012-02-29 19:35:20 +0000
4@@ -1,5 +1,5 @@
5 QMainWindow {
6- background-color: #aea79f;
7+ background-color: #333333;
8 }
9
10 QWidget {

Subscribers

People subscribed via source and target branches

to all changes: