Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-12-08 05:52:32 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed extra separator at end of context menu, b=77596, r=sfraser, sr=kin, a=asa-drivers

Browse Source
This commit is contained in:
cmanske%netscape.com 2001-06-08 05:08:05 +00:00
parent 81cbc7f48b
commit 9a8ffaf2c3
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
editor/ui/composer/content/EditorContextMenu.js
View File

@ -53,6 +53,10 @@ function EditorFillContextMenu(event, contextMenuNode)
HideDisabledItem(children.item(i));
}
// The 'Save image (imagename)' menuitem:
var isImage = (objectName == "img");
ShowMenuItem("menu_saveImage_cm", isImage);
// Remove separators if all items in immediate group above are hidden
// A bit complicated to account if multiple groups are completely hidden!
var haveUndo =
@ -70,7 +74,7 @@ function EditorFillContextMenu(event, contextMenuNode)
IsMenuItemShowing("createLink_cm") ||
IsMenuItemShowing("removeLinksMenuitem_cm");
var haveProps =
var havePropsOrImage =
IsMenuItemShowing("objectProperties_cm") ||
IsMenuItemShowing("menu_saveImage_cm")
@ -78,11 +82,6 @@ function EditorFillContextMenu(event, contextMenuNode)
ShowMenuItem("edit-separator", haveEdit || haveUndo);
//The following is for the 'save image (imagename)' contextmenuitem
var isImage = (objectName=="img");
ShowMenuItem("menu_saveImage_cm", isImage);
if (isImage) //we have an image
{
var saveImageMenuItem= document.getElementById("menu_saveImage_cm");
@ -102,10 +101,11 @@ function EditorFillContextMenu(event, contextMenuNode)
// following separator are ALWAYS enabled,
// so there will always be 1 separator here
//dump("haveStyle = "+haveStyle+", "+haveProps+", inCell"+inCell+"\n");
ShowMenuItem("styles-separator", haveStyle && (haveProps || inCell));
var showStyleSep = haveStyle && (havePropsOrImage || inCell);
ShowMenuItem("styles-separator", showStyleSep);
ShowMenuItem("property-separator", (haveProps && inCell) || !haveStyle);
var showPropSep = (havePropsOrImage && inCell) || !haveStyle;
ShowMenuItem("property-separator", showPropSep);
// Remove table submenus if not in table
ShowMenuItem("tableInsertMenu_cm", inCell);
@ -132,7 +132,6 @@ function HideDisabledItem( item )
if (!item) return false;
var enabled = (item.getAttribute('disabled') !="true");
//dump("HideDisabledItem: "+item.getAttribute("id")+", enabled="+enabled+"\n");
item.setAttribute("collapsed", enabled ? "" : "true");
item.setAttribute('contexthidden', enabled ? "" : "true");
return enabled;