Y-cross section: fixed auto spacer size issue

Author: PierreRaybaut

plotpy/panels/csection/cswidget.py

@@ -1,4 +1,5 @@
 # -*- coding: utf-8 -*-
+
 from guidata.configtools import get_icon
 from guidata.qthelpers import add_actions, create_action
 from guidata.utils.misc import assert_interfaces_valid
@@ -18,8 +19,13 @@
 
 
 class CrossSectionWidget(PanelWidget):
-    """ """
+    """Base class for cross section panels
+
+    Args:
+        parent: parent widget
+    """
 
+    SIG_RESIZED = QC.Signal()  # emitted when the widget is resized
     PANEL_ID = None
     PANEL_TITLE = _("Cross section tool")
     PANEL_ICON = "csection.png"
@@ -53,6 +59,15 @@ def __init__(self, parent=None):
 
         self.setup_widget()
 
+    # Reimplement Qt method for sending a signal when the widget is resized
+    def resizeEvent(self, event):
+        """
+
+        :param event:
+        """
+        super().resizeEvent(event)
+        self.SIG_RESIZED.emit()
+
     def set_options(self, autoscale=None, autorefresh=None, lockscales=None):
         """
 
@@ -193,7 +208,11 @@ def update_plot(self, obj=None):
 
 
 class XCrossSection(CrossSectionWidget):
-    """X-axis cross section widget"""
+    """X-axis cross section panel
+
+    Args:
+        parent: parent widget
+    """
 
     PANEL_ID = ID_XCS
     OTHER_PANEL_ID = ID_YCS
@@ -319,10 +338,12 @@ def setup_actions(self):
 
 
 class YCrossSection(XCrossSection):
-    """
-    Y-axis cross section widget
-    parent (QWidget): parent widget
-    position (string): "left" or "right"
+    """Y-axis cross section panel
+
+    Args:
+        parent: parent widget
+        position: position of the panel ('left' or 'right')
+        xsection_pos: position of the cross section plot ('top' or 'bottom')
     """
 
     PANEL_ID = ID_YCS
@@ -360,7 +381,11 @@ def adjust_height(self, height):
 
 # Oblique cross section panel
 class ObliqueCrossSection(CrossSectionWidget):
-    """Oblique averaged cross section widget"""
+    """Oblique averaged cross section panel
+
+    Args:
+        parent: parent widget
+    """
 
     PANEL_ID = ID_OCS
     CrossSectionPlotKlass = ObliqueCrossSectionPlot

plotpy/plot/plotwidget.py

@@ -160,8 +160,7 @@ def __set_image_layout(self) -> None:
 
         # Avoid circular import
         # pylint: disable=import-outside-toplevel
-        from plotpy.panels.csection.cswidget import (XCrossSection,
-                                                     YCrossSection)
+        from plotpy.panels.csection.cswidget import XCrossSection, YCrossSection
 
         self.ycsw = YCrossSection(
             self,
@@ -174,6 +173,7 @@ def __set_image_layout(self) -> None:
         self.xcsw.setVisible(self.options.show_xsection)
 
         self.xcsw.SIG_VISIBILITY_CHANGED.connect(self.__xcsw_is_visible)
+        self.xcsw.SIG_RESIZED.connect(self.__adjust_ycsw_height)
 
         self.xcsw_splitter = QW.QSplitter(QC.Qt.Orientation.Vertical, self)
         if self.options.xsection_pos == "top":
@@ -182,9 +182,6 @@ def __set_image_layout(self) -> None:
         else:
             self.xcsw_splitter.addWidget(self.plot)
             self.xcsw_splitter.addWidget(self.xcsw)
-        self.xcsw_splitter.splitterMoved.connect(
-            lambda pos, index: self.__adjust_ycsw_height()
-        )
 
         self.ycsw_splitter = QW.QSplitter(QC.Qt.Orientation.Horizontal, self)
         if self.options.ysection_pos == "left":