Changeset c41ce0 for src/UIElements/Qt4/QtMainWindow.cpp

Timestamp:

Aug 11, 2025, 5:43:56 PM (2 months ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.0, stable

Children:

641550

Parents:

6294b6

git-author:

Frederik Heber <frederik.heber@…> (05/03/24 15:25:52)

git-committer:

Frederik Heber <frederik.heber@…> (08/11/25 17:43:56)

Message:

Adds qt_size and qt_position arguments to control Qt window size.

• override is only temporary and is not stored in QSettings.
• this allows using these parameters in scripts to take screenshots in a reliable manner but retain the actual window size for other usages.

File:

• src/UIElements/Qt4/QtMainWindow.cpp (modified) (2 diffs)

src/UIElements/Qt4/QtMainWindow.cpp

@@ -158 +158 @@
   QSettings settings;
   settings.beginGroup("MainWindow");
-  resize(settings.value("size", QSize(400, 400)).toSize());
-  move(settings.value("position", QPoint(200, 200)).toPoint());
+  settingsSize = settings.value("size", QSize(400, 400)).toSize();
+  settingsPosition = settings.value("position", QPoint(200, 200)).toPoint();
+
+  const QStringList args = QCoreApplication::arguments();
+  LOG(2, "DEBUG: QApp's seen arguments are " << args.join(",").toStdString());
+
+  const int indexSize = args.indexOf(QString("--qt_size"));
+  const int indexPosition = args.indexOf(QString("--qt_position"));
+
+  if ((indexSize == -1) || (indexPosition == -1)) {
+    resize(settingsSize);
+    move(settingsPosition);
+  } else {
+    // Take size and position from command-line values instead
+    // NOTE: Qt 4.8's toInt() caps the second argument when it's below 1000 (yes, really!). Hence, we're using stoi()
+    const QSize cmdlineSize = QSize(stoi(args[indexSize + 1].split(",")[0].toStdString()), stoi(args[indexSize + 1].split(",")[1].toStdString()));
+    const QPoint cmdlinePosition = QPoint(stoi(args[indexPosition + 1].split(",")[0].toStdString()), stoi(args[indexPosition + 1].split(",")[1].toStdString()));
+    LOG(2, "DEBUG: Using size and position from the command line: (" << cmdlineSize.width() << "," << cmdlineSize.height()
+        << "), (" << cmdlinePosition.x() << "," << cmdlinePosition.y() << ")");
+    resize(cmdlineSize);
+    move(cmdlinePosition);
+  }
   if (settings.value("maximized", false).toBool())
     showMaximized();
   settings.endGroup();
+  LOG(1, "INFO: Size and position is (" << size().width() << "," << size().height() << "), (" << pos().x() << "," << pos().y() << ")");
 
   connect(glWorldView,SIGNAL(hoverChanged(const atomId_t)), infoBox,SLOT(atomHover(const atomId_t)));
@@ -174 +195 @@
 QtMainWindow::~QtMainWindow()
 {
+  const QStringList args = QCoreApplication::arguments();
+  const bool containsSize = args.contains(QString("--qt_size"));
+  const bool containsPosition = args.contains(QString("--qt_position"));
+
   QSettings settings;
   settings.beginGroup("MainWindow");
-  settings.setValue("size", size());
-  settings.setValue("position", pos());
+  if (!containsSize || !containsPosition) {
+    settings.setValue("size", size());
+    settings.setValue("position", pos());
+  } else {
+    settings.setValue("size", settingsSize);
+    settings.setValue("position", settingsPosition);
+  }
   settings.setValue("maximized", isMaximized());
   settings.endGroup();