fix: improve ui

3 files changed, 11 insertions, 33 deletions

diff --git a/src/ui/dialog/import_export/ExportKeyPackageDialog.cpp b/src/ui/dialog/import_export/ExportKeyPackageDialog.cpp
index 312cd946..b58d09c1 100644
--- a/src/ui/dialog/import_export/ExportKeyPackageDialog.cpp
+++ b/src/ui/dialog/import_export/ExportKeyPackageDialog.cpp
@@ -53,6 +53,10 @@ GpgFrontend::UI::ExportKeyPackageDialog::ExportKeyPackageDialog(
     auto file_name = QFileDialog::getSaveFileName(
         this, _("Export Key Package"), ui_->nameValueLabel->text() + ".gfepack",
         QString(_("Key Package")) + " (*.gfepack);;All Files (*)");
+
+    // check path
+    if (file_name.isEmpty()) return;
+
     ui_->outputPathLabel->setText(file_name);
   });
 
@@ -62,6 +66,9 @@ GpgFrontend::UI::ExportKeyPackageDialog::ExportKeyPackageDialog(
         ui_->nameValueLabel->text() + ".key",
         QString(_("Key File")) + " (*.key);;All Files (*)");
 
+    // check path
+    if (file_name.isEmpty()) return;
+
     if (!KeyPackageOperator::GeneratePassphrase(file_name.toStdString(),
                                                 passphrase_)) {
       QMessageBox::critical(
diff --git a/src/ui/dialog/keypair_details/KeyDetailsDialog.cpp b/src/ui/dialog/keypair_details/KeyDetailsDialog.cpp
index ed578aa7..9ac60a73 100644
--- a/src/ui/dialog/keypair_details/KeyDetailsDialog.cpp
+++ b/src/ui/dialog/keypair_details/KeyDetailsDialog.cpp
@@ -56,11 +56,6 @@ KeyDetailsDialog::KeyDetailsDialog(const GpgKey& key, QWidget* parent)
   this->setWindowTitle(_("Key Details"));
   this->setModal(true);
 
-  // this->setMinimumSize({520, 600});
-
-  // move to center of the parent
-  this->movePosition2CenterOfParent();
-
   this->show();
 }
 }  // namespace GpgFrontend::UI
diff --git a/src/ui/dialog/keypair_details/KeyPairOperaTab.cpp b/src/ui/dialog/keypair_details/KeyPairOperaTab.cpp
index 13c857a0..c5d0670a 100644
--- a/src/ui/dialog/keypair_details/KeyPairOperaTab.cpp
+++ b/src/ui/dialog/keypair_details/KeyPairOperaTab.cpp
@@ -333,34 +333,10 @@ void KeyPairOperaTab::slot_gen_revoke_cert() {
 
   if (dialog.exec()) m_output_file_name = dialog.selectedFiles().front();
 
-  if (!m_output_file_name.isEmpty())
-    CommonUtils::GetInstance()->SlotExecuteGpgCommand(
-        {"--command-fd", "0", "--status-fd", "1", "--no-tty", "-o",
-         m_output_file_name, "--gen-revoke", m_key_.GetFingerprint().c_str()},
-        [](QProcess* proc) -> void {
-          // Code From Gpg4Win
-          while (proc->canReadLine()) {
-            const QString line = QString::fromUtf8(proc->readLine()).trimmed();
-            SPDLOG_DEBUG("line: {}", line.toStdString());
-            if (line == QLatin1String("[GNUPG:] GET_BOOL gen_revoke.okay")) {
-              proc->write("y\n");
-            } else if (line == QLatin1String("[GNUPG:] GET_LINE "
-                                             "ask_revocation_reason.code")) {
-              proc->write("0\n");
-            } else if (line == QLatin1String("[GNUPG:] GET_LINE "
-                                             "ask_revocation_reason.text")) {
-              proc->write("\n");
-            } else if (line ==
-                       QLatin1String(
-                           "[GNUPG:] GET_BOOL openfile.overwrite.okay")) {
-              // We asked before
-              proc->write("y\n");
-            } else if (line == QLatin1String("[GNUPG:] GET_BOOL "
-                                             "ask_revocation_reason.okay")) {
-              proc->write("y\n");
-            }
-          }
-        });
+  if (!m_output_file_name.isEmpty()) {
+    GpgKeyOpera::GetInstance().GenerateRevokeCert(
+        m_key_, m_output_file_name.toStdString());
+  }
 }
 
 void KeyPairOperaTab::slot_modify_password() {