feat: make document tabs easily closable via mouse

KDAB · Mar 3, 2025 · f118094 · f118094
1 parent 95ba0b5
commit f118094
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 4 deletions.
diff --git a/src/gui/mainwindow.cpp b/src/gui/mainwindow.cpp
@@ -179,6 +179,7 @@ MainWindow::MainWindow(QWidget *parent)
         m_palette->showPalette();
     });
     connect(ui->actionCloseDocument, &QAction::triggered, this, &MainWindow::closeDocument);
+    connect(ui->tabWidget, &QTabWidget::tabCloseRequested, this, &MainWindow::closeDocument);
 
     // Script
     ui->apiExecutorWidget->hide();
@@ -513,12 +514,25 @@ void MainWindow::saveDocument()
         document->save();
 }
 
-void MainWindow::closeDocument()
+void MainWindow::closeDocument(int closeIndex)
 {
+    const int currentIndex = ui->tabWidget->currentIndex();
+    bool differentIndex = false;
+    if (closeIndex == 0)
+        closeIndex = currentIndex;
+    else
+        differentIndex = currentIndex != closeIndex;
+    if (differentIndex)
+        ui->tabWidget->setCurrentIndex(closeIndex);
+
     auto document = Core::Project::instance()->currentDocument();
-    if (document)
+    if (document) {
         document->close();
-    ui->tabWidget->removeTab(ui->tabWidget->currentIndex());
+        ui->tabWidget->removeTab(closeIndex);
+    }
+
+    if (differentIndex)
+        ui->tabWidget->setCurrentIndex(currentIndex - (closeIndex < currentIndex ? 1 : 0));
 }
 
 void MainWindow::createQrc()

diff --git a/src/gui/mainwindow.h b/src/gui/mainwindow.h
@@ -59,7 +59,7 @@ class MainWindow : public QMainWindow
     void openProject();
     void saveDocument();
     void saveAllDocuments();
-    void closeDocument();
+    void closeDocument(int);
     void openOptions();
     void returnToEditor();
 

diff --git a/src/gui/mainwindow.ui b/src/gui/mainwindow.ui
@@ -35,6 +35,9 @@
       <property name="documentMode">
        <bool>true</bool>
       </property>
+      <property name="tabsClosable">
+       <bool>true</bool>
+      </property>
      </widget>
     </item>
     <item>