Merge lp:~renatofilho/address-book-app/remove-contact-view-custom-back-action into lp:address-book-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 551
Merged at revision: 558
Proposed branch: lp:~renatofilho/address-book-app/remove-contact-view-custom-back-action
Merge into: lp:address-book-app
Diff against target: 28 lines (+0/-18)
1 file modified
src/imports/Ubuntu/AddressBook/ContactView/ContactViewPage.qml (+0/-18)
To merge this branch: bzr merge lp:~renatofilho/address-book-app/remove-contact-view-custom-back-action
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+287791@code.launchpad.net

Commit message

Remove custom back button from contact view.

This is not necessary, AdaptiveLayout will take care of it.

To post a comment you must log in.
551. By Renato Araujo Oliveira Filho

Remove custom back button from contact view.

This is not necessary, AdaptiveLayout will take care of it.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/imports/Ubuntu/AddressBook/ContactView/ContactViewPage.qml'
--- src/imports/Ubuntu/AddressBook/ContactView/ContactViewPage.qml 2016-02-24 23:06:48 +0000
+++ src/imports/Ubuntu/AddressBook/ContactView/ContactViewPage.qml 2016-03-02 14:41:52 +0000
@@ -50,24 +50,6 @@
50 trailingActionBar {50 trailingActionBar {
51 id: trailingBar51 id: trailingBar
52 }52 }
53 leadingActionBar {
54 id: leadingBar
55 actions: Action {
56 id: backAction
57
58 iconName: "back"
59 name: "cancel"
60 enabled: visible && root.active && root.enabled
61 visible: (pageStack.columns == 1) || (pageStack.columns == undefined)
62 onTriggered: {
63 if (pageStack.removePages)
64 pageStack.removePages(root)
65 else
66 pageStack.pop()
67 }
68 }
69
70 }
71 }53 }
7254
73 Connections {55 Connections {

Subscribers

People subscribed via source and target branches