Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove exit option from Confirmed on Trezor pill modal #11802

Open
MiroslavProchazka opened this issue Mar 28, 2024 · 1 comment
Open

Remove exit option from Confirmed on Trezor pill modal #11802

MiroslavProchazka opened this issue Mar 28, 2024 · 1 comment

Comments

@MiroslavProchazka
Copy link
Contributor

Follow up on #11771, please remove the cross on the pill modal upon the SOL send form, after user confirms the transaction on trezor and the modal from #11515 is waiting on SOL Tx confirmation.
This needs to be done because user might click on the cross and break the TX confirmation.

image

@MiroslavProchazka MiroslavProchazka moved this to 🎯 To do in Issues Suite Mar 28, 2024
@MiroslavProchazka MiroslavProchazka moved this from 🎯 To do to 🚫 Blocked in Issues Suite Mar 28, 2024
@Hannsek
Copy link
Contributor

Hannsek commented Mar 28, 2024

I don't think that is the ideal solution. If we remove the cross, user won't be able to exit the flow if he wants to do it.
Plus this would be another exception in our codebase which is 🫤.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚫 Blocked
Development

No branches or pull requests

2 participants