Fix to make sure that ResizableWindow can't accidentally be dragged without first getting a mouseDown event.

modules/juce_gui_basics/windows/juce_ResizableWindow.cpp

@@ -22,24 +22,22 @@
   ==============================================================================
 */
 
-ResizableWindow::ResizableWindow (const String& name,
-                                  const bool addToDesktop_)
-    : TopLevelWindow (name, addToDesktop_),
+ResizableWindow::ResizableWindow (const String& name, bool shouldAddToDesktop)
+    : TopLevelWindow (name, shouldAddToDesktop),
       ownsContentComponent (false),
       resizeToFitContent (false),
       fullscreen (false),
+      dragStarted (false),
       constrainer (nullptr)
      #if JUCE_DEBUG
       , hasBeenResized (false)
      #endif
 {
-    initialise (addToDesktop_);
+    initialise (shouldAddToDesktop);
 }
 
-ResizableWindow::ResizableWindow (const String& name,
-                                  Colour backgroundColour_,
-                                  const bool addToDesktop_)
-    : TopLevelWindow (name, addToDesktop_),
+ResizableWindow::ResizableWindow (const String& name, Colour bkgnd, bool shouldAddToDesktop)
+    : TopLevelWindow (name, shouldAddToDesktop),
       ownsContentComponent (false),
       resizeToFitContent (false),
       fullscreen (false),
@@ -48,9 +46,9 @@ ResizableWindow::ResizableWindow (const String& name,
       , hasBeenResized (false)
      #endif
 {
-    setBackgroundColour (backgroundColour_);
+    setBackgroundColour (bkgnd);
 
-    initialise (addToDesktop_);
+    initialise (shouldAddToDesktop);
 }
 
 ResizableWindow::~ResizableWindow()
@@ -574,15 +572,23 @@ bool ResizableWindow::restoreWindowStateFromString (const String& s)
 void ResizableWindow::mouseDown (const MouseEvent& e)
 {
     if (! isFullScreen())
+    {
+        dragStarted = true;
         dragger.startDraggingComponent (this, e);
+    }
 }
 
 void ResizableWindow::mouseDrag (const MouseEvent& e)
 {
-    if (! isFullScreen())
+    if (dragStarted)
         dragger.dragComponent (this, e, constrainer);
 }
 
+void ResizableWindow::mouseUp (const MouseEvent&)
+{
+    dragStarted = false;
+}
+
 //==============================================================================
 #if JUCE_DEBUG
 void ResizableWindow::addChildComponent (Component* const child, int zOrder)

modules/juce_gui_basics/windows/juce_ResizableWindow.h

@@ -339,6 +339,8 @@ protected:
     /** @internal */
     void mouseDrag (const MouseEvent&) override;
     /** @internal */
+    void mouseUp (const MouseEvent&) override;
+    /** @internal */
     void lookAndFeelChanged() override;
     /** @internal */
     void childBoundsChanged (Component*) override;
@@ -374,7 +376,7 @@ protected:
 private:
     //==============================================================================
     Component::SafePointer<Component> contentComponent;
-    bool ownsContentComponent, resizeToFitContent, fullscreen;
+    bool ownsContentComponent, resizeToFitContent, fullscreen, dragStarted;
     ComponentDragger dragger;
     Rectangle<int> lastNonFullScreenPos;
     ComponentBoundsConstrainer defaultConstrainer;