fix: 🐛 Fix search text not emptied on close

pancakeswap / pancake-frontend

:pancakes: Pancake main features (farms, pools, IFO, lottery, profiles)

https://pancakeswap.finance

GNU General Public License v3.0

2.7k stars 3.56k forks source link

fix: 🐛 Fix search text not emptied on close #10951

Closed chef-ryan closed 1 day ago

chef-ryan commented 1 week ago

PR-Codex overview

This PR focuses on improving the dismissal handling of the AddGaugeModal component by introducing a dismissHandler function. This function resets the search text and calls the onDismiss callback, ensuring a consistent state when the modal is closed.

Detailed summary

Added dismissHandler function using useCallback to reset search text and handle modal dismissal.
Updated the onDismiss prop in <ModalV2> to use dismissHandler instead of directly calling onDismiss.
Changed the onClick prop of the close button to use dismissHandler.
Updated the onClick prop of <BottomAction> to use dismissHandler.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

changeset-bot[bot] commented 1 week ago

⚠️ No Changeset found

Latest commit: fbb55c04b7bdbc4097c0803a2426c32fef171b1f

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[bot] commented 1 week ago

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

Name	Status	Preview	Comments	Updated (UTC)
web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 26, 2024 5:33am

6 Skipped Deployments