send: disable send button during tx broadcast

src/MainWindow.cpp

@@ -1577,7 +1577,7 @@ void MainWindow::onKeysCorrupted() {
     if (!m_criticalWarningShown) {
         m_criticalWarningShown = true;
         Utils::showError(this, "Wallet keys are corrupted", "WARNING!\n\nTo prevent LOSS OF FUNDS do NOT continue to use this wallet file.\n\nRestore your wallet from seed.\n\nPlease report this incident to the Feather developers.\n\nWARNING!");
-        m_sendWidget->disableSendButton();
+        m_sendWidget->disallowSending();
     }
 }
 

src/SendWidget.cpp

@@ -32,8 +32,10 @@ SendWidget::SendWidget(Wallet *wallet, QWidget *parent)
     QValidator *validator = new QRegularExpressionValidator(rx, this);
     ui->lineAmount->setValidator(validator);
 
-    connect(m_wallet, &Wallet::initiateTransaction, this, &SendWidget::onInitiateTransaction);
+    connect(m_wallet, &Wallet::initiateTransaction, this, &SendWidget::disableSendButton);
-    connect(m_wallet, &Wallet::transactionCreated, this, &SendWidget::onEndTransaction);
+    connect(m_wallet, &Wallet::transactionCreated, this, &SendWidget::enableSendButton);
+    connect(m_wallet, &Wallet::beginCommitTransaction, this, &SendWidget::disableSendButton);
+    connect(m_wallet, &Wallet::transactionCommitted, this, &SendWidget::enableSendButton);
 
     connect(WalletManager::instance(), &WalletManager::openAliasResolved, this, &SendWidget::onOpenAliasResolved);
 
@@ -349,18 +351,19 @@ void SendWidget::payToMany() {
     ui->lineAddress->payToMany();
 }
 
-void SendWidget::onInitiateTransaction() {
+void SendWidget::disableSendButton() {
     ui->btnSend->setEnabled(false);
 }
 
-void SendWidget::onEndTransaction() {
+void SendWidget::enableSendButton() {
-    if (!m_sendDisabled) {
+    if (m_disallowSending) {
-        ui->btnSend->setEnabled(true);
+        return;
     }
+    ui->btnSend->setEnabled(true);
 }
 
-void SendWidget::disableSendButton() {
+void SendWidget::disallowSending() {
-    m_sendDisabled = true;
+    m_disallowSending = true;
     ui->btnSend->setEnabled(false);
 }
 

src/SendWidget.h

@@ -38,11 +38,12 @@ public slots:
     void updateConversionLabel();
     void onOpenAliasResolved(const QString &openAlias, const QString &address, bool dnssecValid);
     void onPreferredFiatCurrencyChanged();
-    void disableSendButton();
     void setWebsocketEnabled(bool enabled);
 
-    void onInitiateTransaction();
+    void disableSendButton();
-    void onEndTransaction();
+    void enableSendButton();
+
+    void disallowSending();
 
 private slots:
     void onDataPasted(const QString &data);
@@ -57,7 +58,7 @@ private:
 
     QScopedPointer<Ui::SendWidget> ui;
     Wallet *m_wallet;
-    bool m_sendDisabled = false;
+    bool m_disallowSending = false;
 };
 
 #endif // FEATHER_SENDWIDGET_H

src/libwalletqt/Wallet.cpp

@@ -834,6 +834,8 @@ void Wallet::onTransactionCreated(Monero::PendingTransaction *mtx, const QVector
 // Phase 3: Commit or dispose
 
 void Wallet::commitTransaction(PendingTransaction *tx, const QString &description) {
+    emit beginCommitTransaction();
+
     // Clear list of selected transfers
     this->setSelectedInputs({});
 

src/libwalletqt/Wallet.h

@@ -431,6 +431,7 @@ signals:
     void deviceButtonPressed();
     void deviceError(const QString &message);
     void walletPassphraseNeeded(bool onDevice);
+    void beginCommitTransaction();
     void transactionCommitted(bool status, PendingTransaction *t, const QStringList& txid, const QMap<QString, QString> &txHexMap);
     void deviceShowAddressShowed();
     void transactionProofVerified(TxProofResult result);