Merge lp:~aacid/unity-scope-click/no_overlay_color into lp:unity-scope-click

Proposed by Albert Astals Cid
Status: Merged
Approved by: dobey
Approved revision: 383
Merged at revision: 383
Proposed branch: lp:~aacid/unity-scope-click/no_overlay_color
Merge into: lp:unity-scope-click
Diff against target: 20 lines (+3/-1)
1 file modified
scope/clickapps/apps-query.cpp (+3/-1)
To merge this branch: bzr merge lp:~aacid/unity-scope-click/no_overlay_color
Reviewer Review Type Date Requested Status
dobey (community) Approve
Michał Sawicz Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+276506@code.launchpad.net

Commit message

Set the overlay color to transparent

We just want the text, not the gray-ish overlay

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Yes!

review: Approve
Revision history for this message
dobey (dobey) wrote :

Is there a bug for this? I don't quite understand what this does exactly. Also, if you want this released on the phone, you'll need a separate MP into lp:unity-scope-click/touch-15-04 as dual landings aren't used for unity-scope-click.

review: Needs Information
Revision history for this message
Albert Astals Cid (aacid) wrote :

> Is there a bug for this? I don't quite understand what this does exactly.
> Also, if you want this released on the phone, you'll need a separate MP into
> lp:unity-scope-click/touch-15-04 as dual landings aren't used for unity-scope-
> click.

As explained on IRC new unity8 fixes a bug you were "exploiting" to get no background on the overlay, now (with silo 21) to get no background on the overlay we need to properly set it to transparent

Revision history for this message
Albert Astals Cid (aacid) wrote :
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scope/clickapps/apps-query.cpp'
2--- scope/clickapps/apps-query.cpp 2014-08-19 21:20:18 +0000
3+++ scope/clickapps/apps-query.cpp 2015-11-03 10:42:38 +0000
4@@ -81,7 +81,8 @@
5 "art": {
6 "aspect-ratio": 0.55,
7 "field": "art"
8- }
9+ },
10+ "overlay-color": "overlay-color"
11 }
12 }
13
14@@ -289,6 +290,7 @@
15 res[click::apps::Query::ResultKeys::MAIN_SCREENSHOT] = "";
16 res[click::apps::Query::ResultKeys::INSTALLED] = true;
17 res[click::apps::Query::ResultKeys::VERSION] = "";
18+ res["overlay-color"] = "transparent";
19 searchReply->push(res);
20 }
21

Subscribers

People subscribed via source and target branches