Skip to content

docs: Added package manager tab group synchronization across the entire installation page#2421

Open
BlazerYoo wants to merge 7 commits into
wxt-dev:mainfrom
BlazerYoo:main
Open

docs: Added package manager tab group synchronization across the entire installation page#2421
BlazerYoo wants to merge 7 commits into
wxt-dev:mainfrom
BlazerYoo:main

Conversation

@BlazerYoo

Copy link
Copy Markdown

Overview

Updates the docs installation page so choosing a package manager in one code group automatically updates the other code groups on the same page.

Manual Testing

  1. Run the docs locally with bun run docs:dev
  2. Open the installation page at /guide/installation
  3. Click Bun in one code group under “From Scratch”
  4. Verify the other code groups on the page switch to Bun as well
  5. Repeat with PNPM or NPM to confirm the selection syncs across all groups
  6. Refresh the page and repeat to make sure it still works after reload

Related Issue

This PR does not close a tracked issue.

@netlify

netlify Bot commented Jun 17, 2026

Copy link
Copy Markdown

Deploy Preview for creative-fairy-df92c4 failed.

Name Link
🔨 Latest commit 86f203c
🔍 Latest deploy log https://app.netlify.com/projects/creative-fairy-df92c4/deploys/6a37555b27abf30008c483fc

@pkg-pr-new

pkg-pr-new Bot commented Jun 18, 2026

Copy link
Copy Markdown

Open in StackBlitz

@wxt-dev/analytics

npm i https://pkg.pr.new/@wxt-dev/analytics@2421

@wxt-dev/auto-icons

npm i https://pkg.pr.new/@wxt-dev/auto-icons@2421

@wxt-dev/browser

npm i https://pkg.pr.new/@wxt-dev/browser@2421

@wxt-dev/i18n

npm i https://pkg.pr.new/@wxt-dev/i18n@2421

@wxt-dev/is-background

npm i https://pkg.pr.new/@wxt-dev/is-background@2421

@wxt-dev/module-react

npm i https://pkg.pr.new/@wxt-dev/module-react@2421

@wxt-dev/module-solid

npm i https://pkg.pr.new/@wxt-dev/module-solid@2421

@wxt-dev/module-svelte

npm i https://pkg.pr.new/@wxt-dev/module-svelte@2421

@wxt-dev/module-vue

npm i https://pkg.pr.new/@wxt-dev/module-vue@2421

@wxt-dev/runner

npm i https://pkg.pr.new/@wxt-dev/runner@2421

@wxt-dev/storage

npm i https://pkg.pr.new/@wxt-dev/storage@2421

@wxt-dev/unocss

npm i https://pkg.pr.new/@wxt-dev/unocss@2421

@wxt-dev/webextension-polyfill

npm i https://pkg.pr.new/@wxt-dev/webextension-polyfill@2421

wxt

npm i https://pkg.pr.new/wxt@2421

commit: 86f203c

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.46%. Comparing base (c8e93b1) to head (86f203c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2421      +/-   ##
==========================================
+ Coverage   78.33%   78.46%   +0.12%     
==========================================
  Files         133      133              
  Lines        3882     3882              
  Branches      875      875              
==========================================
+ Hits         3041     3046       +5     
+ Misses        752      749       -3     
+ Partials       89       87       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@PatrykKuniczak PatrykKuniczak left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Move everything to /utils and there simply call that function in enhanceApp section.
Also if(typeof window ... move inside that helper, to keep index.ts clean 😄

And run prettier.

Anyway that's amazing feature(not a fix, like you've suggested i need to change the PR title), i've seen that in many docs of multiple tools,

Comment thread docs/.vitepress/theme/index.ts Outdated
Comment thread docs/.vitepress/theme/index.ts Outdated
@PatrykKuniczak PatrykKuniczak changed the title fix(docs): sync code group tabs across installation page feat(docs): Add syncing groups of package managers tabs across entire installation page Jun 18, 2026
@PatrykKuniczak PatrykKuniczak changed the title feat(docs): Add syncing groups of package managers tabs across entire installation page feat(docs): Added package manager tab group synchronization across the entire installation page Jun 18, 2026
@BlazerYoo

Copy link
Copy Markdown
Author

Thanks for the review! Just pushed a commit covering all of it:

  • Moved all sync logic into docs/.vitepress/utils/code-groups.ts, exposing a single setupCodeGroupSync() that enhanceApp calls
  • Moved if(typeof window ... inside the helper, index.ts clean
  • Ran prettier; --check passes

Comment thread docs/.vitepress/utils/code-groups-sync.ts

@PatrykKuniczak PatrykKuniczak left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 😃

@PatrykKuniczak PatrykKuniczak enabled auto-merge (squash) June 21, 2026 06:14
@PatrykKuniczak PatrykKuniczak changed the title feat(docs): Added package manager tab group synchronization across the entire installation page docs: Added package manager tab group synchronization across the entire installation page Jun 21, 2026
@PatrykKuniczak

Copy link
Copy Markdown
Collaborator

@BlazerYoo That's not your problem with CI.
I'll take a look on that, maybe tomorrow

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