From 45bd7cfeb7ab20d32d3523e490fe76643502470d Mon Sep 17 00:00:00 2001
From: Christian Herz <christianherz1985@googlemail.com>
Date: Fri, 27 Apr 2018 10:40:37 -0400
Subject: [PATCH] ENH: SegmentEditor used too much space after returning to
 None effect

Removed spacer from ui file and setting QSizePolicy to certain parts
---
 .../Resources/UI/qMRMLSegmentEditorWidget.ui      | 15 +--------------
 .../Widgets/qMRMLSegmentEditorWidget.cxx          |  1 -
 Modules/Scripted/SegmentEditor/SegmentEditor.py   |  3 +++
 3 files changed, 4 insertions(+), 15 deletions(-)

diff --git a/Modules/Loadable/Segmentations/Widgets/Resources/UI/qMRMLSegmentEditorWidget.ui b/Modules/Loadable/Segmentations/Widgets/Resources/UI/qMRMLSegmentEditorWidget.ui
index fca6c97c13d..a7ce7fceef9 100644
--- a/Modules/Loadable/Segmentations/Widgets/Resources/UI/qMRMLSegmentEditorWidget.ui
+++ b/Modules/Loadable/Segmentations/Widgets/Resources/UI/qMRMLSegmentEditorWidget.ui
@@ -217,7 +217,7 @@
    <item>
     <widget class="QGroupBox" name="EffectsGroupBox">
      <property name="sizePolicy">
-      <sizepolicy hsizetype="Preferred" vsizetype="Preferred">
+      <sizepolicy hsizetype="Preferred" vsizetype="Maximum">
        <horstretch>0</horstretch>
        <verstretch>0</verstretch>
       </sizepolicy>
@@ -409,19 +409,6 @@
      </layout>
     </widget>
    </item>
-   <item>
-    <spacer name="verticalSpacer">
-     <property name="orientation">
-      <enum>Qt::Vertical</enum>
-     </property>
-     <property name="sizeHint" stdset="0">
-      <size>
-       <width>20</width>
-       <height>40</height>
-      </size>
-     </property>
-    </spacer>
-   </item>
   </layout>
  </widget>
  <customwidgets>
diff --git a/Modules/Loadable/Segmentations/Widgets/qMRMLSegmentEditorWidget.cxx b/Modules/Loadable/Segmentations/Widgets/qMRMLSegmentEditorWidget.cxx
index 60cca1eaf91..c1c95c6bdff 100644
--- a/Modules/Loadable/Segmentations/Widgets/qMRMLSegmentEditorWidget.cxx
+++ b/Modules/Loadable/Segmentations/Widgets/qMRMLSegmentEditorWidget.cxx
@@ -1544,7 +1544,6 @@ void qMRMLSegmentEditorWidget::updateEffectsSectionFromMRML()
       }
     activeEffect->optionsFrame()->setMinimumHeight(activeEffect->optionsFrame()->sizeHint().height());
     activeEffect->optionsLayout()->activate();
-    this->setMinimumHeight(this->sizeHint().height());
     }
   else
     {
diff --git a/Modules/Scripted/SegmentEditor/SegmentEditor.py b/Modules/Scripted/SegmentEditor/SegmentEditor.py
index adff167c0dc..18941424943 100644
--- a/Modules/Scripted/SegmentEditor/SegmentEditor.py
+++ b/Modules/Scripted/SegmentEditor/SegmentEditor.py
@@ -66,6 +66,9 @@ def setup(self):
     self.addObserver(slicer.mrmlScene, slicer.mrmlScene.EndCloseEvent, self.onSceneEndClose)
     self.addObserver(slicer.mrmlScene, slicer.mrmlScene.EndImportEvent, self.onSceneEndImport)
 
+    if hasattr(self.layout, "addStretch"):
+      self.layout.addStretch()
+
   def selectParameterNode(self):
     # Select parameter set node if one is found in the scene, and create one otherwise
     segmentEditorSingletonTag = "SegmentEditor"