window: Save splitter state to the cache

Author: mitya57

ReText/__init__.py

@@ -93,6 +93,7 @@ def getBundledIcon(iconName):
 	'lastFileList': [],
 	'lastTabIndex': 0,
 	'recentFileList': [],
+	'splitterState': QByteArray(),
 	'windowGeometry': QByteArray(),
 }
 

ReText/window.py

@@ -40,9 +40,9 @@
 except ImportError:
 	enchant = None
 
-from PyQt6.QtCore import QDir, QFile, QFileInfo, QFileSystemWatcher, \
- QIODevice, QLocale, QMarginsF, QStandardPaths, QTextStream, QTimer, \
- QUrl, Qt, pyqtSlot
+from PyQt6.QtCore import QByteArray, QDir, QFile, QFileInfo, \
+ QFileSystemWatcher, QIODevice, QLocale, QMarginsF, QStandardPaths, \
+ QTextStream, QTimer, QUrl, Qt, pyqtSlot
 from PyQt6.QtGui import QAction, QActionGroup, QColor, QDesktopServices, \
  QIcon, QKeySequence, QPageLayout, QPageSize, \
  QPalette, QTextDocument, QTextDocumentWriter
@@ -92,7 +92,10 @@ def __init__(self, parent=None):
 		self.treeView.setVisible(globalSettings.showDirectoryTree)
 		self.tabWidget = QTabWidget(self.splitter)
 		self.initTabWidget()
-		self.splitter.setSizes([self.width() // 5, self.width() * 4 // 5])
+		if globalCache.splitterState:
+			self.splitter.restoreState(globalCache.splitterState)
+		else:
+			self.splitter.setSizes([self.width() // 5, self.width() * 4 // 5])
 		self.setCentralWidget(self.splitter)
 		self.tabWidget.currentChanged.connect(self.changeIndex)
 		self.tabWidget.tabCloseRequested.connect(self.closeTab)
@@ -1238,6 +1241,13 @@ def closeEvent(self, closeevent):
 				return closeevent.ignore()
 		if globalSettings.saveWindowGeometry:
 			globalCache.windowGeometry = self.saveGeometry()
+			if self.treeView.isVisible():
+				globalCache.splitterState = self.splitter.saveState()
+			else:
+				globalCache.splitterState = QByteArray()
+		else:
+			globalCache.windowGeometry = QByteArray()
+			globalCache.splitterState = QByteArray()
 		if globalSettings.openLastFilesOnStartup:
 			files = [tab.fileName for tab in self.iterateTabs()]
 			globalCache.lastFileList = files

configuration.md

@@ -55,8 +55,14 @@ option name                    | type      | description
 If the type is 'file path', then the value should be an absolute path
 to a file.
 
-These options can be set internally by ReText and should never be set
-manually: `recentFileList`, `lastFileList`, `lastTabIndex` and `windowGeometry`.
+There is also a separate file called `cache.conf` which contains options
+that are set internally by ReText and should never be set manually:
+
+- `lastFileList`
+- `lastTabIndex`
+- `recentFileList`
+- `splitterState`
+- `windowGeometry`
 
 Icon themes
 ===========