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
1=== modified file 'scope/clickstore/store-query.cpp'
2--- scope/clickstore/store-query.cpp 2014-08-27 20:07:18 +0000
3+++ scope/clickstore/store-query.cpp 2014-08-28 19:38:45 +0000
4@@ -92,8 +92,7 @@
5 },
6 "components" : {
7 "title" : "title",
8- "subtitle": "subtitle",
9- "attributes": { "field": "attributes", "max-count": 4 },
10+ "attributes": { "field": "attributes", "max-count": 1 },
11 "art" : {
12 "field": "art",
13 "aspect-ratio": 0.55

Subscribers

People subscribed via source and target branches

to all changes: