Merge lp:~vthompson/music-app/remix-songs-page-elide into lp:music-app/remix

Proposed by Victor Thompson
Status: Merged
Approved by: Andrew Hayzen
Approved revision: 649
Merged at revision: 651
Proposed branch: lp:~vthompson/music-app/remix-songs-page-elide
Merge into: lp:music-app/remix
Diff against target: 52 lines (+9/-2)
2 files modified
common/MusicRow.qml (+2/-1)
common/SongsPage.qml (+7/-1)
To merge this branch: bzr merge lp:~vthompson/music-app/remix-songs-page-elide
Reviewer Review Type Date Requested Status
Andrew Hayzen Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+237180@code.launchpad.net

Commit message

* Fix elide in Songs Page header
* Fix elide in MusicRow for Songs Page items

Description of the change

This change fixes the elide on the Songs Page, which is a bit off at the moment.

Before:
http://i.imgur.com/5zqTLGn.png
http://i.imgur.com/Getf5Tp.png

After:
http://i.imgur.com/riIDA3A.png
http://i.imgur.com/41EeEjH.png

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

LGTM :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'common/MusicRow.qml'
2--- common/MusicRow.qml 2014-10-02 00:49:08 +0000
3+++ common/MusicRow.qml 2014-10-05 06:14:12 +0000
4@@ -54,7 +54,8 @@
5 top: parent.top
6 topMargin: units.gu(1)
7 }
8- width: parent.width - coverRow.width - parent.spacing
9+ width: showCovers ? parent.width - coverRow.width - parent.spacing
10+ : parent.width - parent.spacing
11
12 onSourceComponentChanged: {
13 for (var i=0; i < item.children.length; i++) {
14
15=== modified file 'common/SongsPage.qml'
16--- common/SongsPage.qml 2014-10-04 01:41:50 +0000
17+++ common/SongsPage.qml 2014-10-05 06:14:12 +0000
18@@ -90,13 +90,15 @@
19 Label {
20 id: albumLabel
21 wrapMode: Text.NoWrap
22- maximumLineCount: 2
23+ maximumLineCount: 1
24 fontSize: "x-large"
25 color: styleMusic.common.music
26 anchors {
27 top: albumImage.bottom
28 topMargin: units.gu(1)
29 left: albumImage.left
30+ right: parent.right
31+ rightMargin: units.gu(2)
32 }
33 elide: Text.ElideRight
34 text: line2
35@@ -115,6 +117,8 @@
36 top: albumLabel.bottom
37 topMargin: units.gu(0.75)
38 left: albumImage.left
39+ right: parent.right
40+ rightMargin: units.gu(2)
41 }
42 elide: Text.ElideRight
43 text: line1
44@@ -131,6 +135,8 @@
45 : albumLabel.bottom
46 topMargin: units.gu(1)
47 left: albumImage.left
48+ right: parent.right
49+ rightMargin: units.gu(2)
50 }
51 elide: Text.ElideRight
52 text: isAlbum && line1 !== i18n.tr("Genre") ? year + " | " + i18n.tr("%1 song", "%1 songs", albumtrackslist.count).arg(albumtrackslist.count)

Subscribers

People subscribed via source and target branches