Merge lp:~macslow/ubuntu-ui-toolkit/green-darkgrey-gradient into lp:ubuntu-ui-toolkit

Proposed by Mirco Müller
Status: Rejected
Rejected by: Cris Dywan
Proposed branch: lp:~macslow/ubuntu-ui-toolkit/green-darkgrey-gradient
Merge into: lp:ubuntu-ui-toolkit
Diff against target: 26 lines (+18/-0)
1 file modified
modules/Ubuntu/Components/Colors/UbuntuColors.qml (+18/-0)
To merge this branch: bzr merge lp:~macslow/ubuntu-ui-toolkit/green-darkgrey-gradient
Reviewer Review Type Date Requested Status
Cris Dywan Disapprove
Tim Peeters Needs Resubmitting
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+221035@code.launchpad.net

Description of the change

As part of the "Compact widget patterns review" from January 2014, buttons (on snap-decisions and other elements) are meant to use a green-gradient and a dark-grey gradient. This branch adds these needed gradients as symbolic gradients for easy access via the SDK.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

Can you double-check this with design? I heard last week that they want to get rid of the gradients in UbuntuColors.

Second, we don't merge directly to lp:ubuntu-ui-toolkit any more, it first needs to go to lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/staging, so please resubmit the MR for merging to that branch.

review: Needs Resubmitting
Revision history for this message
Cris Dywan (kalikiana) wrote :

I confirm we don't want gradients, in fact I'm going to have a branch transparently turning gradients into plain colors.

review: Disapprove
Revision history for this message
Cris Dywan (kalikiana) wrote :
Revision history for this message
Mirco Müller (macslow) wrote :

Hm... ok. I'm still a bit surprised by this change in direction, because I talked with Jouni - who's with the Design-team - last week in Malta and he did not object to the new gradients.

Unmerged revisions

1025. By Mirco Müller

Added green and dark-grey gradients to allow easy access to the new color- and gradient-styles set forth by the compact widgets document from January 2014.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'modules/Ubuntu/Components/Colors/UbuntuColors.qml'
2--- modules/Ubuntu/Components/Colors/UbuntuColors.qml 2013-07-02 00:15:07 +0000
3+++ modules/Ubuntu/Components/Colors/UbuntuColors.qml 2014-05-27 07:58:10 +0000
4@@ -79,4 +79,22 @@
5 GradientStop { position: 0.0; color: "#888888" }
6 GradientStop { position: 1.0; color: "#BBBBBB" }
7 }
8+ /*!
9+ Ubuntu dark-grey gradient
10+
11+ \sa Button::gradient
12+ */
13+ property Gradient darkgreyGradient: Gradient {
14+ GradientStop { position: 0.0; color: "#534E4B" }
15+ GradientStop { position: 1.0; color: "#5D5856" }
16+ }
17+ /*!
18+ Ubuntu green gradient
19+
20+ \sa Button::gradient
21+ */
22+ property Gradient greenGradient: Gradient {
23+ GradientStop { position: 0.0; color: "#39B44A" }
24+ GradientStop { position: 1.0; color: "#50BE61" }
25+ }
26 }

Subscribers

People subscribed via source and target branches

to status/vote changes: