BuidlGuidl / burnerwallet

https://burner.buidlguidl.com
MIT License
0 stars 1 forks source link

Add switch network drawer #71

Closed damianmarti closed 2 weeks ago

damianmarti commented 3 weeks ago

Added the switch network feature as a drawer.

localhost_3000_ (51)

But I'm having an issue when the "Warning: Your Burner Wallet Has Funds!" is shown after switching the current network from this drawer. When you click the "Close Warning" button, the drawer confirms the transaction is closed.

warning

Tried to set the modal property from the Drawer to false, so the Confirmation Drawer does not close when the user interacts outside the drawer, but this doesn't work when the user interacts with a form component, like when the user clicks the "Close Warning" button.

I will check the Drawer issues if this is a reported bug, but maybe @ChangoMan has some tips about it since he implemented the drawers inside our project. It can be reproduced by adding modal false to the send drawer, the drawer will not close if you click outside the drawer but it gets closed when you click the networks select.

vercel[bot] commented 3 weeks ago

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

Name Status Preview Comments Updated (UTC)
burnerwallet-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 14, 2024 2:15pm
damianmarti commented 2 weeks ago

I tested this and I'm able to reproduce the bug. I am not sure how to fix it, there is something weird between mixing Daisy UI modals and the Radix UI drawers/modals.

I do have a solution though, I can use Radix UI for that Balance Warning Modal. From my testing, this fixes the issue. The PR for that is here: #72

@ChangoMan Working great now!! Thanks!!!