Results 1 to 3 of 3

Thread: CompositeElement and Ext.EventManager

    Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-2616 in a recent build.
  1. #1
    Sencha Premium Member
    Join Date
    Apr 2009
    Posts
    285
    Vote Rating
    21
      0  

    Default CompositeElement and Ext.EventManager

    I've been looking at some code which uses a CompositeElement and noticed that the code won't run unless I use the compatibility library. When using the regular library I get an error when some code in CompositeElementLite tries to use Ext.EventManager. The code in question from CompositeElementLite looks like this:

    Code:
    // fixes scope with flyweight
    addListener: function(eventName, handler, scope, opt) {
         var els = this.elements,
               len = els.length,
               i, e;
         for (i = 0; i < len; i++) {
                e = els[i];
                if (e) {
                    Ext.EventManager.on(e, eventName, handler, scope || e, opt);
                }
        }
        return this;
    }
    
    According to the documentation, Ext.EventManager is deprecated and shouldn't be used. If so, it doesn't seem like a base Sencha Touch class should be using it either.

    Did this one slip through?

    Thanks.

    Jim

  2. #2
    Sencha - Sr Software Engineer mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    39,551
    Vote Rating
    1271
      0  

    Default

    Appears it has.
    Mitchell Simoens @LikelyMitch
    Sencha Inc, Senior Software Engineer
    ________________
    Learn BBCode and use it! Checkout the CODE tag!

    Check out my GitHub, lots of nice things for Ext JS and Sencha Touch
    https://github.com/mitchellsimoens

  3. #3
    Sencha User rdougan's Avatar
    Join Date
    Oct 2008
    Posts
    1,159
    Vote Rating
    7
      0  

    Default

    Fixed for the next release. Thanks.
    Sencha Inc.
    Robert Dougan - @rdougan
    Sencha Touch 2 and Ext JS 4 Core Team Member, SASS/Theming Wizard.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •