unity-scope-click

Merge lp:~aacid/unity-scope-click/no_overlay_color_1504 into lp:unity-scope-click/touch-15-04

  1. no_overlay_color_1504
  2. Merge into touch-15-04
Proposed by Albert Astals Cid
Status: Merged
Approved by: dobey
Approved revision: 338
Merged at revision: 338
Proposed branch: lp:~aacid/unity-scope-click/no_overlay_color_1504
Merge into: lp:unity-scope-click/touch-15-04
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_1504
Reviewer Review Type Date Requested Status
dobey (community) Approve
Michał Sawicz Approve
Review via email: mp+276548@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
Michał Sawicz (saviq) wrote :

Yes please.

review: Approve
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
=== modified file 'scope/clickapps/apps-query.cpp'
--- scope/clickapps/apps-query.cpp 2014-08-19 21:20:18 +0000
+++ scope/clickapps/apps-query.cpp 2015-11-03 15:21:55 +0000
@@ -81,7 +81,8 @@
81 "art": {81 "art": {
82 "aspect-ratio": 0.55,82 "aspect-ratio": 0.55,
83 "field": "art"83 "field": "art"
84 }84 },
85 "overlay-color": "overlay-color"
85 }86 }
86}87}
8788
@@ -289,6 +290,7 @@
289 res[click::apps::Query::ResultKeys::MAIN_SCREENSHOT] = "";290 res[click::apps::Query::ResultKeys::MAIN_SCREENSHOT] = "";
290 res[click::apps::Query::ResultKeys::INSTALLED] = true;291 res[click::apps::Query::ResultKeys::INSTALLED] = true;
291 res[click::apps::Query::ResultKeys::VERSION] = "";292 res[click::apps::Query::ResultKeys::VERSION] = "";
293 res["overlay-color"] = "transparent";
292 searchReply->push(res);294 searchReply->push(res);
293}295}
294296

Subscribers

People subscribed via source and target branches