Merge lp:~ahayzen/music-app/remix-perf-002 into lp:music-app/remix

Proposed by Andrew Hayzen
Status: Merged
Approved by: Victor Thompson
Approved revision: 692
Merged at revision: 693
Proposed branch: lp:~ahayzen/music-app/remix-perf-002
Merge into: lp:music-app/remix
Diff against target: 57 lines (+7/-0)
4 files modified
common/BlurredHeader.qml (+2/-0)
common/CardView.qml (+2/-0)
common/ListItemWithActions.qml (+2/-0)
common/MusicRow.qml (+1/-0)
To merge this branch: bzr merge lp:~ahayzen/music-app/remix-perf-002
Reviewer Review Type Date Requested Status
Victor Thompson Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+239243@code.launchpad.net

Commit message

* Use async for loaders and unused loaders invisible

Description of the change

* Use async for loaders and unused loaders invisible

To post a comment you must log in.
lp:~ahayzen/music-app/remix-perf-002 updated
692. By Andrew Hayzen

* Add additional async

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
Victor Thompson (vthompson) wrote :

LGTM, we will want to profile the improvements this makes as well.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'common/BlurredHeader.qml'
2--- common/BlurredHeader.qml 2014-10-20 15:39:09 +0000
3+++ common/BlurredHeader.qml 2014-10-22 17:39:14 +0000
4@@ -54,6 +54,7 @@
5 left: coversImage.right
6 leftMargin: units.gu(2)
7 }
8+ asynchronous: true
9 }
10
11 Loader {
12@@ -65,5 +66,6 @@
13 top: coversImage.bottom
14 topMargin: units.gu(1)
15 }
16+ asynchronous: true
17 }
18 }
19
20=== modified file 'common/CardView.qml'
21--- common/CardView.qml 2014-10-14 15:46:07 +0000
22+++ common/CardView.qml 2014-10-22 17:39:14 +0000
23@@ -37,6 +37,8 @@
24
25 Loader {
26 id: headerLoader
27+ asynchronous: true
28+ visible: sourceComponent !== undefined
29 width: parent.width
30 }
31
32
33=== modified file 'common/ListItemWithActions.qml'
34--- common/ListItemWithActions.qml 2014-10-21 21:12:55 +0000
35+++ common/ListItemWithActions.qml 2014-10-22 17:39:14 +0000
36@@ -449,8 +449,10 @@
37 left: main.left
38 verticalCenter: main.verticalCenter
39 }
40+ asynchronous: true // CUSTOM
41 width: (status === Loader.Ready) ? item.implicitWidth : 0
42 visible: (status === Loader.Ready) && (item.width === item.implicitWidth)
43+
44 Behavior on width {
45 NumberAnimation {
46 duration: UbuntuAnimation.SnapDuration
47
48=== modified file 'common/MusicRow.qml'
49--- common/MusicRow.qml 2014-10-20 20:51:31 +0000
50+++ common/MusicRow.qml 2014-10-22 17:39:14 +0000
51@@ -55,6 +55,7 @@
52 top: parent.top
53 topMargin: units.gu(1)
54 }
55+ asynchronous: true
56 width: !showCovers ? parent.width - parent.spacing
57 : parent.width - coverGrid.width - parent.spacing
58

Subscribers

People subscribed via source and target branches