Comment 0 for bug 1326963

Revision history for this message
Nicholas Skaggs (nskaggs) wrote : New header toolbar popover eats events

Simply put, if you push a new page as part of an ontriggered event for a toolbar button, and a 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