Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Description

Adds overflowWrap: "anywhere" CSS property to all Toaster components from react-hot-toast to improve text wrapping behavior in toast notifications. This prevents long text content (such as error messages, URLs, or technical strings) from overflowing toast containers and instead allows text to break and wrap to the next line for better readability.

Changes Made:

  • Updated 5 Toaster components across different app sections:
    • Console main (apps/crossmint-nextjs/src/app/console/components/ClientProviders.tsx)
    • Wallet app (apps/crossmint-nextjs/src/app/(wallet)/components/ClientProviders.tsx)
    • Alpha console (apps/crossmint-nextjs/src/app/alpha/(console)/console/components/ClientProviders.tsx)
    • Checkout flow (apps/crossmint-nextjs/src/app/checkout/components/ClientProviders.tsx)
    • Legacy pages (apps/crossmint-nextjs/src/pages/_app.tsx)

Requested by: Guille ([email protected])
Link to Devin run: https://app.devin.ai/sessions/75b0c010f0d041a892e5bc2d0e460d79

Test plan

⚠️ Manual testing required - These changes have not been tested in a running application yet. Please verify:

  • Visual verification: Test toast notifications with long text content (error messages, URLs, technical strings) to ensure:
    • Text wraps properly instead of overflowing
    • Existing toast styling and themes remain intact
    • Toast positioning and animation behavior is unchanged
  • Cross-browser compatibility: Verify overflowWrap: "anywhere" works correctly in supported browsers
  • Functional testing: Confirm toast notifications still appear and dismiss correctly across all affected app sections

Areas to test manually:

  • Console error/success toasts
  • Wallet operation notifications
  • Checkout flow messages
  • Alpha console notifications
  • Legacy page toasts

Package updates

No package updates required - this is a CSS property addition only.


Review Focus Areas:

  • Verify all Toaster instances from react-hot-toast were found and updated
  • Check that no existing toast styling or functionality is broken
  • Confirm the CSS overflowWrap property achieves the intended text wrapping improvement

@devin-ai-integration
Copy link
Contributor Author

Original prompt from Guille
@Devin Addd this prop to all Toasters from `react-hot-toast`
```toastOptions={{ style: { overflowWrap: "anywhere" } }} ```

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@changeset-bot
Copy link

changeset-bot bot commented Aug 6, 2025

⚠️ No Changeset found

Latest commit: 7ebe8f5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Aug 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
smart-wallet-auth-demo ⬜️ Ignored (Inspect) Aug 6, 2025 2:25pm

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.

0 participants