Fix issue with qt6.8 (#66)

It seems that dialog->exec() may have some issue if a dialog opens
another dialog. Try avoid use QDialog::exec() to show main window,
use show() instead.

Author: wengxt

qt5/guiwrapper/mainwindow.cpp

@@ -9,18 +9,21 @@
 #include "fcitxqtconfiguifactory.h"
 #include "fcitxqtcontrollerproxy.h"
 #include "fcitxqtwatcher.h"
+#include <QCloseEvent>
 #include <QDebug>
 #include <QLocale>
 #include <QMessageBox>
 #include <QPushButton>
+#include <QShowEvent>
+#include <QWidget>
 #include <QWindow>
 #include <fcitx-utils/i18n.h>
 
 namespace fcitx {
 
 MainWindow::MainWindow(const QString &path, FcitxQtConfigUIWidget *pluginWidget,
                        QWidget *parent)
-    : QDialog(parent), path_(path), watcher_(new FcitxQtWatcher(this)),
+    : QWidget(parent), path_(path), watcher_(new FcitxQtWatcher(this)),
       pluginWidget_(pluginWidget), proxy_(0) {
     setupUi(this);
     watcher_->setConnection(QDBusConnection::sessionBus());
@@ -145,6 +148,12 @@ void MainWindow::showEvent(QShowEvent *event) {
     connect(this, &QObject::destroyed, mainWindow, &QObject::deleteLater);
     subWindow->setTransientParent(mainWindow);
 
-    QDialog::showEvent(event);
+    QWidget::showEvent(event);
+}
+
+void MainWindow::closeEvent(QCloseEvent *event) {
+    QWidget::closeEvent(event);
+
+    qApp->quit();
 }
 } // namespace fcitx

qt5/guiwrapper/mainwindow.h

@@ -14,12 +14,13 @@
 #include "fcitxqtconfiguiwidget.h"
 #include "ui_mainwindow.h"
 #include <QDBusPendingCallWatcher>
+#include <QWidget>
 
 namespace fcitx {
 
 class FcitxQtControllerProxy;
 class FcitxQtWatcher;
-class MainWindow : public QDialog, public Ui::MainWindow {
+class MainWindow : public QWidget, public Ui::MainWindow {
     Q_OBJECT
 public:
     explicit MainWindow(const QString &path,
@@ -35,6 +36,7 @@ public Q_SLOTS:
 
 protected:
     void showEvent(QShowEvent *event) override;
+    void closeEvent(QCloseEvent *event) override;
 
 private Q_SLOTS:
     void saveFinished();

qt5/guiwrapper/wrapperapp.cpp

@@ -14,6 +14,7 @@
 #include <QWindow>
 #include <fcitx-utils/i18n.h>
 #include <fcitx-utils/standardpath.h>
+#include <qnamespace.h>
 
 namespace fcitx {
 
@@ -69,15 +70,14 @@ void WrapperApp::init() {
         mainWindow_ = new MainWindow(path, widget);
         if (ok && winid) {
             mainWindow_->setParentWindow(winid);
+            mainWindow_->setWindowModality(Qt::WindowModal);
+            mainWindow_->setWindowFlag(Qt::Dialog);
         }
         QMetaObject::invokeMethod(this, "run", Qt::QueuedConnection);
     }
 }
 
-void WrapperApp::run() {
-    mainWindow_->exec();
-    QMetaObject::invokeMethod(this, "quit", Qt::QueuedConnection);
-}
+void WrapperApp::run() { mainWindow_->show(); }
 
 WrapperApp::~WrapperApp() {
     if (mainWindow_) {