fix(web): blur site header behind search palette - #217
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (4)
💤 Files with no reviewable changes (2)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughFixed navigation and mobile overlay stacking indices were lowered. The BrandMenu positioner class and a global CSS rule that raised context-menu positioners above navigation were removed. ChangesNavigation and context-menu stacking
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~5 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to Users should see the search palette above the fixed headers and the mobile navigation overlay below its header. The context menu retains its component layer, and no concrete stacking regression remains; the changes appear ready to merge. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit hops past the header in the night Comment |
There was a problem hiding this comment.
All reported issues were addressed across 3 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Summary
Validation
pnpm --filter web typecheckpnpm --filter web lintpnpm exec oxfmt --check apps/web/src/components/layout/navigation-bar.tsx apps/web/src/components/layout/mini-nav-bar.tsx apps/web/src/styles/globals.cssgit diff --checkSummary by cubic
Fixes stacking so the Fumadocs search palette overlay renders above the fixed site headers instead of underneath them.
Written for commit d6c27c8. Summary will update on new commits.
Summary by CodeRabbit