Fixed restore procedure and deletion of floating widgts

Author: Uwe Kindler

src/DockContainerWidget.cpp

@@ -430,7 +430,9 @@ bool DockContainerWidgetPrivate::restoreDockArea(QDataStream& stream,
 
 		qDebug() << "Dock Widget found - parent " << DockWidget->parent();
 		DockArea->addDockWidget(DockWidget);
-		DockWidget->toggleView(!Closed);
+
+		DockArea->hide();
+		DockWidget->setProperty("closed", Closed);
 		DockWidget->setProperty("dirty", false);
 	}
 
@@ -444,9 +446,13 @@ bool DockContainerWidgetPrivate::restoreDockArea(QDataStream& stream,
 		delete DockArea;
 		DockArea = nullptr;
 	}
+	else
+	{
+		DockArea->setProperty("currentIndex", CurrentIndex);
+	}
 	CreatedWidget = DockArea;
 	DockAreas.append(DockArea);
-	DockArea->setCurrentIndex(CurrentIndex);
+	//DockArea->setCurrentIndex(CurrentIndex);
 	return true;
 }
 

src/DockManager.cpp

@@ -37,13 +37,15 @@
 #include <QDebug>
 #include <QFile>
 #include <QApplication>
+#include <QAction>
 
 #include "FloatingDockContainer.h"
 #include "DockOverlay.h"
 #include "DockWidget.h"
 #include "ads_globals.h"
 #include "DockStateSerialization.h"
 #include "DockWidgetTitleBar.h"
+#include "DockAreaWidget.h"
 
 namespace ads
 {
@@ -199,9 +201,11 @@ bool DockManagerPrivate::restoreState(const QByteArray &state,  int version)
     // Delete remaining empty floating widgets
     int FloatingWidgetIndex = i - 1;
     int DeleteCount = FloatingWidgets.count() - FloatingWidgetIndex;
+    /*std::cout << "DeleteCount " << DeleteCount << " FloatingWidgets.count() "
+    	<< FloatingWidgets.count() << " FloatingWdgetIndex " << FloatingWidgetIndex << std::endl;*/
     for (int i = 0; i < DeleteCount; ++i)
     {
-    	FloatingWidgets[FloatingWidgetIndex]->deleteLater();
+    	FloatingWidgets[FloatingWidgetIndex + i]->deleteLater();
     }
 
     return Result;
@@ -335,17 +339,11 @@ bool CDockManager::restoreState(const QByteArray &state, int version)
     	DockWidget->setProperty("dirty", true);
     }
 
-    //this->hide();
-    QMainWindow* MainWindow = internal::findParent<QMainWindow*>(this);
-    MainWindow->hide();
-    QApplication::processEvents();
     if (!d->restoreState(state, version))
     {
     	qDebug() << "restoreState: Error restoring state!!!!!!!";
     	return false;
     }
-    MainWindow->show();
-   // this->show();
 
     // All dock widgets, that have not been processed in the restore state
     // function are invisible to the user now and have no assigned dock area
@@ -357,6 +355,23 @@ bool CDockManager::restoreState(const QByteArray &state, int version)
     	{
     		DockWidget->flagAsUnassigned();
     	}
+    	else if (!DockWidget->property("closed").toBool())
+    	{
+    		DockWidget->toggleView(true);
+    	}
+    }
+
+    for (auto DockContainer : d->Containers)
+    {
+    	for (int i = 0; i < DockContainer->dockAreaCount(); ++i)
+    	{
+    		CDockAreaWidget* DockArea = DockContainer->dockArea(i);
+    		int CurrentIndex = DockArea->property("currentIndex").toInt();
+    		if (CurrentIndex < DockArea->count() && DockArea->count() > 1 && CurrentIndex > -1)
+    		{
+    			DockArea->setCurrentIndex(CurrentIndex);
+    		}
+    	}
     }
 
     return true;

src/DockWidget.h

@@ -39,6 +39,7 @@ class CDockWidgetTitleBar;
 class CDockManager;
 class CDockContainerWidget;
 class CDockAreaWidget;
+struct DockContainerWidgetPrivate;
 
 /**
  * The QDockWidget class provides a widget that can be docked inside a
@@ -56,6 +57,7 @@ class CDockWidget : public QFrame
 	friend class CDockAreaWidget;
 	friend class CFloatingDockContainer;
 	friend class CDockManager;
+	friend class DockContainerWidgetPrivate;
 
 	/**
 	 * Assigns the dock manager that manages this dock widget