Fix/shortcut hover flicker#983
Conversation
|
@ramlal-01 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
There was a problem hiding this comment.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
|
I have solved the issue. |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Thanks for the fix! A couple of things to address before merge:
-
** deleted** — This PR removes all 55 lines from . That file documents required environment variables for contributors setting up the project locally. Please restore it.
-
Conflict with #923 — PR #923 fixes the same hover-flicker issue with similar changes to and . Please rebase on latest main and resolve any conflicts.
-
** churn** — The lockfile shows unrelated dependency changes. Please restore it by running
up to date, audited 542 packages in 5s
176 packages are looking for funding
run npm fund for details
16 vulnerabilities (7 moderate, 9 high)
To address issues that do not require attention, run:
npm audit fix
To address all issues (including breaking changes), run:
npm audit fix --force
Run npm audit for details. on a clean checkout of latest main before pushing.
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Thanks for the fix! A couple of things to address before merge:
-
.env.exampledeleted — This PR removes all 55 lines from.env.example. That file documents required environment variables for contributors. Please restore it. -
Conflict with #923 — PR #923 fixes the same hover-flicker issue with overlapping changes to
KeyboardShortcuts.tsxandShortcutsModal.tsx. Please rebase on latest main and resolve conflicts. -
package-lock.jsonchurn — Lockfile shows unrelated dependency changes. Please regenerate it from a clean checkout of latest main.
8a9953f to
d017c91
Compare
|
Thanks for the review!
Please review again @Priyanshu-byte-coder |
🚀 Fix: Shortcuts Panel Hover Flickering
🐛 Issue
The "Shortcuts" panel was behaving inconsistently on hover:
🎯 Expected Behavior
The panel should remain stable in size and layout once opened, without unexpected transitions based on cursor movement.
The layout was being controlled by hover-based styling, where the hover area dynamically changed (width/position).
This caused repeated
mouseenter/mouseleavetriggers, leading to flickering.✅ Fix Implemented
🧪 Testing
🔗 Related Issue
Fixes #917
📌 Result
The shortcuts panel now behaves consistently and provides a smooth user experience without flickering.