-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
fix: Improve redesigned staking deposit confirmation navigation #13743
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
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good, I left some small feedbacks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewing earn-related changed. Overall this looks good! I just have a minor change request before approving.
app/components/UI/Stake/Views/StakeInputView/StakeInputView.tsx
Outdated
Show resolved
Hide resolved
|
|
Description
This PR aims to improve staking deposit confirmation navigation.
GasImpactModal
flow if redesign enabled.GasImpactModal
Transactions
view on confirm of redesigned staking deposit confirmation.GasFeeDetails
Also adds detailed test given cases above.
Related issues
Fixes:
Manual testing steps
Redesigned staking deposit flow works as expected.
Screenshots/Recordings
Before
After
Smart transactions enabled: (Note that styling might be broken in STX modal - it's not related with this PR)
StakingDeposit.with.STX.mp4
Without smart transactions:
Staking.Deposit.without.STX.mp4
Pre-merge author checklist
Pre-merge reviewer checklist