-
Notifications
You must be signed in to change notification settings - Fork 5.5k
[Bug]: Move ToastContainer and Toasts to Home component #25035
Copy link
Copy link
Closed
Labels
Sev3-lowLow severity; minimal to no impact upon usersLow severity; minimal to no impact upon usersstaleissues and PRs marked as staleissues and PRs marked as staleteam-core-extension-uxCore Extension UX teamCore Extension UX teamtype-bugSomething isn't workingSomething isn't working
Description
Describe the bug
At present, since the <ToastContainer> and <Toast>s are in routes.component.js, these toasts (like the Privacy Policy) can display on top of confirmations. Instead, we should move these to home.component.js so that they only display on the wallet screen.
Expected behavior
No response
Screenshots/Recordings
No response
Steps to reproduce
- Hardcode a toast in
routes.component.jsto display - Open a confirmation
- See the toast
Error messages or log output
No response
Version
develop
Build type
None
Browser
Chrome
Operating system
MacOS
Hardware wallet
No response
Additional context
No response
Severity
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Sev3-lowLow severity; minimal to no impact upon usersLow severity; minimal to no impact upon usersstaleissues and PRs marked as staleissues and PRs marked as staleteam-core-extension-uxCore Extension UX teamCore Extension UX teamtype-bugSomething isn't workingSomething isn't working