Skip to content

RainbowToasts #6744

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

Open
wants to merge 139 commits into
base: develop
Choose a base branch
from
Open

RainbowToasts #6744

wants to merge 139 commits into from

Conversation

natew
Copy link
Contributor

@natew natew commented Jul 9, 2025

Fixes APP-2818

What changed (plus any additional context for devs)

Added a new toast and expanded toast for Swap, Send, and Mint.

Screen recordings / screenshots

Note the right side of the expanded toast in real transactions should show properly formatted information just like how it's shown in the ActivityList, the dev helpers don't have that data to show:

CleanShot.2025-07-16.at.10.02.09.mp4

(Slight bug on one interaction here I fixed):

CleanShot.2025-07-16.at.09.26.13.mp4

Stacking:

CleanShot 2025-07-16 at 10 05 53@2x

What to test

I've added "Rainbow Toasts" feature flag so you'll need to enable that.

  • Swap, Send, Mint and other dapp transactions
  • Android, iOS
  • Light and dark mode
  • Interactions with both small and expanded toast (swipe left/right to remove small, swipe up/down to remove big)
  • After success it should clear itself after 3 seconds
  • Should stack toasts if >1 appears at a time, limited to max 3 mini toasts

Copy link

linear bot commented Jul 9, 2025

@natew
Copy link
Contributor Author

natew commented Jul 15, 2025

/testflight

@natew natew marked this pull request as ready for review July 17, 2025 00:42
@jinchung
Copy link
Member

Launch in simulator or device for 7832a79

@jinchung
Copy link
Member

Launch in simulator or device for 15c4fc6

@jinchung
Copy link
Member

Launch in simulator or device for fbb402a

@jinchung
Copy link
Member

Launch in simulator or device for f6e011c

@jinchung
Copy link
Member

Launch in simulator or device for ac69a43

@jinchung jinchung marked this pull request as draft August 13, 2025 23:18
@janicduplessis janicduplessis marked this pull request as ready for review August 19, 2025 02:04
@jinchung
Copy link
Member

Launch in simulator or device for 40bf9de

@jinchung
Copy link
Member

Launch in simulator or device for 23b67b1

@jinchung
Copy link
Member

Launch in simulator or device for 8dfc9c3

Copy link
Contributor

@ibrahimtaveras00 ibrahimtaveras00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All issues have been resolved, QA Passed 👍🏽

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

Successfully merging this pull request may close these issues.

7 participants