feat: (#1154) undo close tab

src/kiwixapp.cpp

@@ -418,7 +418,9 @@ void KiwixApp::createActions()
     CREATE_ACTION_SHORTCUTS(CloseCurrentTabAction, gt("close-tab"), QList<QKeySequence>({QKeySequence(Qt::CTRL | Qt::Key_F4), QKeySequence(Qt::CTRL | Qt::Key_W)}));
 
     CREATE_ACTION_SHORTCUT(ReopenClosedTabAction, gt("reopen-closed-tab"), QKeySequence(Qt::CTRL | Qt::SHIFT | Qt::Key_T));
-    HIDE_ACTION(ReopenClosedTabAction);
+    mpa_actions[ReopenClosedTabAction]->setEnabled(false);
+    connect(mpa_actions[ReopenClosedTabAction], &QAction::triggered,
+            this, &KiwixApp::reopenLastClosedTab);
 
     CREATE_ACTION_SHORTCUT(BrowseLibraryAction, gt("browse-library"), QKeySequence(Qt::CTRL | Qt::Key_E));
     HIDE_ACTION(BrowseLibraryAction);
@@ -589,3 +591,20 @@ QString KiwixApp::getPrevSaveDir() const
   QDir dir(prevSaveDir);
   return dir.exists() ? prevSaveDir : DEFAULT_SAVE_DIR;
 }
+
+void KiwixApp::pushClosedTab(const QString& url, const QString& title) {
+    if (url.isEmpty() || title.isEmpty())
+        return;
+    m_closedTabs.push({url, title});
+    mpa_actions[ReopenClosedTabAction]->setEnabled(true);
+}
+
+void KiwixApp::reopenLastClosedTab() {
+    if (m_closedTabs.isEmpty())
+        return;
+
+    auto tab = m_closedTabs.pop();
+    openUrl(tab.url, true);
+    if (m_closedTabs.isEmpty())
+        mpa_actions[ReopenClosedTabAction]->setEnabled(false);
+}

src/kiwixapp.h

@@ -20,6 +20,7 @@
 
 #include <mutex>
 #include <iostream>
+#include <QStack>
 
 typedef TabBar::TabType TabType;
 
@@ -141,6 +142,17 @@ public slots:
 
     QString findLibraryDirectory();
     void loadAndInstallTranslations(QTranslator& translator, const QString& filename, const QString& directory);
+
+    struct ClosedTabInfo {
+        QString url;
+        QString title;
+    };
+    QStack<ClosedTabInfo> m_closedTabs;
+
+public:
+    void pushClosedTab(const QString& url, const QString& title);
+    bool hasClosedTabs() const { return !m_closedTabs.isEmpty(); }
+    void reopenLastClosedTab();
 };
 
 QString gt(const QString &key);

src/tabbar.cpp

@@ -310,11 +310,21 @@ QStringList TabBar::getTabZimIds() const
 
 void TabBar::closeTab(int index)
 {
-    // The first and last tabs (i.e. the library tab and the + (new tab) button)
-    // cannot be closed
+    // First and last tabs cannot be closed
     if (index <= 0 || index >= this->realTabCount())
         return;
 
+    // Save tab info before closing
+    if (ZimView* zv = qobject_cast<ZimView*>(mp_stackedWidget->widget(index))) {
+        auto webView = zv->getWebView();
+        if (webView) {
+            KiwixApp::instance()->pushClosedTab(
+                webView->url().toString(),
+                webView->title()
+            );
+        }
+    }
+
     if ( index == currentIndex() ) {
         setCurrentIndex(index + 1 == realTabCount() ? index - 1 : index + 1);
     }
@@ -549,3 +559,13 @@ void TabBar::onTabMoved(int from, int to)
 
     KiwixApp::instance()->saveListOfOpenTabs();
 }
+
+void TabBar::contextMenuEvent(QContextMenuEvent *event)
+{
+    int tabIndex = tabAt(event->pos());
+    if (tabIndex == -1) {  // Clicked outside tabs
+        QMenu menu;
+        menu.addAction(KiwixApp::instance()->getAction(KiwixApp::ReopenClosedTabAction));
+        menu.exec(event->globalPos());
+    }
+}

src/tabbar.h

@@ -62,6 +62,7 @@ class TabBar : public QTabBar
     void tabRemoved(int index) override;
     void tabInserted(int index) override;
     void resizeEvent(QResizeEvent *) override;
+    void contextMenuEvent(QContextMenuEvent *event) override;
 
 signals:
     void webActionEnabledChanged(QWebEnginePage::WebAction action, bool enabled);