Repository navigation
refactor(devtools): drive the sidebar from a configurable visible-tabs list - #8826
Merged
wmertens merged 1 commit intoJul 14, 2026
Conversation
…s list Add an ordered visibleTabIds list to devtools state (defaulting to every tab in registry order) and render the sidebar from it via a getVisibleTabs helper that resolves ids and applies the extension vite-only filter. Behavior-preserving foundation for the upcoming customizable sidebar tabs; the default keeps the sidebar identical to today.
🦋 Changeset detectedLatest commit: df94da4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@qwik.dev/core
@qwik.dev/router
eslint-plugin-qwik
create-qwik
@qwik.dev/optimizer
@qwik.dev/devtools
commit: |
Contributor
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Contributor
This was referenced Jul 13, 2026
wmertens
approved these changes
Jul 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
Foundation (PR 1 of ~4) for the Customizable Sidebar Tabs feature. This PR is intentionally behavior-preserving: it introduces the data model the later PRs build on, with no visible change yet.
Change
visibleTabIds: DevtoolsTabId[]toDevtoolsState, defaulting to every tab in registry order.sidebar-tabs.tsmodel:getDefaultVisibleTabIds()andgetVisibleTabs(visibleTabIds, isExtension), which resolves the ordered ids to their configs (via a privatetabsByIdmap) and applies the existing extension vite-only filter.DevtoolsSidebarrendersgetVisibleTabs(state.visibleTabIds, state.isExtension)instead of iterating the registry directly.The default (all tabs, registry order) makes the rendered sidebar identical to today, including the extension's vite-only filtering.
Follow-up PRs (this is an epic, split for reviewability)
This PR ships the model only. The rest of the feature lands in separate PRs on top of it:
The customization UI in PRs 2 to 4 is gated to the Vite overlay (
!state.isExtension); the extension keeps the default sidebar.Verification
tscon the devtools UI passes (0 errors); the model is fully typed with no casts.sidebar-tabs.unit.tscovers the default list, order preservation, and vite-only filtering in overlay vs extension.References
QwikDev project board, feature "Support Customizable Sidebar Tabs".