Merge lp:~dobey/unity-scope-click/less-attrs into lp:unity-scope-click/devel

Proposed by dobey
Status: Merged
Approved by: Alejandro J. Cura
Approved revision: 432
Merged at revision: 432
Proposed branch: lp:~dobey/unity-scope-click/less-attrs
Merge into: lp:unity-scope-click/devel
Prerequisite: lp:~dobey/unity-scope-click/fix-crossbuild
Diff against target: 13 lines (+1/-2)
1 file modified
scope/clickstore/store-query.cpp (+1/-2)
To merge this branch: bzr merge lp:~dobey/unity-scope-click/less-attrs
Reviewer Review Type Date Requested Status
Alejandro J. Cura (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+232623@code.launchpad.net

This proposal supersedes a proposal from 2014-08-28.

Commit message

Do not show the subtitle for scopes cards.
Only show the first attributes value (price) for scopes.

Description of the change

A screenshot of this fix is at https://launchpadlibrarian.net/183466031/scopes-less-attrs.png . The centered title text in the overlays is unrelated, and controlled by Unity8.

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
Alejandro J. Cura (alecu) wrote :

Much better.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'scope/clickstore/store-query.cpp'
--- scope/clickstore/store-query.cpp 2014-08-27 20:07:18 +0000
+++ scope/clickstore/store-query.cpp 2014-08-28 19:38:45 +0000
@@ -92,8 +92,7 @@
92 },92 },
93 "components" : {93 "components" : {
94 "title" : "title",94 "title" : "title",
95 "subtitle": "subtitle",95 "attributes": { "field": "attributes", "max-count": 1 },
96 "attributes": { "field": "attributes", "max-count": 4 },
97 "art" : {96 "art" : {
98 "field": "art",97 "field": "art",
99 "aspect-ratio": 0.5598 "aspect-ratio": 0.55

Subscribers

People subscribed via source and target branches

to all changes: