Ext.ux.grid.GridViewMenuPlugin

[Matthias_WB]

Hello community,

because many developer forget binding the css-file, I have the style properties in the JS file placed.

In "init"-function near line 103 you add the red marked line.

init : function(grid)
    {
        if (grid.enableHdMenu === false) {
            return;
        }

        styleExtention=Ext.util.CSS.createStyleSheet('.ext-ux-grid-gridviewmenuplugin-menuBtn.x-grid3-hd-btn {display:block}');

Then you don't bind the CSS file. When you open a grid with added plugin-property, you see a button with "Arrow down"-symbol.




When posibile, integrated my idea on the next version.


♫

[fangzhouxing]

I found a bug in IE7, when using hideHeaders config item in grid, the whole grid cannot show!
In FF 3.5, it is fine.

[makana]

Hi Thorsten!
Thanks for sharing this great plugin! I like it!

I found out, that

this.colMenu.getEl()

is undefined until you open the menu. So there's an error in your destroy-method.

I simply changed it to

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

        if(this._menuBtn){
            this._menuBtn.remove();
            delete this._menuBtn;
        }
    }

The error occured when I closed a tab which contains a GridPanel with your plugin and I didn't open the menu at least once.
(But it's reproducable in a simple example. I modified the array-grid example and added your plugin. Then I called grid.destroy() on a button click. Just the same error.)

Maybe the problem occurs in your app as well and you can apply the little change to your code.

Keep up the great work!
Best regards

[ThorstenSuckow]

I found out, that

this.colMenu.getEl()

is undefined until you open the menu. So there's an error in your destroy-method.

Hi Makana,

thanks for your error report. I'll fix this for the next release!

[fangzhouxing]

Using 'Ext.util.CSS.createStyleSheet' will have problem in IE7.

see: http://extjs.com/forum/showthread.php?p=369735

[tonedeaf]

I would like to have both menus, too! If theres technically and logically nothing to say against it, why would you want to prevent it?
It would be useful for example when you have other plugins like columnfilters in hd-menu of columns and this plugin for column-show/hide menu on the right.

+1 for keeping the existing enableHdMenu option and operating this plugin separately of the column dropdown menus.
I use the GridFilters extension and there doesn't look any way of using both at the moment.

[futpol]

Hi Makana,

thanks for your error report. I'll fix this for the next release!

As I have the same problem I would appreciate a fix in svn, to prevent local change to be overwritten by clean download of plugin.

Thanks for great plugin!

[ThorstenSuckow]

Hey there, I have just updated the plugin to work with Ext 3.1, along with some minor bug fixes.

[warren72]

Is there a way to show the menu from a click of a button that is somewhere else on the page? It works as specified, but I have a need to be able to display it when a button is clicked. I can't seem to figure how to get the handle/reference to the menu object.

[moegal]

Hi Thorsten!
Thanks for sharing this great plugin! I like it!

I found out, that

this.colMenu.getEl()

is undefined until you open the menu. So there's an error in your destroy-method.

I simply changed it to

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

        if(this._menuBtn){
            this._menuBtn.remove();
            delete this._menuBtn;
        }
    }

The error occured when I closed a tab which contains a GridPanel with your plugin and I didn't open the menu at least once.
(But it's reproducable in a simple example. I modified the array-grid example and added your plugin. Then I called grid.destroy() on a button click. Just the same error.)

Maybe the problem occurs in your app as well and you can apply the little change to your code.

Keep up the great work!
Best regards

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