-
Notifications
You must be signed in to change notification settings - Fork 538
[Signing UX v2] Epic #5416
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
Comments
Please do not merge to dev until we run automated regression tests. @francovenica @liliya-soroka fyi |
@liliya-soroka @francovenica it's merged into dev and can be tested on there. |
I think the single action of clicking signing and the action being already triggered in the review step instead of the receipt step fells kinda confusing. In MM is ok cuz the popup shows up fine, but for native ledger you gotta see that the device is asking for the signing, you reach to that point of "What I do now?" until you realize |
@francovenica good point, we reverted that behavior. So the Sign button will be only on the last screen. |
A parent ticket for the next iteration of Signing UX improvements. See v1.
Links
Designs
Epic on Notion
The text was updated successfully, but these errors were encountered: