[j.mask132b] minor cleanup, projections are segfaulting anyway ()

Merging branch 'j.mask132b'  into 'main'.

See merge request !2273

GUI/View/Scene/MaskGraphicsScene.cpp

@@ -53,7 +53,6 @@ template <class T> bool areaContains(QList<QGraphicsItem*> items)
     return false;
 }
 
-
 } // namespace
 
 
@@ -62,7 +61,6 @@ MaskGraphicsScene::MaskGraphicsScene()
     , m_drawing_in_progress(false)
     , m_mask_value(MaskFlags::MASK_ON)
 {
-    setSceneRect(0, 0, 800, 600);
     connect(this, &QGraphicsScene::selectionChanged, [this] { onSceneSelectionChanged(); });
 }
 
@@ -116,12 +114,10 @@ void MaskGraphicsScene::dissociateItems()
 
 void MaskGraphicsScene::updateSize(const QSize& newSize)
 {
-    for (QGraphicsItem* graphicsItem : items()) {
-        if (auto* proxy = dynamic_cast<MaskGraphicsProxy*>(graphicsItem)) {
-            proxy->resize(newSize);
-            setSceneRect(0, 0, newSize.width(), newSize.height());
-            proxy->setPos(0, 0);
-        }
+    if (m_proxy) {
+        m_proxy->resize(newSize);
+        setSceneRect(0, 0, newSize.width(), newSize.height());
+        m_proxy->setPos(0, 0);
     }
 }
 
@@ -387,12 +383,14 @@ void MaskGraphicsScene::updateProxyWidget()
     if (m_proxy)
         return;
 
-    m_proxy = new MaskGraphicsProxy; // TODO: check if m_proxy is released at the end
-    m_proxy->setParent(this);
     m_plot = new ColorMap; // TODO: potential memory leak
     m_plot->itemToMap(m_data_item);
+
+    m_proxy = new MaskGraphicsProxy; // TODO: check if m_proxy is released at the end
+    m_proxy->setParent(this);
     if (m_proxy->widget() != m_plot)
         m_proxy->setWidget(m_plot);
+
     addItem(m_proxy);
 }