Skip to content

fix: windows installation tab on the docs#28125

Open
Zamiell wants to merge 3 commits intooven-sh:mainfrom
Zamiell:fix-website-windows
Open

fix: windows installation tab on the docs#28125
Zamiell wants to merge 3 commits intooven-sh:mainfrom
Zamiell:fix-website-windows

Conversation

@Zamiell
Copy link

@Zamiell Zamiell commented Mar 14, 2026

What does this PR do?

Fixes #25042.

How did you verify your code works?

I ran mintlify dev and watched the Windows tab get auto selected.

@Zamiell Zamiell changed the title fix: windows fix: windows installation tab on the docs Mar 14, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 98a43127-4ad6-453a-acad-2b6ab12b2219

📥 Commits

Reviewing files that changed from the base of the PR and between a9d9b14 and d5369d8.

📒 Files selected for processing (1)
  • docs/snippets/windows-tab-selector.jsx

Walkthrough

Adds a Windows-specific tab selector UI component and a small client script. The component is imported into docs/installation.mdx; docs/os-tab-selector.js detects Windows and auto-selects the Windows installation tab, including SPA navigation handling.

Changes

Cohort / File(s) Summary
Installation Documentation
docs/installation.mdx
Imported and rendered the WindowsTabSelector component to surface Windows-specific install instructions on the installation page.
Windows Tab UI Snippet
docs/snippets/windows-tab-selector.jsx
New React component WindowsTabSelector that renders a Windows instruction block and PowerShell command snippet; exported as default.
Client-side OS Detection Script
docs/os-tab-selector.js
New self-executing script that detects Windows via navigator, clicks the "Windows" role="tab" on load, and re-applies selection on SPA navigation using a MutationObserver.
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: adding Windows tab auto-selection functionality to the documentation.
Description check ✅ Passed The PR description follows the repository template with both required sections completed: it references the issue and explains verification steps.
Linked Issues check ✅ Passed The code changes directly address issue #25662 by implementing Windows OS detection and auto-selecting the Windows tab on the installation docs page.
Out of Scope Changes check ✅ Passed All changes are scoped to implementing Windows tab auto-selection: adding a React component, a detection script, and integrating them into the installation page.

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

📝 Coding Plan
  • Generate coding plan for human review comments

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
Contributor

@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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/installation.mdx`:
- Around line 6-8: The import references a missing React component
WindowsTabSelector; create the component file exports used by the docs or update
the import path. Add a new file docs/snippets/windows-tab-selector.jsx that
exports a default WindowsTabSelector React component (function or arrow) which
renders the expected tab UI (props if used by the docs) and export default
WindowsTabSelector, or if the component already exists elsewhere, change the
import in docs/installation.mdx to point to its actual location and
named/default export (WindowsTabSelector) to match the implementation.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: a2653ae4-f81b-4b9f-a09f-58e575f811a0

📥 Commits

Reviewing files that changed from the base of the PR and between 10bdb48 and a9d9b14.

📒 Files selected for processing (2)
  • docs/installation.mdx
  • docs/os-tab-selector.js

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.

bun completions does not work on Git Bash

1 participant