From 2c5bfab567a35a7fbff2943fa9010de8369a1aec Mon Sep 17 00:00:00 2001
From: Jens Ulferts <j.ulferts@finn.de>
Date: Wed, 26 Apr 2017 09:39:36 +0200
Subject: [PATCH] close menu regardless of success

---
 .../settings-menu/settings-menu.controller.ts              | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/frontend/app/components/context-menus/settings-menu/settings-menu.controller.ts b/frontend/app/components/context-menus/settings-menu/settings-menu.controller.ts
index 850d8eb810..b98e11eba0 100644
--- a/frontend/app/components/context-menus/settings-menu/settings-menu.controller.ts
+++ b/frontend/app/components/context-menus/settings-menu/settings-menu.controller.ts
@@ -108,20 +108,21 @@ function SettingsDropdownMenuController($scope:IMyScope,
   $scope.saveQuery = function (event:JQueryEventObject) {
     event.stopPropagation();
     if (!query.id && allowQueryAction(event, 'updateImmediately')) {
-      closeAnyContextMenu();
       saveModal.activate();
     } else if (query.id && allowQueryAction(event, 'updateImmediately')) {
       closeAnyContextMenu();
-      wpListService.save();
     }
+
+    closeAnyContextMenu();
   };
 
   $scope.deleteQuery = function (event:JQueryEventObject) {
     event.stopPropagation();
     if (allowQueryAction(event, 'delete') && deleteConfirmed()) {
       wpListService.delete();
-      closeAnyContextMenu();
     }
+
+    closeAnyContextMenu();
   };
 
   // Modals