Ubuntu UI Toolkit

Merge lp:~tpeeters/ubuntu-ui-toolkit/noBackButton into lp:ubuntu-ui-toolkit/staging

noBackButton
Merge into staging

Proposed by Tim Peeters on 2014-07-30

Status:	Merged
Approved by:	Cris Dywan on 2014-07-30
Approved revision:	1171
Merged at revision:	1170
Proposed branch:	lp:~tpeeters/ubuntu-ui-toolkit/noBackButton
Merge into:	lp:ubuntu-ui-toolkit/staging
Prerequisite:	lp:~tpeeters/ubuntu-ui-toolkit/iconFix
Diff against target:	135 lines (+106/-3) 3 files modified modules/Ubuntu/Components/Themes/Ambiance/PageHeadButton.qml (+2/-2) modules/Ubuntu/Components/Themes/Ambiance/PageHeadStyle.qml (+1/-1) tests/unit_x11/tst_components/tst_headActions.qml (+103/-0)
To merge this branch:	bzr merge lp:~tpeeters/ubuntu-ui-toolkit/noBackButton
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot	continuous-integration	2014-07-30	Approve on 2014-07-30
Cris Dywan		2014-07-30	Approve on 2014-07-30
Review via email: mp+228836@code.launchpad.net

This proposal supersedes a proposal from 2014-07-28.

Commit message

Allow app developers to disable the back button by setting an invisible back action for the page.

Description of the change

Allow app developers to disable the back button by setting an invisible back action for the page.

Use case: dialer app pushes call page on the pagestack when in a phone call, but the user cannot use the 'back' button to leave the call page. When the phone call is terminated, the call page is popped from the pagestack.

Revision history for this message

Leo Arias (elopio) wrote on 2014-07-28: Posted in a previous version of this proposal

why don't you put this

99 + pageStack.pop();

and this

117 + page2.head.backAction = null;

on the cleanup function?

I'm trying to learn more QML, so I'll put my nose on some of your new branches. Please bare with my questions while I get up to speed.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-28: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/674/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-28: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/675/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Cris Dywan (kalikiana) wrote on 2014-07-29: Posted in a previous version of this proposal

QWARN : components::HeaderBackButton::test_custom_back_button() file:///tmp/buildd/ubuntu-ui-toolkit-0.1.50+14.10.20140724.2bzr1167pkg0utopic507/modules/Ubuntu/Components/Themes/Ambiance/PageHeadButton.qml:29:5: QML QQuickImage: Failed to get image from provider: image://theme/search

Revision history for this message

Cris Dywan (kalikiana) wrote on 2014-07-29: Posted in a previous version of this proposal

Looks good.

I've seen the warning in another branch - I wonder if we're missing a dependency somewhere, though the package depends on suru-icon-theme already.

review: Approve

Revision history for this message

Tim Peeters (tpeeters) wrote on 2014-07-29: Posted in a previous version of this proposal

> Looks good.
>
> I've seen the warning in another branch - I wonder if we're missing a
> dependency somewhere, though the package depends on suru-icon-theme already.

it may be this bug: https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1349769

Revision history for this message

Tim Peeters (tpeeters) wrote on 2014-07-29: Posted in a previous version of this proposal

Don't happrove this yet. I am abusing this MR to test whether the fix in the iconFix branch works.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-29: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/678/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-29: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/680/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-29: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/681/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-29: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/682/rebuild

review: Approve (continuous-integration)

Revision history for this message

Cris Dywan (kalikiana) on 2014-07-30:

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-30: Posted in a previous version of this proposal

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/684/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) on 2014-07-30:

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

Subscribers

People subscribed via source and target branches

to all changes:

Dan Chapman 

Tim Peeters

Tom Mirmar

Ubuntu SDK team

 === modified file 'modules/Ubuntu/Components/Themes/Ambiance/PageHeadButton.qml'
 --- modules/Ubuntu/Components/Themes/Ambiance/PageHeadButton.qml	2014-07-22 15:11:03 +0000
 +++ modules/Ubuntu/Components/Themes/Ambiance/PageHeadButton.qml	2014-07-30 12:14:09 +0000
@@ -20,8 +20,8 @@
  AbstractButton {
      id: button
--    property int iconWidth: units.gu(2)
--    property int iconHeight: iconWidth
++    property real iconWidth: units.gu(2)
++    property real iconHeight: iconWidth
      width: visible ? units.gu(4) : 0
      height: parent ? parent.height : undefined
 === modified file 'modules/Ubuntu/Components/Themes/Ambiance/PageHeadStyle.qml'
 --- modules/Ubuntu/Components/Themes/Ambiance/PageHeadStyle.qml	2014-07-28 10:38:00 +0000
 +++ modules/Ubuntu/Components/Themes/Ambiance/PageHeadStyle.qml	2014-07-30 12:14:09 +0000
@@ -134,7 +134,7 @@
              visible: styledItem.pageStack !== null &&
                       styledItem.pageStack !== undefined &&
                       styledItem.pageStack.depth > 1 &&
--                     !customBackButton.visible
++                     !styledItem.config.backAction
              text: "back"
 === added file 'tests/unit_x11/tst_components/tst_headActions.qml'
 --- tests/unit_x11/tst_components/tst_headActions.qml	1970-01-01 00:00:00 +0000
 +++ tests/unit_x11/tst_components/tst_headActions.qml	2014-07-30 12:14:09 +0000
@@ -0,0 +1,103 @@
++/*
++ * Copyright 2014 Canonical Ltd.
++ *
++ * This program is free software; you can redistribute it and/or modify
++ * it under the terms of the GNU Lesser General Public License as published by
++ * the Free Software Foundation; version 3.
++ *
++ * This program is distributed in the hope that it will be useful,
++ * but WITHOUT ANY WARRANTY; without even the implied warranty of
++ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
++ * GNU Lesser General Public License for more details.
++ *
++ * You should have received a copy of the GNU Lesser General Public License
++ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
++ */
++
++import QtQuick 2.2
++import Ubuntu.Test 1.0
++import Ubuntu.Components 1.1
++
++// NOTE: Other parts of the page head API are tested with autopilot in
++// ubuntuuitoolkit.tests.components.test_header
++
++Item {
++    width: units.gu(50)
++    height: units.gu(80)
++
++    MainView {
++        id: mainView
++        anchors.fill: parent
++        useDeprecatedToolbar: false
++
++        PageStack {
++            id: pageStack
++            Page {
++                id: page1
++                title: "First page"
++            }
++            Page {
++                id: page2
++                title: "Second page"
++
++                Action {
++                    id: customBackAction
++                    iconName: "search"
++                    text: "Search"
++                }
++                Action {
++                    id: invisibleAction
++                    visible: false
++                }
++            }
++        }
++        Component.onCompleted: {
++            pageStack.push(page1);
++        }
++    }
++
++    UbuntuTestCase {
++        name: "HeaderBackButton"
++        when: windowShown
++        id: testCase
++
++        property var app_header
++        property var back_button
++        property var custom_back_button
++
++        function initTestCase() {
++            testCase.app_header = findChild(mainView, "MainView_Header");
++            testCase.back_button = findChild(app_header, "backButton");
++            testCase.custom_back_button = findChild(app_header, "customBackButton");
++
++            compare(page2.head.backAction, null, "Back action set by default.");
++            compare(back_button.visible, false, "Back button visible with only 1 page on the stack.");
++            compare(custom_back_button.visible, false, "Custom back button visible without custom back action.")
++        }
++
++        function test_default_back_button() {
++            pageStack.push(page2);
++            compare(back_button.visible, true, "Back button not visible with 2 pages on stack.");
++            compare(custom_back_button.visible, false, "Showing custom back button without custom back action.");
++            pageStack.pop();
++        }
++
++        function test_custom_back_button() {
++            page2.head.backAction = customBackAction;
++            pageStack.push(page2);
++            compare(back_button.visible, false, "Default back button visible with custom back action.");
++            compare(custom_back_button.visible, true, "Custom back button invisible with back action.");
++            pageStack.pop();
++            page2.head.backAction = null;
++        }
++
++        function test_no_back_button() {
++            page2.head.backAction = invisibleAction;
++            pageStack.push(page2);
++            compare(back_button.visible, false, "Default back button visible with invisible custom back action.");
++            compare(custom_back_button.visible, false, "Custom back button visible with invisible custom back action.");
++            pageStack.pop();
++            page2.head.backAction = null;
++        }
++    }
++}