From e3a62c0dfe165ed2b564be7c5646ded141af75d3 Mon Sep 17 00:00:00 2001 From: zorkow Date: Mon, 14 Sep 2015 01:43:57 +0100 Subject: [PATCH] Fixes the menu role. --- unpacked/extensions/MathMenu.js | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/unpacked/extensions/MathMenu.js b/unpacked/extensions/MathMenu.js index 8bcbd0163..1bb2834af 100644 --- a/unpacked/extensions/MathMenu.js +++ b/unpacked/extensions/MathMenu.js @@ -280,7 +280,7 @@ onmouseup: MENU.Mouseup, ondblclick: FALSE, ondragstart: FALSE, onselectstart: FALSE, oncontextmenu: FALSE, menuItem: this, className: "MathJax_Menu", onkeydown: MENU.Keydown, - role: "navigation" + role: "menu" }); if (!forceLTR) {MathJax.Localization.setCSS(menu)} @@ -897,9 +897,8 @@ role: "menuitemradio", Attributes: function(def) { - if (CONFIG.settings[this.variable] === this.value) { - def = HUB.Insert({"aria-checked": "true"}, def); - } + var checked = CONFIG.settings[this.variable] === this.value ? "true" : "false"; + def = HUB.Insert({"aria-checked": checked}, def); def = this.SUPER(arguments).Attributes.call(this, def); return def; }, @@ -947,9 +946,8 @@ role: "menuitemcheckbox", Attributes: function(def) { - if (CONFIG.settings[this.variable]) { - def = HUB.Insert({"aria-checked": "true"}, def); - } + var checked = CONFIG.settings[this.variable] ? "true" : "false"; + def = HUB.Insert({"aria-checked": checked}, def); def = this.SUPER(arguments).Attributes.call(this, def); return def; },