Tabs current selection changes when pushed down a PageStack

Bug #1316736 reported by Michael Hall
26
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Fix Released
Critical
Tim Peeters
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Unassigned
Vivid
New
Undecided
Unassigned

Bug Description

If you place a Tabs component as the first page in a PageStack, as suggested in the API docs[1], the selected tab will be reset to the first tab if you push a page on top of it and then go back.

App that experiences this problem: https://code.launchpad.net/~mhall119/loco-team-portal/ubuntu-app

This is a regression, utopic r2 worked correctly (selected tab remained selected), but in r13 the selected tab changes back to the first.

I also encounter it on my desktop (trusty) once I switched on the SDK PPA, I don't recall it happening when I was using the SDK packages in the trusty archives.

[1] http://developer.ubuntu.com/api/qml/sdk-14.10/Ubuntu.Components.Tabs/

Related branches

Michael Hall (mhall119)
tags: added: header tabs
description: updated
Revision history for this message
Michael Hall (mhall119) wrote :

Attached is what is printed to the console when running my app from the commandline.

description: updated
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
assignee: nobody → Tim Peeters (tpeeters)
importance: Undecided → High
Revision history for this message
Tim Peeters (tpeeters) wrote :

do you have a small test program that exposes the bug?

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
importance: High → Critical
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

12:15 PM <nik90> t1mp: with the old headers, in the clock app, when I go to the add alarm page by pushing a page into the pagestack, when I press back it does not return to the original Alarm tab. But instead it returns to the clock tab
12:15 PM <nik90> t1mp: so Clock Tab -> Alarm Tab -> Push Page to go into Add Alarm Page -> Press Back Button -> Returns to the Clock Tab
12:15 PM <nik90> t1mp: this bug is not happening with the new headers
12:32 PM <t1mp> nik90: is it this bug? https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1316736
12:32 PM <nik90> t1mp: yup that's the bug
12
:33 PM <nik90> t1mp: can we raise its priority since it is causing all my alarms tests to fail essentially blocking code merges
12:33 PM <t1mp> nik90: I didn't realize that it happens with the old tabs only.. that's a regression

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/staging at revision None, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+14.10.20140512-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+14.10.20140512-0ubuntu1) utopic; urgency=low

  [ Tarmac ]
  * [ Tim Peeters ] Make it possible to replace the title in the header
    with a.   custom Item. (LP: #1316736, #1208833, #1314948)
 -- Ubuntu daily release <email address hidden> Mon, 12 May 2014 12:27:41 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Released
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.