Merge lp:~danielholm/music-app/resolve-missing-images-plus-new-ones into lp:music-app/trusty

Proposed by Daniel Holm
Status: Merged
Approved by: Victor Thompson
Approved revision: 40
Merged at revision: 40
Proposed branch: lp:~danielholm/music-app/resolve-missing-images-plus-new-ones
Merge into: lp:music-app/trusty
Diff against target: 49 lines (+1/-1)
1 file modified
MusicTracks.qml (+1/-1)
To merge this branch: bzr merge lp:~danielholm/music-app/resolve-missing-images-plus-new-ones
Reviewer Review Type Date Requested Status
Victor Thompson Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+173540@code.launchpad.net

Commit message

Resolve bug #1198889 and added some other new icons by Lucas.

Description of the change

Resolved bug #1198889 that images/icons for playlist and scrobble was missing. Also some new icons from Lucas to be used later, perhaps.

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

FAILED: Continuous integration, rev:39
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~danielholm/music-app/resolve-missing-images-plus-new-ones/+merge/173540/+edit-commit-message

http://91.189.93.70:8080/job/music-app-ci/2/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/music-app-quantal-amd64-ci/2
    SUCCESS: http://91.189.93.70:8080/job/music-app-raring-amd64-ci/2

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/music-app-ci/2/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Victor Thompson (vthompson) wrote :

Daniel, I'd like to fix 2 things. 1) Could you remove the link "settings@8.png" and 2) Could you commit the following change to change "settings@8.png" to "settings.png"?

bzr diff=== modified file 'MusicTracks.qml'
--- MusicTracks.qml 2013-07-04 07:25:42 +0000
+++ MusicTracks.qml 2013-07-09 00:18:24 +0000
@@ -60,7 +60,7 @@
             // Settings dialog
             ToolbarButton {
                 objectName: "settingsaction"
- iconSource: Qt.resolvedUrl("images/settings@8.png")
+ iconSource: Qt.resolvedUrl("images/settings.png")
                 text: i18n.tr("Settings")

                 onTriggered: {

Revision history for this message
Victor Thompson (vthompson) wrote :

Also, could you keep the existing "folder.png" and rename the one you propose to merge "music_folder.png"?

Revision history for this message
Daniel Holm (danielholm) wrote :

Sure. I'll fix it later on today.

40. By Daniel Holm

Remove link to old settings icon; take back previous dir icon, and removed default-album icon.

Revision history for this message
Daniel Holm (danielholm) wrote :

Done!

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 :

Thanks! Looks perfect!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'MusicTracks.qml'
2--- MusicTracks.qml 2013-07-04 07:25:42 +0000
3+++ MusicTracks.qml 2013-07-09 09:56:29 +0000
4@@ -60,7 +60,7 @@
5 // Settings dialog
6 ToolbarButton {
7 objectName: "settingsaction"
8- iconSource: Qt.resolvedUrl("images/settings@8.png")
9+ iconSource: Qt.resolvedUrl("images/settings.png")
10 text: i18n.tr("Settings")
11
12 onTriggered: {
13
14=== added file 'images/add-to-playback.png'
15Binary files images/add-to-playback.png 1970-01-01 00:00:00 +0000 and images/add-to-playback.png 2013-07-09 09:56:29 +0000 differ
16=== added file 'images/add-to-playlist.png'
17Binary files images/add-to-playlist.png 1970-01-01 00:00:00 +0000 and images/add-to-playlist.png 2013-07-09 09:56:29 +0000 differ
18=== added file 'images/clear.png'
19Binary files images/clear.png 1970-01-01 00:00:00 +0000 and images/clear.png 2013-07-09 09:56:29 +0000 differ
20=== removed file 'images/default-album.png'
21Binary files images/default-album.png 2013-05-08 13:13:37 +0000 and images/default-album.png 1970-01-01 00:00:00 +0000 differ
22=== added file 'images/favourite.png'
23Binary files images/favourite.png 1970-01-01 00:00:00 +0000 and images/favourite.png 2013-07-09 09:56:29 +0000 differ
24=== added file 'images/lastfm.png'
25Binary files images/lastfm.png 1970-01-01 00:00:00 +0000 and images/lastfm.png 2013-07-09 09:56:29 +0000 differ
26=== added file 'images/lastfm_color.png'
27Binary files images/lastfm_color.png 1970-01-01 00:00:00 +0000 and images/lastfm_color.png 2013-07-09 09:56:29 +0000 differ
28=== added file 'images/morebutton.png'
29Binary files images/morebutton.png 1970-01-01 00:00:00 +0000 and images/morebutton.png 2013-07-09 09:56:29 +0000 differ
30=== added file 'images/music_folder.png'
31Binary files images/music_folder.png 1970-01-01 00:00:00 +0000 and images/music_folder.png 2013-07-09 09:56:29 +0000 differ
32=== added file 'images/no-favourite.png'
33Binary files images/no-favourite.png 1970-01-01 00:00:00 +0000 and images/no-favourite.png 2013-07-09 09:56:29 +0000 differ
34=== added file 'images/playlist.png'
35Binary files images/playlist.png 1970-01-01 00:00:00 +0000 and images/playlist.png 2013-07-09 09:56:29 +0000 differ
36=== added file 'images/search.png'
37Binary files images/search.png 1970-01-01 00:00:00 +0000 and images/search.png 2013-07-09 09:56:29 +0000 differ
38=== added file 'images/select-press.png'
39Binary files images/select-press.png 1970-01-01 00:00:00 +0000 and images/select-press.png 2013-07-09 09:56:29 +0000 differ
40=== added file 'images/select.png'
41Binary files images/select.png 1970-01-01 00:00:00 +0000 and images/select.png 2013-07-09 09:56:29 +0000 differ
42=== added file 'images/settings.png'
43Binary files images/settings.png 1970-01-01 00:00:00 +0000 and images/settings.png 2013-07-09 09:56:29 +0000 differ
44=== removed file 'images/settings@8.png'
45Binary files images/settings@8.png 2013-06-04 07:04:01 +0000 and images/settings@8.png 1970-01-01 00:00:00 +0000 differ
46=== added file 'images/ubuntuonemusic.png'
47Binary files images/ubuntuonemusic.png 1970-01-01 00:00:00 +0000 and images/ubuntuonemusic.png 2013-07-09 09:56:29 +0000 differ
48=== added file 'images/wikipedia.png'
49Binary files images/wikipedia.png 1970-01-01 00:00:00 +0000 and images/wikipedia.png 2013-07-09 09:56:29 +0000 differ

Subscribers

People subscribed via source and target branches

to status/vote changes: