Merge lp:~macslow/unity/unity.fix-913717 into lp:unity

Proposed by Mirco Müller
Status: Merged
Approved by: Mirco Müller
Approved revision: no longer in the source branch.
Merged at revision: 1828
Proposed branch: lp:~macslow/unity/unity.fix-913717
Merge into: lp:unity
Diff against target: 19 lines (+2/-3)
1 file modified
plugins/unityshell/src/DashSearchBar.cpp (+2/-3)
To merge this branch: bzr merge lp:~macslow/unity/unity.fix-913717
Reviewer Review Type Date Requested Status
John Lea (community) design Approve
Andrea Cimitan (community) Approve
Andrea Azzarone (community) Approve
Review via email: mp+88314@code.launchpad.net

Description of the change

Changed rendering of filling of Dash's search-bar according to design-suggestion... http://people.canonical.com/~mmueller/fix-913717.png

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

+1

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

Is it using the same values as design? It seems pretty close, difficult to test with different backgrounds and dash background color.
If so, +1 for me

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

Yep, just read the bugreport.

Revision history for this message
John Lea (johnlea) wrote :

Looks great, matches visual design, many thanks!

review: Approve (design)
Revision history for this message
Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-unity/115/console reported an error when processing this lp:~macslow/unity/unity.fix-913717 branch.
Not merging it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/unityshell/src/DashSearchBar.cpp'
2--- plugins/unityshell/src/DashSearchBar.cpp 2012-01-02 19:26:26 +0000
3+++ plugins/unityshell/src/DashSearchBar.cpp 2012-01-12 08:53:24 +0000
4@@ -319,13 +319,12 @@
5 cairo_operator_t op = CAIRO_OPERATOR_OVER;
6 op = cairo_get_operator (cr);
7 cairo_set_operator (cr, CAIRO_OPERATOR_OVER);
8- cairo_set_source_rgba(cr, 0.0f, 0.0f, 0.0f, 0.5f);
9+ cairo_set_source_rgba(cr, 0.0f, 0.0f, 0.0f, 0.35f);
10 cairo_fill_preserve(cr);
11 cairo_set_operator (cr, op);
12- cairo_set_source_rgba(cr, 0.0f, 0.0f, 0.0f, 0.5f);
13+ cairo_set_source_rgba(cr, 0.0f, 0.0f, 0.0f, 0.35f);
14 cairo_fill_preserve(cr);
15 cairo_set_source_rgba(cr, 1.0f, 1.0f, 1.0f, 0.8f);
16- //cairo_set_line_width(cr, 1.0);
17 cairo_stroke(cr);
18
19 cairo_destroy(cr);