Permalink
Browse files

The XUL popup element is deprecated, so use menupopup instead, which …

…apparently works exactly the same way but is spelled different. Weird.
  • Loading branch information...
slaarti committed Mar 27, 2011
1 parent a305496 commit 1b8b28be00110e31b4c608a2846ed66d094db93a
Showing with 3 additions and 3 deletions.
  1. +1 −1 content/ljlogin.js
  2. +2 −2 content/ljlogin.xul
View
@@ -126,7 +126,7 @@ function LJlogin_statusbar_refresh() {
thepanel.setAttribute("popup", "ljlogin-" + siteid + "-menu");
sb.insertBefore(thepanel, ljsb);
var themenu = document.createElement("popup");
var themenu = document.createElement("menupopup");
themenu.setAttribute("id", "ljlogin-" + siteid + "-menu");
themenu.setAttribute("position", "after_start");
themenu.setAttribute("onpopupshowing",
View
@@ -29,11 +29,11 @@ try {
</statusbar>
<popupset id="mainPopupSet">
<popup id="ljlogin-menu" position="after_start">
<menupopup id="ljlogin-menu" position="after_start">
<menuitem label="LJlogin" disabled="true" />
<menuseparator />
<menuitem label="Preferences" oncommand="LJlogin_prefs('');" />
</popup>
</menupopup>
</popupset>
</overlay>

0 comments on commit 1b8b28b

Please sign in to comment.