Ext.ux.grid.GridViewMenuPlugin

[mystix]

I changed it the following to get it to work:

if(typeof(this.colMenu.getEl())!="undefined"){ 
     if (this.colMenu.getEl()) this.colMenu.getEl().remove();
}
//this.colMenu.getEl().remove();

Thanks, Marty

i believe it should be sufficient to simply change that to

    _destroy : function() {
        if (this.colMenu) {
            this.colMenu.removeAll();
            Ext.menu.MenuMgr.unregister(this.colMenu);
            Ext.destroy(this.colMenu); // this.colMenu.el.remove();
            delete this.colMenu;
        }

        if (this._menuBtn) {
            Ext.destroy(this._menuBtn); // this._menuBtn.remove();
            delete this._menuBtn;
        }
    }

[moegal]

mystix,

thanks but then I get another error after i use the menu and then close it

me.dom has no properties

Thanks, Marty

[moegal]

mystix,

I spoke too soon. I am not getting the error now but I cannot reopen the menu after it has been closed.

For example I have a tab with a grid. I open the tab, display the grid and open the gridviewmenu. I close the tab. I reopen the tab and I cannot access the gidviewmenu. on this or any other gid in any tab.

Thanks,

Marty

[mystix]

@moegal: sounds like you're attempting to reuse a single instance of this plugin with multiple grids. got a link to a test page somewhere where i can take a look?

[moegal]

That may be it. I am giving each grid a unique name and id though:

var grid1 = new Ext.grid.GridPanel({
id: 'grid1panel',
store: gridstore1,
plugins:[
new Ext.ux.grid.GridViewMenuPlugin()
],
enableHdMenu: true,
...

AND

var grid2 = new Ext.grid.GridPanel({
id: 'grid2panel',
store: gridstore2,
plugins:[
new Ext.ux.grid.GridViewMenuPlugin()
],
enableHdMenu: true,
...

Am I doing this wrong? Should I be assigning each plug-in to a var 1st?

Thanks, Marty

[mystix]

you're doing the right thing -- i.e. creating a new plugin instance for each unique grid.

can't tell what's wrong from the code you posted though.

[moegal]

let me check the version, must be something to do with that.

thanks, Marty

[WixSL]

The plugin stop working with Ext 3.3.0 beta.
Clicking a menu item result in the following error:

this.config[col] is undefined (on GridView.getDataIndex)

This is because GridView.hdCtxIndex doesn't get set.
Probably because GridView._handleHdDown gets fired before GridView.handleHdDown.

[WixSL]

I've fixed this by initializing the hdCtxIndex attribute.

init : function(grid){
	if(grid.enableHdMenu === true){
		throw("Ext.ux.grid.GridViewMenuPlugin - grid\"s \"enableHdMenu\" property has to be set to \"false\"");
	}
	this._view = grid.getView();
	this._view.initElements = this._view.initElements.createSequence(this.initElements,this);
	this._view.initData = this._view.initData.createSequence(this.initData,this);
	this._view.destroy = this._view.destroy.createInterceptor(this._destroy,this);
	this._view.hdCtxIndex = 0;
	this.colMenu = new Ext.menu.Menu();
	this.colMenu.on("beforeshow", this._beforeColMenuShow, this);
	this.colMenu.on("itemclick", this._handleHdMenuClick, this);
}

[Matthias_WB]

The fix working fine under ExtJS3.30..
Thank you very much!