1. #1
    Ext JS Premium Member gevik's Avatar
    Join Date
    May 2009
    Location
    The Netharlands
    Posts
    265
    Vote Rating
    8
    gevik will become famous soon enough

      0  

    Question [BUG?] Ext.grid.plugin.* inconsistency

    [BUG?] Ext.grid.plugin.* inconsistency


    I noticed the following:

    [1] Not all grid plugins ultimately inherit from Ext.AbstractPlugin, especially the Ext.grid.plugin.Editing, I wonder why it has to be like that.

    [2] The HeaderResizer plugin inherits from Ext.util.Observable, should't it use the observable as a mixin?

    Please clarify.

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    36,994
    Vote Rating
    846
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Ext.AbstractPlugin doesn't really do a whole lot. It has some standard template methods but that's really it. Ext.util.Observable can be a mixin or it's own class so extending Ext.util.Observable is still valid. Ext.AbstractPlugin extends Ext.Base and Ext.util.Observable also extends Ext.Base so they are sisters and since Ext.AbstractPlugin doesn't do a whole lot, extending Ext.util.Observable may have better performance than using it as a mixing and extending Ext.AbstractPlugin. Inconsistent... your argument is still valid there.
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    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 in print!

    When posting code, please use BBCode's CODE tags.

Thread Participants: 1