feat(web): make the top navigation bar sticky - #966
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview Aligns other sticky/overlay UI with the bar height using Raises Radix portal popups (theme toggle, etc.) above the sticky bar via Minor changeset bumps Reviewed by Cursor Bugbot for commit bc8cf11. Bugbot is set up for automated code reviews on this repo. Configure here. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #966 +/- ##
=======================================
Coverage 86.63% 86.63%
=======================================
Files 195 195
Lines 17938 17938
Branches 1632 1632
=======================================
Hits 15540 15540
Misses 2392 2392
Partials 6 6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d5219c7. Configure here.
The NavBar's invisible mobile menu checkbox is translated 200% above the bar. Inside the sticky wrapper's stacking context it painted above the announcement banner and intercepted clicks on its close button.
|
Does this really need to be sticky? iirc unsticking was intentiona |
Personally, I prefer convenient and quick access to search functions in various places. In contrast, many other tool websites seem to maintain this format. Of course, I genuinely don't know that this was a deliberate design choice. Could you explain your reasoning? Thanks! |

Description
Keep the logo, search box, and theme toggle visible while scrolling.
It allows you to quickly search for information anywhere on the page without having to scroll to the top.
Details
Validation
Related Issues
Check List
node --run testand all tests passed.node --run format:check&node --run lint.