]> Nutra Git (v1) - gamesguru/feather.git/commitdiff
Trezor: fix splash dialog for ambiguous button code
authortobtoht <thotbot@protonmail.com>
Fri, 2 Jul 2021 16:06:19 +0000 (18:06 +0200)
committertobtoht <thotbot@protonmail.com>
Fri, 2 Jul 2021 16:06:19 +0000 (18:06 +0200)
src/MainWindow.cpp

index 920a318e600217d93a582867c2b635ea235d84e9..b0e8aed5ebeae2569888b9eec7923a1afb81e739 100644 (file)
@@ -1121,13 +1121,17 @@ void MainWindow::onDeviceError(const QString &error) {
 void MainWindow::onDeviceButtonRequest(quint64 code) {
     if (m_ctx->wallet->isTrezor()) {
         switch (code) {
-            case 8: // Confirm refresh: Do you really want to start refresh?
+            case 8:
             {
-                if (m_constructingTransaction) { // This code is also used when signing a tx...
+                // Annoyingly, this code is used for a variety of actions, including:
+                // Confirm refresh: Do you really want to start refresh?
+                // Confirm export: Do you really want to export tx_key?
+
+                if (m_constructingTransaction) { // This code is also used when signing a tx, we handle this elsewhere
                     break;
                 }
 
-                m_splashDialog->setMessage("Confirm refresh on device to proceed.");
+                m_splashDialog->setMessage("Confirm action on device to proceed");
                 m_splashDialog->setIcon(QPixmap(":/assets/images/confirmed.png"));
                 m_splashDialog->show();
                 m_splashDialog->setEnabled(true);