satoshipay / solar

🌞 Stellar wallet. Secure and user-friendly.
https://solarwallet.io
MIT License
191 stars 59 forks source link

Fix no spinner on confirm or retry button #1249

Closed ebma closed 3 years ago

ebma commented 3 years ago

Adds a loading indicator to the retry button.

Adding the 'return' to TransactionSender l. 203 might also fix the sometimes very short spinner on the confirm button because in the ReviewForm the onConfirm() is awaited and since onConfirm() is basically just submitTransaction() adding the return statement here should make onConfirm also await the submission of the signed transaction. But I was not able to test it because it seems like the spinner is working as expected if the horizon response times are reasonably short.

Closes #1245.