Merge pull request #1216 from srcejon/v7_stack_layout

V7: Add stack window layout

plugins/channelrx/demodais/aisdemodgui.ui

@@ -487,7 +487,7 @@
        <widget class="QComboBox" name="udpFormat">
         <property name="minimumSize">
          <size>
-          <width>60</width>
+          <width>66</width>
           <height>0</height>
          </size>
         </property>

plugins/feature/antennatools/antennatoolsgui.ui

@@ -6,25 +6,25 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>374</width>
+    <width>360</width>
     <height>584</height>
    </rect>
   </property>
   <property name="sizePolicy">
-   <sizepolicy hsizetype="Fixed" vsizetype="Fixed">
+   <sizepolicy hsizetype="Expanding" vsizetype="Fixed">
     <horstretch>0</horstretch>
     <verstretch>0</verstretch>
    </sizepolicy>
   </property>
   <property name="minimumSize">
    <size>
-    <width>374</width>
+    <width>360</width>
     <height>584</height>
    </size>
   </property>
   <property name="maximumSize">
    <size>
-    <width>374</width>
+    <width>560</width>
     <height>584</height>
    </size>
   </property>
@@ -59,7 +59,7 @@
     <string>Calculators</string>
    </property>
    <property name="currentIndex">
-    <number>1</number>
+    <number>0</number>
    </property>
    <widget class="QWidget" name="dipoleTab">
     <attribute name="title">
@@ -70,7 +70,7 @@
       <widget class="QComboBox" name="dipoleFrequencySelect">
        <property name="minimumSize">
         <size>
-         <width>94</width>
+         <width>90</width>
          <height>0</height>
         </size>
        </property>
@@ -266,7 +266,7 @@
        </property>
        <property name="minimumSize">
         <size>
-         <width>94</width>
+         <width>90</width>
          <height>0</height>
         </size>
        </property>

plugins/feature/aprs/aprsgui.cpp

@@ -668,10 +668,6 @@ void APRSGUI::resizeEvent(QResizeEvent* size)
     plotWeather();
     plotTelemetry();
     plotMotion();
-    int maxWidth = getRollupContents()->maximumWidth();
-    int minHeight = getRollupContents()->minimumHeight() + getAdditionalHeight();
-    resize(width() < maxWidth ? width() : maxWidth, minHeight);
-    size->accept();
 }
 
 void APRSGUI::onMenuDialogCalled(const QPoint &p)

plugins/feature/aprs/aprsgui.ui

@@ -11,7 +11,7 @@
    </rect>
   </property>
   <property name="sizePolicy">
-   <sizepolicy hsizetype="Expanding" vsizetype="Fixed">
+   <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
     <horstretch>0</horstretch>
     <verstretch>0</verstretch>
    </sizepolicy>
@@ -22,12 +22,6 @@
     <height>0</height>
    </size>
   </property>
-  <property name="maximumSize">
-   <size>
-    <width>700</width>
-    <height>16777215</height>
-   </size>
-  </property>
   <property name="font">
    <font>
     <family>Liberation Sans</family>
@@ -154,7 +148,7 @@
     </rect>
    </property>
    <property name="sizePolicy">
-    <sizepolicy hsizetype="Preferred" vsizetype="Preferred">
+    <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
      <horstretch>0</horstretch>
      <verstretch>0</verstretch>
     </sizepolicy>
@@ -187,6 +181,12 @@
        <number>0</number>
       </property>
       <widget class="QWidget" name="stationTab">
+       <property name="sizePolicy">
+        <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+         <horstretch>0</horstretch>
+         <verstretch>0</verstretch>
+        </sizepolicy>
+       </property>
        <attribute name="title">
         <string>Stations and Objects</string>
        </attribute>
@@ -568,6 +568,12 @@
            </layout>
           </widget>
           <widget class="QWidget" name="weatherTab">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>0</horstretch>
+             <verstretch>0</verstretch>
+            </sizepolicy>
+           </property>
            <attribute name="title">
             <string>Weather</string>
            </attribute>
@@ -840,6 +846,12 @@
            </layout>
           </widget>
           <widget class="QWidget" name="motionTab">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>0</horstretch>
+             <verstretch>0</verstretch>
+            </sizepolicy>
+           </property>
            <attribute name="title">
             <string>Motion</string>
            </attribute>
@@ -1007,6 +1019,12 @@
            </layout>
           </widget>
           <widget class="QWidget" name="telemetryTab">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>0</horstretch>
+             <verstretch>0</verstretch>
+            </sizepolicy>
+           </property>
            <attribute name="title">
             <string>Telemetry</string>
            </attribute>
@@ -1271,6 +1289,12 @@
            </layout>
           </widget>
           <widget class="QWidget" name="statusTab">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>0</horstretch>
+             <verstretch>0</verstretch>
+            </sizepolicy>
+           </property>
            <attribute name="title">
             <string>Status</string>
            </attribute>
@@ -1323,6 +1347,12 @@
            </layout>
           </widget>
           <widget class="QWidget" name="packetsTab">
+           <property name="sizePolicy">
+            <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+             <horstretch>0</horstretch>
+             <verstretch>0</verstretch>
+            </sizepolicy>
+           </property>
            <attribute name="title">
             <string>Packets</string>
            </attribute>
@@ -1374,6 +1404,12 @@
        </layout>
       </widget>
       <widget class="QWidget" name="messagesTab">
+       <property name="sizePolicy">
+        <sizepolicy hsizetype="Expanding" vsizetype="Expanding">
+         <horstretch>0</horstretch>
+         <verstretch>0</verstretch>
+        </sizepolicy>
+       </property>
        <attribute name="title">
         <string>Messages</string>
        </attribute>
@@ -1453,17 +1489,17 @@
   </widget>
  </widget>
  <customwidgets>
+  <customwidget>
+   <class>ButtonSwitch</class>
+   <extends>QToolButton</extends>
+   <header>gui/buttonswitch.h</header>
+  </customwidget>
   <customwidget>
    <class>RollupContents</class>
    <extends>QWidget</extends>
    <header>gui/rollupcontents.h</header>
    <container>1</container>
   </customwidget>
-  <customwidget>
-   <class>ButtonSwitch</class>
-   <extends>QToolButton</extends>
-   <header>gui/buttonswitch.h</header>
-  </customwidget>
   <customwidget>
    <class>QChartView</class>
    <extends>QGraphicsView</extends>

plugins/feature/simpleptt/simplepttgui.ui

@@ -24,7 +24,7 @@
   </property>
   <property name="maximumSize">
    <size>
-    <width>360</width>
+    <width>560</width>
     <height>155</height>
    </size>
   </property>

plugins/samplesource/sdrplayv3/sdrplayv3gui.ui

@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>410</width>
+    <width>360</width>
     <height>234</height>
    </rect>
   </property>
@@ -18,13 +18,13 @@
   </property>
   <property name="minimumSize">
    <size>
-    <width>410</width>
+    <width>360</width>
     <height>234</height>
    </size>
   </property>
   <property name="maximumSize">
    <size>
-    <width>410</width>
+    <width>360</width>
     <height>234</height>
    </size>
   </property>

plugins/samplesource/testsource/testsourcegui.ui

@@ -6,7 +6,7 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>381</width>
+    <width>360</width>
     <height>331</height>
    </rect>
   </property>
@@ -18,13 +18,13 @@
   </property>
   <property name="minimumSize">
    <size>
-    <width>381</width>
+    <width>360</width>
     <height>331</height>
    </size>
   </property>
   <property name="maximumSize">
    <size>
-    <width>381</width>
+    <width>360</width>
     <height>331</height>
    </size>
   </property>

sdrgui/channel/channelgui.h

@@ -95,7 +95,7 @@ protected:
     void mouseMoveEvent(QMouseEvent* event) override;
     void resetContextMenuType() { m_contextMenuType = ContextMenuNone; }
     void updateIndexLabel();
-    int getAdditionalHeight() const { return 25 + 22; }
+    int getAdditionalHeight() const { return 22 + 22; }  // height of top and bottom bars
     void setHighlighted(bool highlighted);
 
     DeviceType m_deviceType;

sdrgui/device/devicegui.h

@@ -83,7 +83,7 @@ protected:
     void mouseReleaseEvent(QMouseEvent* event) override;
     void mouseMoveEvent(QMouseEvent* event) override;
     void resetContextMenuType() { m_contextMenuType = ContextMenuNone; }
-    int getAdditionalHeight() const { return 25 + 22; }
+    int getAdditionalHeight() const { return 26 + 22; }  // height of top and bottom bars
 
     DeviceType m_deviceType;
     int m_deviceSetIndex;

sdrgui/feature/featuregui.h

@@ -72,7 +72,7 @@ protected:
     void mouseReleaseEvent(QMouseEvent* event) override;
     void mouseMoveEvent(QMouseEvent* event) override;
     void resetContextMenuType() { m_contextMenuType = ContextMenuNone; }
-    int getAdditionalHeight() const { return 25 + 22; }
+    int getAdditionalHeight() const { return 22 + 22; } // height of top and bottom bars
 
     int m_featureIndex;
     QString m_helpURL;

sdrgui/gui/workspace.cpp

@@ -16,6 +16,8 @@
 // along with this program. If not, see <http://www.gnu.org/licenses/>.          //
 ///////////////////////////////////////////////////////////////////////////////////
 
+#include <algorithm>
+
 #include <QHBoxLayout>
 #include <QLabel>
 #include <QPushButton>
@@ -23,14 +25,23 @@
 #include <QMdiArea>
 #include <QMdiSubWindow>
 #include <QFrame>
+#include <QDebug>
+#include <QApplication>
 
 #include "gui/samplingdevicedialog.h"
+#include "gui/rollupcontents.h"
+#include "channel/channelgui.h"
+#include "feature/featuregui.h"
+#include "device/devicegui.h"
+#include "mainspectrum/mainspectrumgui.h"
 #include "workspace.h"
 
 Workspace::Workspace(int index, QWidget *parent, Qt::WindowFlags flags) :
     QDockWidget(parent, flags),
     m_index(index),
-    m_featureAddDialog(this)
+    m_featureAddDialog(this),
+    m_stacking(false),
+    m_userChannelMinWidth(0)
 {
     m_mdi = new QMdiArea(this);
     m_mdi->setHorizontalScrollBarPolicy(Qt::ScrollBarAsNeeded);
@@ -100,6 +111,19 @@ Workspace::Workspace(int index, QWidget *parent, Qt::WindowFlags flags) :
     m_tileSubWindows->setToolTip("Tile sub windows");
     m_tileSubWindows->setFixedSize(20, 20);
 
+    m_stackSubWindows = new QPushButton("S");
+    //QIcon stackSubWindowsIcon(":/stack.png");     // FIXME
+    //m_stackSubWindows->setIcon(stackSubWindowsIcon);
+    m_stackSubWindows->setToolTip("Stack sub windows");
+    m_stackSubWindows->setFixedSize(20, 20);
+
+    m_autoStackSubWindows = new QPushButton("AS");
+    m_autoStackSubWindows->setCheckable(true);
+    //QIcon autoStackSubWindowsIcon(":/autostack.png"); // FIXME
+    //m_autoStackSubWindows->setIcon(autoStackSubWindowsIcon);
+    m_autoStackSubWindows->setToolTip("Automatically stack sub windows");
+    m_autoStackSubWindows->setFixedSize(20, 20);
+
     m_normalButton = new QPushButton();
     QIcon normalIcon(":/dock.png");
     m_normalButton->setIcon(normalIcon);
@@ -122,6 +146,8 @@ Workspace::Workspace(int index, QWidget *parent, Qt::WindowFlags flags) :
     m_titleBarLayout->addWidget(m_vline2);
     m_titleBarLayout->addWidget(m_cascadeSubWindows);
     m_titleBarLayout->addWidget(m_tileSubWindows);
+    m_titleBarLayout->addWidget(m_stackSubWindows);
+    m_titleBarLayout->addWidget(m_autoStackSubWindows);
     m_titleBarLayout->addStretch(1);
     m_titleBarLayout->addWidget(m_normalButton);
     m_titleBarLayout->addWidget(m_closeButton);
@@ -176,6 +202,20 @@ Workspace::Workspace(int index, QWidget *parent, Qt::WindowFlags flags) :
         &Workspace::tileSubWindows
     );
 
+    QObject::connect(
+        m_stackSubWindows,
+        &QPushButton::clicked,
+        this,
+        &Workspace::stackSubWindows
+    );
+
+    QObject::connect(
+        m_autoStackSubWindows,
+        &QPushButton::clicked,
+        this,
+        &Workspace::autoStackSubWindows
+    );
+
     QObject::connect(
         m_normalButton,
         &QPushButton::clicked,
@@ -191,6 +231,7 @@ Workspace::Workspace(int index, QWidget *parent, Qt::WindowFlags flags) :
         this,
         &Workspace::addFeatureEmitted
     );
+
 }
 
 Workspace::~Workspace()
@@ -198,6 +239,8 @@ Workspace::~Workspace()
     qDebug("Workspace::~Workspace");
     delete m_closeButton;
     delete m_normalButton;
+    delete m_autoStackSubWindows;
+    delete m_stackSubWindows;
     delete m_tileSubWindows;
     delete m_cascadeSubWindows;
     delete m_vline2;
@@ -288,15 +331,344 @@ void Workspace::tileSubWindows()
     m_mdi->tileSubWindows();
 }
 
+void Workspace::orderByIndex(QList<ChannelGUI *> &list)
+{
+    std::sort(list.begin(), list.end(),
+        [](const ChannelGUI *a, const ChannelGUI *b) -> bool
+        {
+            if (a->getDeviceSetIndex() == b->getDeviceSetIndex()) {
+                return a->getIndex() < b->getIndex();
+            } else {
+                return a->getDeviceSetIndex() < b->getDeviceSetIndex();
+            }
+        });
+}
+
+void Workspace::orderByIndex(QList<FeatureGUI *> &list)
+{
+    std::sort(list.begin(), list.end(),
+        [](const FeatureGUI *a, const FeatureGUI *b) -> bool
+        {
+            return a->getIndex() < b->getIndex();
+        });
+}
+
+void Workspace::orderByIndex(QList<DeviceGUI *> &list)
+{
+    std::sort(list.begin(), list.end(),
+        [](const DeviceGUI *a, const DeviceGUI *b) -> bool
+        {
+            return a->getIndex() < b->getIndex();
+        });
+}
+
+void Workspace::orderByIndex(QList<MainSpectrumGUI *> &list)
+{
+    std::sort(list.begin(), list.end(),
+        [](const MainSpectrumGUI *a, const MainSpectrumGUI *b) -> bool
+        {
+            return a->getIndex() < b->getIndex();
+        });
+}
+
+// Try to arrange windows somewhat like in earlier versions of SDRangel
+// Devices and fixed size features stacked on left
+// Spectrum and expandable features stacked in centre
+// Channels stacked on right
+void Workspace::stackSubWindows()
+{
+    // Set a flag so event handler knows if it's this code or the user that
+    // resizes a window
+    m_stacking = true;
+
+    // Categorise windows according to type
+    QList<QMdiSubWindow *> windows = m_mdi->subWindowList(QMdiArea::CreationOrder);
+    QList<DeviceGUI *> devices;
+    QList<MainSpectrumGUI *> spectrums;
+    QList<ChannelGUI *> channels;
+    QList<FeatureGUI *> fixedFeatures;
+    QList<FeatureGUI *> features;
+
+    for (auto window : windows)
+    {
+        if (window->isVisible())
+        {
+            if (window->inherits("DeviceGUI")) {
+                devices.append(qobject_cast<DeviceGUI *>(window));
+            } else if (window->inherits("MainSpectrumGUI")) {
+                spectrums.append(qobject_cast<MainSpectrumGUI *>(window));
+            } else if (window->inherits("ChannelGUI")) {
+                channels.append(qobject_cast<ChannelGUI *>(window));
+            } else if (window->inherits("FeatureGUI")) {
+                if (window->sizePolicy().verticalPolicy() == QSizePolicy::Fixed) {  // Test vertical, as horizontal can be adjusted a little bit
+                    fixedFeatures.append(qobject_cast<FeatureGUI *>(window));
+                } else {
+                    features.append(qobject_cast<FeatureGUI *>(window));
+                }
+            }
+        }
+    }
+
+    // Order windows by device/feature/channel index
+    orderByIndex(devices);
+    orderByIndex(spectrums);
+    orderByIndex(channels);
+    orderByIndex(fixedFeatures);
+    orderByIndex(features);
+
+    // Spacing between windows
+    const int spacing = 2;
+
+    // Calculate width and height needed for devices
+    int deviceMinWidth = 0;
+    int deviceTotalMinHeight = 0;
+    for (auto window : devices)
+    {
+        int winMinWidth = std::max(window->minimumSizeHint().width(), window->minimumWidth());
+        deviceMinWidth = std::max(deviceMinWidth, winMinWidth);
+        deviceTotalMinHeight += window->minimumSizeHint().height() + spacing;
+    }
+
+    // Calculate width & height needed for spectrums
+    int spectrumMinWidth = 0;
+    int spectrumTotalMinHeight = 0;
+    int expandingSpectrums = 0;
+    for (auto window : spectrums)
+    {
+        int winMinWidth = std::max(window->minimumSizeHint().width(), window->minimumWidth());
+        spectrumMinWidth = std::max(spectrumMinWidth, winMinWidth);
+        spectrumTotalMinHeight += window->minimumSizeHint().height() + spacing;
+        expandingSpectrums++;
+    }
+
+    // Calculate width & height needed for channels
+    int channelMinWidth = m_userChannelMinWidth;
+    int channelTotalMinHeight = 0;
+    int expandingChannels = 0;
+    for (auto window : channels)
+    {
+        int winMinWidth = std::max(window->minimumSizeHint().width(), window->minimumWidth());
+        channelMinWidth = std::max(channelMinWidth, winMinWidth);
+        channelTotalMinHeight += window->minimumSizeHint().height() + spacing;
+        if (window->sizePolicy().verticalPolicy() == QSizePolicy::Expanding) {
+            expandingChannels++;
+        }
+    }
+
+    // Calculate width & height needed for features
+    // These are spilt in to two groups - fixed size and expandable
+    int fixedFeaturesWidth = 0;
+    int fixedFeaturesTotalMinHeight = 0;
+    int featuresMinWidth = 0;
+    int featuresTotalMinHeight = 0;
+    int expandingFeatures = 0;
+    for (auto window : fixedFeatures)
+    {
+        int winMinWidth = std::max(window->minimumSizeHint().width(), window->minimumWidth());
+        fixedFeaturesWidth = std::max(fixedFeaturesWidth, winMinWidth);
+        fixedFeaturesTotalMinHeight += window->minimumSizeHint().height() + spacing;
+    }
+    for (auto window : features)
+    {
+        int winMinWidth = std::max(window->minimumSizeHint().width(), window->minimumWidth());
+        featuresMinWidth = std::max(featuresMinWidth, winMinWidth);
+        featuresTotalMinHeight += window->minimumSizeHint().height() + spacing;
+        expandingFeatures++;
+    }
+
+    // Calculate width for left hand column
+    int devicesFeaturesWidth = std::max(deviceMinWidth, fixedFeaturesWidth);
+    // Calculate min width for centre column
+    int spectrumFeaturesMinWidth = std::max(spectrumMinWidth, featuresMinWidth);
+
+    // Calculate spacing between columns
+    int spacing1 = devicesFeaturesWidth > 0 ? spacing : 0;
+    int spacing2 = spectrumFeaturesMinWidth > 0 ? spacing : 0;
+
+    // Will we need scroll bars?
+    QSize mdiSize = m_mdi->size();
+    int minWidth = devicesFeaturesWidth + spacing1 + spectrumFeaturesMinWidth + spacing2 + channelMinWidth;
+    int minHeight = std::max(std::max(deviceTotalMinHeight + fixedFeaturesTotalMinHeight, channelTotalMinHeight), channelTotalMinHeight + featuresTotalMinHeight);
+    bool requiresHScrollBar = minWidth > mdiSize.width();
+    bool requiresVScrollBar = minHeight > mdiSize.height();
+
+    // Reduce available size if scroll bars needed
+    int sbWidth = qApp->style()->pixelMetric(QStyle::PM_ScrollBarExtent);
+    if (requiresVScrollBar) {
+        mdiSize.setWidth(mdiSize.width() - sbWidth);
+    }
+    if (requiresHScrollBar) {
+        mdiSize.setHeight(mdiSize.height() - sbWidth);
+    }
+
+    // Now position the windows
+    int x = 0;
+    int y = 0;
+
+    // Put devices down left hand side
+    for (auto window : devices)
+    {
+        window->move(x, y);
+        y += window->size().height() + spacing;
+    }
+
+    // Put fixed height features underneath devices
+    // Resize them to be same width
+    for (auto window : fixedFeatures)
+    {
+        window->move(x, y);
+        window->resize(devicesFeaturesWidth, window->size().height());
+        y += window->size().height() + spacing;
+    }
+
+    // Calculate width needed for spectrum and features in the centre - use all available space
+    int spectrumFeaturesWidth = std::max(mdiSize.width() - channelMinWidth - devicesFeaturesWidth - spacing1 - spacing2, spectrumFeaturesMinWidth);
+
+    // Put channels on right hand side
+    // Try to resize them horizontally so they are the same width
+    // Share any available vertical space between expanding channels
+
+    x = devicesFeaturesWidth + spacing1 + spectrumFeaturesWidth + spacing2;
+    y = 0;
+    int extraSpacePerWindow;
+    int extraSpaceFirstWindow;
+    if ((channelTotalMinHeight < mdiSize.height()) && (expandingChannels > 0))
+    {
+        extraSpacePerWindow = (mdiSize.height() - channelTotalMinHeight) / expandingChannels;
+        extraSpaceFirstWindow = (mdiSize.height() - channelTotalMinHeight) % expandingChannels;
+    }
+    else
+    {
+        extraSpacePerWindow = 0;
+        extraSpaceFirstWindow = 0;
+    }
+
+    for (auto window : channels)
+    {
+        window->move(x, y);
+        int channelHeight = window->minimumSizeHint().height();
+        if (window->sizePolicy().verticalPolicy() == QSizePolicy::Expanding)
+        {
+            channelHeight += extraSpacePerWindow + extraSpaceFirstWindow;
+            extraSpaceFirstWindow = 0;
+        }
+        window->resize(channelMinWidth, channelHeight);
+        y += window->size().height() + spacing;
+    }
+
+    // Split remaining space in the middle between spectrums and expandable features, with spectrums stacked on top
+    x = devicesFeaturesWidth + spacing1;
+    y = 0;
+    if ((spectrumTotalMinHeight + featuresTotalMinHeight < mdiSize.height()) && (expandingSpectrums + expandingFeatures > 0))
+    {
+        int h = mdiSize.height() - spectrumTotalMinHeight - featuresTotalMinHeight;
+        int f = expandingSpectrums + expandingFeatures;
+        extraSpacePerWindow = h / f;
+        extraSpaceFirstWindow = h % f;
+    }
+    else
+    {
+        extraSpacePerWindow = 0;
+        extraSpaceFirstWindow = 0;
+    }
+
+    for (auto window : spectrums)
+    {
+        window->move(x, y);
+        int w = spectrumFeaturesWidth;
+        int h = window->minimumSizeHint().height() + extraSpacePerWindow + extraSpaceFirstWindow;
+        window->resize(w, h);
+        extraSpaceFirstWindow = 0;
+        y += window->size().height() + spacing;
+    }
+    for (auto window : features)
+    {
+        window->move(x, y);
+        int w = spectrumFeaturesWidth;
+        int h = window->minimumSizeHint().height() + extraSpacePerWindow + extraSpaceFirstWindow;
+        window->resize(w, h);
+        extraSpaceFirstWindow = 0;
+        y += window->size().height() + spacing;
+    }
+
+    m_stacking = false;
+}
+
+void Workspace::autoStackSubWindows()
+{
+    // FIXME: Need to save whether this is checked as a preference
+    if (m_autoStackSubWindows->isChecked()) {
+        stackSubWindows();
+    }
+}
+
+void Workspace::resizeEvent(QResizeEvent *event)
+{
+    QDockWidget::resizeEvent(event);
+    autoStackSubWindows();
+}
+
 void Workspace::addToMdiArea(QMdiSubWindow *sub)
 {
+    // Add event handler to auto-stack when sub window shown or hidden
+    sub->installEventFilter(this);
+    // Can't use Close event, as it's before window is closed, so
+    // catch sub-window destroyed signal instead
+    connect(sub, &QObject::destroyed, this, &Workspace::autoStackSubWindows);
     m_mdi->addSubWindow(sub);
     sub->show();
+    // Auto-stack when sub-window's  widgets are rolled up
+    ChannelGUI *channel = qobject_cast<ChannelGUI *>(sub);
+    if (channel) {
+        connect(channel->getRollupContents(), &RollupContents::widgetRolled, this, &Workspace::autoStackSubWindows);
+    }
+    FeatureGUI *feature = qobject_cast<FeatureGUI *>(sub);
+    if (feature) {
+        connect(feature->getRollupContents(), &RollupContents::widgetRolled, this, &Workspace::autoStackSubWindows);
+    }
 }
 
 void Workspace::removeFromMdiArea(QMdiSubWindow *sub)
 {
     m_mdi->removeSubWindow(sub);
+    sub->removeEventFilter(this);
+    disconnect(sub, &QObject::destroyed, this, &Workspace::autoStackSubWindows);
+    ChannelGUI *channel = qobject_cast<ChannelGUI *>(sub);
+    if (channel) {
+        disconnect(channel->getRollupContents(), &RollupContents::widgetRolled, this, &Workspace::autoStackSubWindows);
+    }
+    FeatureGUI *feature = qobject_cast<FeatureGUI *>(sub);
+    if (feature) {
+        disconnect(feature->getRollupContents(), &RollupContents::widgetRolled, this, &Workspace::autoStackSubWindows);
+    }
+}
+
+bool Workspace::eventFilter(QObject *obj, QEvent *event)
+{
+    if (event->type() == QEvent::Show)
+    {
+        autoStackSubWindows();
+    }
+    else if (event->type() == QEvent::Hide)
+    {
+        autoStackSubWindows();
+    }
+    else if (event->type() == QEvent::Resize)
+    {
+        if (!m_stacking && m_autoStackSubWindows->isChecked())
+        {
+            ChannelGUI *channel = qobject_cast<ChannelGUI *>(obj);
+            if (channel)
+            {
+                // Allow width of channels column to be set by user when they
+                // resize a channel window
+                QResizeEvent *resizeEvent = static_cast<QResizeEvent *>(event);
+                m_userChannelMinWidth = resizeEvent->size().width();
+                stackSubWindows();
+            }
+        }
+    }
+    return QDockWidget::eventFilter(obj, event);
 }
 
 int Workspace::getNumberOfSubWindows() const

sdrgui/gui/workspace.h

@@ -31,6 +31,10 @@ class QStringList;
 class QMdiArea;
 class QMdiSubWindow;
 class QFrame;
+class ChannelGUI;
+class FeatureGUI;
+class DeviceGUI;
+class MainSpectrumGUI;
 
 class SDRGUI_API Workspace : public QDockWidget
 {
@@ -49,6 +53,10 @@ public:
     QByteArray saveMdiGeometry();
     void restoreMdiGeometry(const QByteArray& blob);
     QList<QMdiSubWindow *> getSubWindowList() const;
+    void orderByIndex(QList<ChannelGUI *> &list);
+    void orderByIndex(QList<FeatureGUI *> &list);
+    void orderByIndex(QList<DeviceGUI *> &list);
+    void orderByIndex(QList<MainSpectrumGUI *> &list);
 
 private:
     int m_index;
@@ -61,6 +69,8 @@ private:
     QFrame *m_vline2;
     QPushButton *m_cascadeSubWindows;
     QPushButton *m_tileSubWindows;
+    QPushButton *m_stackSubWindows;
+    QPushButton *m_autoStackSubWindows;
     QWidget *m_titleBar;
     QHBoxLayout *m_titleBarLayout;
     QLabel *m_titleLabel;
@@ -68,6 +78,12 @@ private:
     QPushButton *m_closeButton;
     FeatureAddDialog m_featureAddDialog;
     QMdiArea *m_mdi;
+    bool m_stacking;                // Set when stackSubWindows() is running
+    int m_userChannelMinWidth;      // Minimum width of channels column for stackSubWindows(), set by user resizing a channel window
+
+protected:
+    void resizeEvent(QResizeEvent *event) override;
+    bool eventFilter(QObject *obj, QEvent *event) override;
 
 private slots:
     void addRxDeviceClicked();
@@ -77,6 +93,8 @@ private slots:
     void featurePresetsDialog();
     void cascadeSubWindows();
     void tileSubWindows();
+    void stackSubWindows();
+    void autoStackSubWindows();
     void addFeatureEmitted(int featureIndex);
     void toggleFloating();