Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-1562 in a recent build.
  1. #1
    Touch Premium Member
    Join Date
    Jan 2012
    Location
    Los Angeles, CA
    Posts
    32
    Vote Rating
    1
    melevine is on a distinguished road

      0  

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

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


    Ext version tested:
    • Sencha Touch 2.0PR4
    Browser versions tested against:
    • Chrome 15 (Linux)
    Description: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:

    Code:
    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
    Code:
    me.fireEvent('pop', this, activeItem);
    to
    Code:
    me.fireEvent('pop', this, oldActiveItem);
    Last edited by melevine; 24 Jan 2012 at 6:37 PM. Reason: reported wrong ver of chrome

  2. #2
    Sencha User Jamie Avins's Avatar
    Join Date
    Mar 2007
    Location
    Redwood City, California
    Posts
    3,661
    Vote Rating
    18
    Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough Jamie Avins is a jewel in the rough

      0  

    Default


    Thank you for the report.

    Sencha Inc

    Jamie Avins

    @jamieavins

Thread Participants: 1