From 1325b260c8763b0cff7d7cd6d4bc6999a9ba88c3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20D=C3=ADaz?= <alexandre.diaz@tecnativa.com>
Date: Wed, 20 Nov 2019 17:36:48 +0100
Subject: [PATCH] [FIX] web_responsive: modify action handling

Previous this commit, AppMenu doesn't hide in some situations.
Steps to reproduce (sometimes works, sometimes doesn't):
From Discuss App (refresh) open AppMenu and search some menu,
select the second in the list and press enter.
The action is launched correctly, but the AppMenu doesn't hide.

Now uses 'defer' to ensure that all has done to do the task
---
 .../static/src/js/web_responsive.js           | 34 ++++++++++---------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/web_responsive/static/src/js/web_responsive.js b/web_responsive/static/src/js/web_responsive.js
index 0d9bfa793..4febf0434 100644
--- a/web_responsive/static/src/js/web_responsive.js
+++ b/web_responsive/static/src/js/web_responsive.js
@@ -442,22 +442,24 @@ odoo.define('web_responsive', function (require) {
         * The executed action
         */
         _hideMenusByAction: function (action) {
-            var uniq_sel = '[data-action-id='+action.id+']';
-            // Need close AppDrawer?
-            var menu_apps_dropdown = document.querySelector(
-                '.o_menu_apps .dropdown');
-            $(menu_apps_dropdown).has('.dropdown-menu.show')
-                .has(uniq_sel).find('> a').dropdown('toggle');
-            // Need close Sections Menu?
-            // TODO: Change to 'hide' in modern Bootstrap >4.1
-            var menu_sections = document.querySelector(
-                '.o_menu_sections li.show');
-            $(menu_sections).has(uniq_sel).find('.dropdown-toggle')
-                .dropdown('toggle');
-            // Need close Mobile?
-            var menu_sections_mobile = document.querySelector(
-                '.o_menu_sections.show');
-            $(menu_sections_mobile).has(uniq_sel).collapse('hide');
+            _.defer(function () {
+                var uniq_sel = '[data-action-id='+action.id+']';
+                // Need close AppDrawer?
+                var menu_apps_dropdown = document.querySelector(
+                    '.o_menu_apps .dropdown');
+                $(menu_apps_dropdown).has('.dropdown-menu.show')
+                    .has(uniq_sel).find('> a').dropdown('toggle');
+                // Need close Sections Menu?
+                // TODO: Change to 'hide' in modern Bootstrap >4.1
+                var menu_sections = document.querySelector(
+                    '.o_menu_sections li.show');
+                $(menu_sections).has(uniq_sel).find('.dropdown-toggle')
+                    .dropdown('toggle');
+                // Need close Mobile?
+                var menu_sections_mobile = document.querySelector(
+                    '.o_menu_sections.show');
+                $(menu_sections_mobile).has(uniq_sel).collapse('hide');
+            });
         },
 
         _handleAction: function (action) {