Merge lp:~artmello/gallery-app/gallery-app-create_album_after_confirm into lp:gallery-app

Proposed by Arthur Mello
Status: Merged
Approved by: Bill Filler
Approved revision: 1119
Merged at revision: 1119
Proposed branch: lp:~artmello/gallery-app/gallery-app-create_album_after_confirm
Merge into: lp:gallery-app
Diff against target: 25 lines (+3/-1)
2 files modified
rc/qml/AlbumEditor/AlbumEditor.qml (+3/-0)
rc/qml/AlbumsOverview.qml (+0/-1)
To merge this branch: bzr merge lp:~artmello/gallery-app/gallery-app-create_album_after_confirm
Reviewer Review Type Date Requested Status
Bill Filler (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+244371@code.launchpad.net

Commit message

Delay the adding of a new album until the editor is finished

Description of the change

Delay the adding of a new album until the editor is finished

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Bill Filler (bfiller) wrote :

tested, works

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'rc/qml/AlbumEditor/AlbumEditor.qml'
2--- rc/qml/AlbumEditor/AlbumEditor.qml 2014-10-17 07:40:40 +0000
3+++ rc/qml/AlbumEditor/AlbumEditor.qml 2014-12-11 01:44:40 +0000
4@@ -219,6 +219,9 @@
5 onHidden: {
6 if(album.newAlbum)
7 albumModel.destroyAlbum(album);
8+ else
9+ albumCollectionModel.addOrphan(album);
10+
11 overview.popPage();
12 albumEditor.closeRequested(albumEditor.album, true);
13 }
14
15=== modified file 'rc/qml/AlbumsOverview.qml'
16--- rc/qml/AlbumsOverview.qml 2014-12-09 02:56:17 +0000
17+++ rc/qml/AlbumsOverview.qml 2014-12-11 01:44:40 +0000
18@@ -237,7 +237,6 @@
19 var album = albumCollectionModel.createOrphan();
20 album.title = i18n.tr("New Photo Album");
21 album.subtitle = i18n.tr("Subtitle");
22- albumCollectionModel.addOrphan(album);
23
24 albumEditor.album = album;
25 overview.pushPage(albumEditor);

Subscribers

People subscribed via source and target branches