Skip to content

Commit

Permalink
Merge #2863: [Bug] [GUI] Fix focus on send transaction
Browse files Browse the repository at this point in the history
f7ed184 Fix focus on send (Alessandro Rezzi)

Pull request description:

  This is a trivial but quite annoying bug fix. Basically once a transaction is actually sent from the GUI (so a few seconds after the user presses on "SEND" in the confirmation screen) the focus goes back to the "Recipient address" field regardless the screen the user is currently on.

  Now once the "SEND" button is pressed the focus on "Recipient address" is already (and istantly) gained by the function `showEvent` in send.cpp, so to fix this issue I simply removed the call to `setFocusOnLastEntry` that happens when the transaction is actually sent (which is therefore completely useless).

  This PR also solves #2597 and #1566

ACKs for top commit:
  Liquid369:
    tACK f7ed184
  Fuzzbawls:
    ACK f7ed184

Tree-SHA512: d6c98c3d8af6196b5f63ef0274043e9a3d064f9cba6bc057f2e8188b3c2edbf8c85455eb9767728d7281b60b835748ee0ed48cf2eaf4a973d15ebb557486fb81
  • Loading branch information
Fuzzbawls committed Aug 10, 2023
2 parents d0cf093 + f7ed184 commit 6c26a0e
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/qt/pivx/send.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -457,7 +457,6 @@ void SendWidget::ProcessSend(QList<SendCoinsRecipient>& recipients, bool hasShie
if (sendFinalStep()) {
updateEntryLabels(ptrModelTx->getRecipients());
}
setFocusOnLastEntry();
} else if (!processingResultError->isEmpty()){
inform(*processingResultError);
}
Expand Down

0 comments on commit 6c26a0e

Please sign in to comment.