From e105aa41427e5216dcf53f8b31b9b8c689a064ba Mon Sep 17 00:00:00 2001 From: Alive Kuo Date: Thu, 27 Dec 2012 20:36:12 +0800 Subject: [PATCH] Bug 824880 - Disable multi-level context menu, r=vivien, a=blocking-basecamp --- apps/system/js/list_menu.js | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/apps/system/js/list_menu.js b/apps/system/js/list_menu.js index c25c949ab36e..7962fb43ae01 100644 --- a/apps/system/js/list_menu.js +++ b/apps/system/js/list_menu.js @@ -60,14 +60,10 @@ var ListMenu = { var button = document.createElement('a'); button.setAttribute('role', 'button'); if (item.type && item.type == 'menu') { - this.currentLevel += 1; - this.currentParent = containerDiv.id; - this.buildMenu(item.items); - this.currentLevel -= 1; - item_div.classList.add('submenu'); - - button.href = '#' + this.currentChild; - button.textContent = item.label; + // XXX: We disallow multi-level menu at this moment + // See https://bugzilla.mozilla.org/show_bug.cgi?id=824928 + // for UX design and dev implementation tracking + return; } else if (item.type && item.type == 'menuitem') { button.dataset.value = item.id; button.textContent = item.label;