Add needsAutoPosition property to monitor

Author: adazem009

include/scratchcpp/monitor.h

@@ -86,6 +86,7 @@ class LIBSCRATCHCPP_EXPORT Monitor : public Entity
         void setDiscrete(bool discrete);
 
         static Rect getInitialPosition(const std::vector<std::shared_ptr<Monitor>> &other, int monitorWidth, int monitorHeight);
+        bool needsAutoPosition() const;
 
     private:
         spimpl::unique_impl_ptr<MonitorPrivate> impl;

src/scratch/monitor.cpp

@@ -180,6 +180,7 @@ int Monitor::x() const
 void Monitor::setX(int x)
 {
     impl->x = x;
+    impl->needsAutoPosition = false;
 
     if (impl->iface)
         impl->iface->onXChanged(x);
@@ -195,6 +196,7 @@ int Monitor::y() const
 void Monitor::setY(int y)
 {
     impl->y = y;
+    impl->needsAutoPosition = false;
 
     if (impl->iface)
         impl->iface->onYChanged(y);
@@ -251,6 +253,12 @@ void Monitor::setDiscrete(bool discrete)
     impl->discrete = discrete;
 }
 
+/*! Returns true if the monitor needs auto positioning. */
+bool Monitor::needsAutoPosition() const
+{
+    return impl->needsAutoPosition;
+}
+
 /*! Returns the initial position of a monitor. */
 Rect Monitor::getInitialPosition(const std::vector<std::shared_ptr<Monitor>> &other, int monitorWidth, int monitorHeight)
 {

src/scratch/monitor_p.h

@@ -32,6 +32,7 @@ struct MonitorPrivate
         double sliderMin = 0;
         double sliderMax = 0;
         bool discrete = false;
+        bool needsAutoPosition = true;
         static IRandomGenerator *rng;
 };
 

test/scratch_classes/monitor_test.cpp

@@ -29,6 +29,7 @@ TEST(MonitorTest, Constructors)
     ASSERT_EQ(monitor.opcode(), "test");
     ASSERT_TRUE(monitor.block());
     ASSERT_EQ(monitor.block()->opcode(), "test");
+    ASSERT_TRUE(monitor.needsAutoPosition());
 }
 
 TEST(MonitorTest, Id)
@@ -189,6 +190,7 @@ TEST(MonitorTest, X)
     EXPECT_CALL(handler, onXChanged(-78));
     monitor.setX(-78);
     ASSERT_EQ(monitor.x(), -78);
+    ASSERT_FALSE(monitor.needsAutoPosition());
 }
 
 TEST(MonitorTest, Y)
@@ -203,6 +205,7 @@ TEST(MonitorTest, Y)
     EXPECT_CALL(handler, onYChanged(150));
     monitor.setY(150);
     ASSERT_EQ(monitor.y(), 150);
+    ASSERT_FALSE(monitor.needsAutoPosition());
 }
 
 TEST(MonitorTest, Visible)