feat: add analytics and fix mobile sidebar - #145
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe PR adds Vercel Analytics to the root component and adds a mobile CSS rule that removes backdrop blur when the sidebar is visible. ChangesSite experience updates
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 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. Comment |
There was a problem hiding this comment.
🟡 Not ready to approve
The new mobile sidebar CSS selector likely won’t match the actual “sidebar open” state, so the intended UI fix may not take effect.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Adds Vercel Analytics tracking to the Docusaurus site by introducing a Root theme wrapper, and includes a small mobile navbar/sidebar styling adjustment aimed at improving the sidebar open state.
Changes:
- Added
@vercel/analyticsdependency and corresponding lockfile entries. - Introduced
src/theme/Root.tsxto render the Vercel<Analytics />component site-wide. - Adjusted mobile CSS to disable
backdrop-filterwhen the sidebar is shown.
File summaries
| File | Description |
|---|---|
| src/theme/Root.tsx | Wraps the app root to include Vercel Analytics tracking. |
| src/css/custom.css | Tweaks mobile navbar/sidebar styling related to backdrop filtering. |
| package.json | Adds @vercel/analytics dependency. |
| pnpm-lock.yaml | Locks @vercel/analytics and related dependency metadata. |
Review details
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
- Files reviewed: 3/4 changed files
- Comments generated: 1
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
This pull request introduces Vercel Analytics to the project and includes a minor CSS adjustment for the sidebar. The main changes are the addition and integration of the
@vercel/analyticspackage and a small UI tweak to the navbar sidebar.Analytics Integration:
@vercel/analyticsas a dependency inpackage.jsonand updatedpnpm-lock.yamlto include the new package and its peer dependencies. [1] [2] [3] [4]src/theme/Root.tsxto wrap the app with theAnalyticscomponent, enabling Vercel Analytics tracking across the site.UI Improvements:
.navbar.navbar-sidebar--showinsrc/css/custom.cssto disable thebackdrop-filter, which may improve sidebar appearance or performance.Summary by CodeRabbit