Merge lp:~vthompson/music-app/fix-blank-cover-art-and-misc into lp:music-app/trusty

Proposed by Victor Thompson
Status: Merged
Approved by: Daniel Holm
Approved revision: 34
Merged at revision: 33
Proposed branch: lp:~vthompson/music-app/fix-blank-cover-art-and-misc
Merge into: lp:music-app/trusty
Diff against target: 72 lines (+10/-5)
4 files modified
LibraryLoader.qml (+7/-2)
MusicTracks.qml (+1/-1)
debian/changelog (+1/-1)
music-app.qml (+1/-1)
To merge this branch: bzr merge lp:~vthompson/music-app/fix-blank-cover-art-and-misc
Reviewer Review Type Date Requested Status
Daniel Holm Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+170721@code.launchpad.net

Commit message

Changes:
* Fix icons and images when cover art is not available.
* Fix issue with scrobble that prevented the next track from being played.
* Place the current path at the bottom of the screen in the LibraryLoader so it is always visible.

Description of the change

Changes:
* Fix icons and images when cover art is not available.
* Fix issue with scrobble that prevented the next track from being played.
* Place the current path at the bottom of the screen in the LibraryLoader so it is always visible.

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)
34. By Victor Thompson

for some reason extracting deb files do not like invalid email addresses.

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
Daniel Holm (danielholm) wrote :

It looks great.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'LibraryLoader.qml'
--- LibraryLoader.qml 2013-06-16 17:09:19 +0000
+++ LibraryLoader.qml 2013-06-21 00:55:29 +0000
@@ -100,7 +100,6 @@
100 ListView {100 ListView {
101 id: folderSelecterList101 id: folderSelecterList
102 anchors.fill: parent102 anchors.fill: parent
103 anchors.topMargin: units.gu(3)
104103
105 model: folderModel104 model: folderModel
106 delegate: folderSelecterDelegate105 delegate: folderSelecterDelegate
@@ -178,7 +177,11 @@
178 Component.onCompleted: {177 Component.onCompleted: {
179 if (!model.isDir) {178 if (!model.isDir) {
180 console.log("Debug: Scanner fileDelegate onComplete")179 console.log("Debug: Scanner fileDelegate onComplete")
181 Library.setMetadata(filePath, trackTitle, trackArtist, trackAlbum, "image://cover-art/" + filePath, trackYear, trackNumber, trackLength)180 if ("" === trackCover) {
181 Library.setMetadata(filePath, trackTitle, trackArtist, trackAlbum, "", trackYear, trackNumber, trackLength)
182 } else {
183 Library.setMetadata(filePath, trackTitle, trackArtist, trackAlbum, "image://cover-art/" + filePath, trackYear, trackNumber, trackLength)
184 }
182 }185 }
183 }186 }
184 }187 }
@@ -189,6 +192,8 @@
189 Rectangle {192 Rectangle {
190 width: parent.width193 width: parent.width
191 height: units.gu(3)194 height: units.gu(3)
195 anchors.bottom: parent.bottom
196 anchors.bottomMargin: units.gu(8)
192 Label {197 Label {
193 id: currentpath198 id: currentpath
194 text: folderModel.path199 text: folderModel.path
195200
=== modified file 'MusicTracks.qml'
--- MusicTracks.qml 2013-06-19 12:08:44 +0000
+++ MusicTracks.qml 2013-06-21 00:55:29 +0000
@@ -177,7 +177,7 @@
177 property string cover: model.cover177 property string cover: model.cover
178 property string length: model.length178 property string length: model.length
179 property string file: model.file179 property string file: model.file
180 icon: cover === "" ? (file.match("\\.mp3") ? Qt.resolvedUrl("images/audio-x-mpeg.png") : Qt.resolvedUrl("images/audio-x-vorbis+ogg.png")) : "image://cover-art/"+file180 icon: track.cover === "" ? (track.file.match("\\.mp3") ? Qt.resolvedUrl("images/audio-x-mpeg.png") : Qt.resolvedUrl("images/audio-x-vorbis+ogg.png")) : "image://cover-art/"+file
181 iconFrame: false181 iconFrame: false
182 Label {182 Label {
183 id: trackTitle183 id: trackTitle
184184
=== modified file 'debian/changelog'
--- debian/changelog 2013-06-18 12:49:22 +0000
+++ debian/changelog 2013-06-21 00:55:29 +0000
@@ -21,7 +21,7 @@
2121
22 * Changing name of application to ubuntu-music-app22 * Changing name of application to ubuntu-music-app
2323
24 -- Daniel Holm <daniel@daniel-laptop> Tue, 04 Jun 2013 16:43:32 +020024 -- Daniel Holm <d.holmen@gmail.com> Tue, 04 Jun 2013 16:43:32 +0200
2525
26music-app (0.2) raring; urgency=low26music-app (0.2) raring; urgency=low
2727
2828
=== modified file 'music-app.qml'
--- music-app.qml 2013-06-19 12:08:44 +0000
+++ music-app.qml 2013-06-21 00:55:29 +0000
@@ -132,7 +132,7 @@
132 if (status == MediaPlayer.EndOfMedia) {132 if (status == MediaPlayer.EndOfMedia) {
133 // scrobble it133 // scrobble it
134 if (Settings.getSetting("scrobble") === "1") {134 if (Settings.getSetting("scrobble") === "1") {
135 scrobblemodel.scrobble(player.source,artist,timestamp)135 Scrobble.scrobble(player.source,artist,timestamp)
136 }136 }
137 else {137 else {
138 console.debug("Debug: no scrobbling")138 console.debug("Debug: no scrobbling")

Subscribers

People subscribed via source and target branches

to status/vote changes: