search

cadorn / ace-extjs

ACE Editor for ExtJS
122 stars 41 forks source link

Extending Ext.panel.Panel #6

Open dedalo opened 12 years ago

dedalo commented 12 years ago

Are you sure this is the best component to extend? Panel is a container, so its function is to hold several items. and has some special items as toolbars, etc.

I tried to add a toolbar to Ext.ux.AceEditor and it fails on the DOM search of child nodes... I thing if you extend Ext.Component instead of panel, things would be more easier on the Ext way... also you should add an alias configuration so it behaves as other components inside ext...

cadorn commented 12 years ago

When I started this component I did not know much about ExtJS :)

Feel free to patch or I'll get to it after Issue #4.