From a90940283c232f15d934f916f5177c23b4805179 Mon Sep 17 00:00:00 2001
From: ed <eddavies95@gmail.com>
Date: Mon, 13 May 2019 09:22:37 +0100
Subject: [PATCH] Fixed an issue where PopupMenus with a parent component were
 not being dismissed by dismissAllActiveMenus()

---
 modules/juce_gui_basics/menus/juce_PopupMenu.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/juce_gui_basics/menus/juce_PopupMenu.cpp b/modules/juce_gui_basics/menus/juce_PopupMenu.cpp
index cb10aad047..c4af0d037a 100644
--- a/modules/juce_gui_basics/menus/juce_PopupMenu.cpp
+++ b/modules/juce_gui_basics/menus/juce_PopupMenu.cpp
@@ -258,10 +258,10 @@ struct MenuWindow  : public Component
                           | ComponentPeer::windowIgnoresKeyPresses
                           | lf.getMenuWindowFlags());
 
-            getActiveWindows().add (this);
             Desktop::getInstance().addGlobalMouseListener (this);
         }
 
+        getActiveWindows().add (this);
         lf.preparePopupMenuWindow (*this);
 
         getMouseState (Desktop::getInstance().getMainMouseSource()); // forces creation of a mouse source watcher for the main mouse