Fix bug # 28238: (work around bug # 26402) get context menus in bookmarks window/sidebar working, by setting "oncommand" attribute AFTER inserting node into content model.

This commit is contained in:
rjc%netscape.com 2000-03-17 01:40:05 +00:00
Родитель ef27072d80
Коммит b22afec3a7
1 изменённых файлов: 5 добавлений и 4 удалений

Просмотреть файл

@ -598,10 +598,10 @@ function fillContextMenu(name)
debug("Command #" + cmdIndex + ": id='" + cmdResource.Value + "' name='" + cmdName + "'");
var menuItem = document.createElement("menuitem");
menuItem.setAttribute("value", cmdName);
menuItem.setAttribute("oncommand", "return doContextCmd('" + cmdResource.Value + "');");
menuItem.setAttribute("value", cmdName);
popupNode.appendChild(menuItem);
// work around bug # 26402 by setting "oncommand" attribute AFTER appending menuitem
menuItem.setAttribute("oncommand", "return doContextCmd('" + cmdResource.Value + "');");
}
// if one and only one node is selected
@ -625,8 +625,9 @@ function fillContextMenu(name)
var menuItem = document.createElement("menuitem");
menuItem.setAttribute("value", propMenuName);
menuItem.setAttribute("oncommand", "return BookmarkProperties();");
popupNode.appendChild(menuItem);
// work around bug # 26402 by setting "oncommand" attribute AFTER appending menuitem
menuItem.setAttribute("oncommand", "return BookmarkProperties();");
}
}