Skip to content

Port multi-row tabs setting to main#356

Open
danshapiro wants to merge 3 commits into
mainfrom
port/multirow-tabs-main-20260518
Open

Port multi-row tabs setting to main#356
danshapiro wants to merge 3 commits into
mainfrom
port/multirow-tabs-main-20260518

Conversation

@danshapiro
Copy link
Copy Markdown
Owner

Summary

Source and scope

Verification

  • npm run test:vitest -- test/unit/shared/settings.test.ts test/unit/client/store/browserPreferencesPersistence.test.ts test/unit/client/components/TabBar.multirow.test.tsx --run passed: 3 files, 43 tests.
  • npm run typecheck:client passed.
  • npm run build:client passed, with existing Vite dynamic-import and chunk-size warnings.
  • npm run lint passed with 14 existing warnings and 0 errors.
  • npm run typecheck:server failed before this branch's touched code due to existing origin/main parse errors in server/mcp/freshell-tool.ts lines 73 and 472.
  • npm run build:server failed on the same existing parse errors.
  • npm run test:e2e:chromium -- test/e2e-browser/specs/multirow-tabs.spec.ts was blocked in global setup because it runs npm run build:client && npm run build:server, and build:server fails on the same existing parse errors.

Supersedes

This PR supersedes the #335 multi-row tabs setting payload currently embedded in stale #342.

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.

1 participant