Page MenuHomePhabricator

[cashtab] Success modal animation edits
ClosedPublic

Authored by johnkuney on Sep 25 2025, 20:36.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABC42f1faba09fd: [cashtab] Success modal animation edits
Summary

Modifying the success modal styles and animation to be hopefully a bit more refined

  • adjusted timing
  • removed the progress bars and added a little exit "burst" to the checkmark instead
  • simplified the modal content. Since the modal is only visible for just a moment. I thought the txid and copy button were not really practical. So simplifying to just a link to open the tx

Best way to compare is test it out in master and then in this branch to see if these adjustments feel better, as its pretty quick

Test Plan

preview and check. Send a tx using the url generated send (ie: /#/send?bip21=ecash:.....)

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

johnkuney published this revision for review.Sep 25 2025, 20:44
johnkuney edited the summary of this revision. (Show Details)
johnkuney edited the test plan for this revision. (Show Details)
johnkuney edited the summary of this revision. (Show Details)
bytesofman added a subscriber: bytesofman.

const name change and propagation but otherwise gtg, v nice

cashtab/src/components/Send/styled.ts
69 ↗(On Diff #55881)

we should rename this since the progress bar is gone, say SUCCESS_MODAL_DURATION_MS

the time choice looks good tho, i'm a fan of the "success" experience

This revision now requires changes to proceed.Sep 25 2025, 21:12
johnkuney edited the summary of this revision. (Show Details)
johnkuney edited the test plan for this revision. (Show Details)

update const name

This revision is now accepted and ready to land.Sep 25 2025, 21:49
This revision was automatically updated to reflect the committed changes.