From 69d4aa363b659e201d12451458181dbb7e46416c Mon Sep 17 00:00:00 2001 From: "Denis Vermylen (dve)" Date: Thu, 23 Jun 2016 16:50:35 +0200 Subject: [PATCH] [FIX] web: properly hides submenu (ir_actions target=fullscreen) This commit fixes the action target="fullscreen" in community, where the submenu on the left would sometimes remain visible. The fullscreen feature was added in commit : d65d251c57bd2a72e3ac91c579f0ff604087e577 --- addons/web/static/src/js/menu.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/web/static/src/js/menu.js b/addons/web/static/src/js/menu.js index c244305b7aa..cab9e0b8794 100644 --- a/addons/web/static/src/js/menu.js +++ b/addons/web/static/src/js/menu.js @@ -160,12 +160,12 @@ var Menu = Widget.extend({ $sub_menu.show(); // Hide/Show the leftbar menu depending of the presence of sub-items - this.$secondary_menus.toggle(!!$sub_menu.children().length); + this.$secondary_menus.toggleClass('o_hidden', !$sub_menu.children().length); // Activate current menu item and show parents this.$secondary_menus.find('.active').removeClass('active'); if ($main_menu !== $clicked_menu) { - $clicked_menu.parents().show(); + $clicked_menu.parents().removeClass('o_hidden'); if ($clicked_menu.is('.oe_menu_toggler')) { $clicked_menu.toggleClass('oe_menu_opened').siblings('.oe_secondary_submenu:first').toggle(); } else {