-
28 Mar 2012 4:01 AM #1
CompositeElement and Ext.EventManager
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:
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.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; }
Did this one slip through?
Thanks.
Jim
-
28 Mar 2012 5:17 AM #2Sencha - Senior Forum Manager
- Join Date
- Mar 2007
- Location
- St. Louis, MO
- Posts
- 33,624
- Vote Rating
- 435
Appears it has.
Mitchell Simoens @SenchaMitch
Sencha Inc, Senior Forum Manager
________________
http://www.JSONPLint.com - Source to lint your JSONP!
Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
https://github.com/mitchellsimoens
Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/
Need more help with your app? Hire Sencha Services services@sencha.com
Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is almost in print!
When posting code, please use BBCode's CODE tags.
-
28 Mar 2012 4:19 PM #3
Fixed for the next release. Thanks.
Sencha Inc.
Robert Dougan - @rdougan
Sencha Touch 2 and Ext JS 4 Core Team Member, SASS/Theming Wizard.
Success! Looks like we've fixed this one. According to our records the fix was applied for
TOUCH-2616
in
2.0.


Reply With Quote