Skip to content

Comments

fix: hover highlight#490

Merged
moshloop merged 1 commit intomainfrom
fix-hover-highlight
Feb 20, 2026
Merged

fix: hover highlight#490
moshloop merged 1 commit intomainfrom
fix-hover-highlight

Conversation

@yashmehrotra
Copy link
Member

@yashmehrotra yashmehrotra commented Feb 20, 2026

Summary by CodeRabbit

  • Chores
    • Updated documentation navigation configuration for the Overview, Integrations, and User Guide sections to enhance navigation consistency and structure.

@vercel
Copy link

vercel bot commented Feb 20, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Feb 20, 2026 3:14am

Request Review

@netlify
Copy link

netlify bot commented Feb 20, 2026

Deploy Preview for canarychecker canceled.

Name Link
🔨 Latest commit 630e014
🔍 Latest deploy log https://app.netlify.com/projects/canarychecker/deploys/6997d0d4344134000841c06a

@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

Walkthrough

Updated Docusaurus navbar configuration to use explicit docSidebar references instead of static route targets. Three navbar items (Overview, Integrations, User Guide) were modified to specify sidebarId and type: 'docSidebar' for more explicit navigation resolution.

Changes

Cohort / File(s) Summary
Docusaurus Navbar Configuration
mission-control/docusaurus.config.ts
Replaced static route targets (to, activeBasePath) with docSidebar configuration for three navbar items: Overview (sidebarId: 'overview'), Integrations (sidebarId: 'integrations'), and User Guide (sidebarId: 'guide').
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'fix: hover highlight' is vague and does not clearly describe the actual changes, which involve replacing static route targets with docSidebar configuration in the navbar. Clarify the title to reflect the main change, such as 'refactor: use docSidebar configuration for navbar navigation' or 'fix: resolve navbar active state using docSidebar'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-hover-highlight

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
mission-control/docusaurus.config.ts (1)

375-380: Consider converting the Reference item to type: 'docSidebar' for consistency.

The Reference navbar item still uses the old to/activeBasePath pattern that caused the highlight bug in the other three items. If a reference sidebar key exists in sidebars.js, converting it the same way would be consistent and avoid the same hover-highlight issue there.

♻️ Proposed refactor (if a `reference` sidebar exists)
-            {
-              to: '/docs/reference',
-              activeBasePath: '/docs/reference',
-              label: 'Reference',
-              position: 'left'
-            },
+            {
+              type: 'docSidebar',
+              sidebarId: 'reference',
+              label: 'Reference',
+              position: 'left'
+            },
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@mission-control/docusaurus.config.ts` around lines 375 - 380, The navbar item
for label 'Reference' uses to/activeBasePath and should be converted to use
type: 'docSidebar' for consistency and to avoid the highlight bug; update the
nav item object (the one with label 'Reference') to replace the
to/activeBasePath entries with type: 'docSidebar' and specify the sidebarId/key
'reference' (matching the 'reference' key in sidebars.js) so Docusaurus uses the
docSidebar behavior.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@mission-control/docusaurus.config.ts`:
- Around line 375-380: The navbar item for label 'Reference' uses
to/activeBasePath and should be converted to use type: 'docSidebar' for
consistency and to avoid the highlight bug; update the nav item object (the one
with label 'Reference') to replace the to/activeBasePath entries with type:
'docSidebar' and specify the sidebarId/key 'reference' (matching the 'reference'
key in sidebars.js) so Docusaurus uses the docSidebar behavior.

@moshloop moshloop merged commit f076088 into main Feb 20, 2026
15 checks passed
@moshloop moshloop deleted the fix-hover-highlight branch February 20, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants