Ext.navigation.view - the pop event sends the wrong view to listener

[melevine]

Ext version tested:

• Sencha Touch 2.0PR4

Browser versions tested against:

• Chrome 15 (Linux)

Description:

• Ext.navigation.view - the pop event is sending the newly active item instead of the popped view http://docs.sencha.com/touch/2-0/#!/...View-event-pop

Steps to reproduce the problem:

• Create Ext.navigation.View, hook up a listener to the pop event, push 2 views, pop the top view. The 2nd argument to the listener is the the first view not the second.

The result that was expected:

• The 2nd argument to the pop event listener should be the popped view

The result that occurs instead:

• The 2nd argument to the pop event listener is the active view

Test Case:

var view = Ext.create('Ext.navigation.View', {
     constructor : function(config) {
          this.callParent(arguements);
          this.push({ title : 'view a', xtype : 'list'};
          this.push({ title : 'view b', xtype : 'button'};
          this.addListener('pop', this.onPopped, this);
     },
     onPopped : function(view, poppedView, eOpts) {
          console.log(poppedView); // will be the list not the button
     },
});

Possible Fix:

• In doSetActiveItem change the following

me.fireEvent('pop', this, activeItem);

to

me.fireEvent('pop', this, oldActiveItem);

[Jamie Avins]

Thank you for the report.