touch-packages team mailing list archive

[Zoltan Balogh <zoltan.balogh@xxxxxxxxxxxxx>]

** Changed in: ubuntu-ui-toolkit (Ubuntu)
       Status: New => Incomplete

** Changed in: ubuntu-ui-toolkit (Ubuntu)
       Status: Incomplete => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to ubuntu-ui-toolkit in
Ubuntu.
https://bugs.launchpad.net/bugs/1326963

Title:
  PageStack.push() eats header action popover close

Status in Ubuntu UI Toolkit:
  Fix Released
Status in “ubuntu-ui-toolkit” package in Ubuntu:
  Fix Released

Bug description:
  Simply put, if you push a new page as part of an ontriggered event for
  a toolbar button, and that button is part of a popover, the next event
  on the pushed page is eaten by the popover which fails to close.

  A simple example can be seen here:

  lp:~nskaggs/+junk/pagestack-push

  Run the included example.qml file and click the toobar button entitled
  'Click Me!'. Notice the first click event to the textfield doesn't
  transfer focus as it should.

  Including Tim's running commentary while investigating for more info
  :-)

  <t1mp> the popover that shows the two actions doesn't close properly after the new page is pushed
  <t1mp> so that is eating the events.
  <t1mp> if I remove the pageStack.push from the action, it does close the popover
  <t1mp> it seems it all works fine, except if in onTriggered, there is a pageStack.push().. then the Popover.close() that is called in onTriggered for the listitem (which represents the action) is not executed
  <t1mp> when pushing a new page, the list of actions in the header changes, which destroys the list items in the popover that shows the actions
  <t1mp> ^ and that happens before the onTriggered of the list item which should close the popover is executed
  <t1mp> that messes up things

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1326963/+subscriptions