#17 Double bug in cmEnableItem

[fguerzoni]

Hi, I found 2 bugs in cmEnableItem:

function cmEnableItem (item, prefix)
{
if (!item)
return;
var menuItem = cmGetMenuItem (item);
if (!menuItem)
return;
if (item.menu)
menu.className = prefix + 'MainItem';
else
menu.className = prefix + 'MenuItem';
item.isDisabled = true;
}

1) you must use menuItem instead of menu which doesn't exists.
2) you must set isDisabled to false.

I've added 2 more functions. Maybe they could be interesting to you:

//
// hide a menu item
//
function cmHideItem (item, prefix)
{
if (!item)
return;
var menuItem = cmGetMenuItem (item);
if (!menuItem)
return;
menuItem.style.display = 'none';
item.isDisabled = true;
}

//
// show a menu item
//
function cmShowItem (item, prefix)
{
if (!item)
return;
var menuItem = cmGetMenuItem (item);
if (!menuItem)
return;
menuItem.style.display = '';
item.isDisabled = false;
}

You are free to use them as you want.
Thanks for your great work.

[Nobody/Anonymous]

Logged In: NO

this bug is pretty severe if you use iframes (and maybe other controls) in IE which are overlapped by the menu. However it does not seem to affect Firefox 2.