Skip to content

fix: prevent duplicate notification panels on rapid bell clicks#577

Open
saurabhhhcodes wants to merge 3 commits into
indresh404:mainfrom
saurabhhhcodes:fix/duplicate-notification-panels-560
Open

fix: prevent duplicate notification panels on rapid bell clicks#577
saurabhhhcodes wants to merge 3 commits into
indresh404:mainfrom
saurabhhhcodes:fix/duplicate-notification-panels-560

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Closes #560\n\n### Problem\nClicking the notification bell rapidly opened multiple stacked panels because the toggle used a stale closure value.\n\n### Fix\nChanged setShowNotifications(!showNotifications) to setShowNotifications(prev => !prev) so each click toggles from the latest state.\n\n### Files changed\n- src/components/layout/Navbar.jsx

Saurabh Kumar Bajpai added 3 commits June 17, 2026 13:10
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ranker-hub Ready Ready Preview, Comment Jun 17, 2026 8:01am

@github-actions github-actions Bot added backend Backend/Firebase related changes bug Something isn't working frontend Frontend related changes (HTML/CSS/JS/React) gssoc GirlScript Summer of Code gssoc26 GirlScript Summer of Code 2026 nsoc NSoC NSoC'26 NSoC 2026 pending-review PR is pending review labels Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend/Firebase related changes bug Something isn't working frontend Frontend related changes (HTML/CSS/JS/React) gssoc GirlScript Summer of Code gssoc26 GirlScript Summer of Code 2026 nsoc NSoC NSoC'26 NSoC 2026 pending-review PR is pending review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Clicking notification bell multiple times opens duplicate notification panels

1 participant