Skip to content

feat(docs): add tab overflow mode switch to demo tab context menu#1483

Merged
mathuo merged 1 commit into
v8-branchfrom
claude/demo-tab-mode-context-menu
Jul 17, 2026
Merged

feat(docs): add tab overflow mode switch to demo tab context menu#1483
mathuo merged 1 commit into
v8-branchfrom
claude/demo-tab-mode-context-menu

Conversation

@mathuo

@mathuo mathuo commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Add a radio-style "Overflow dropdown" / "Wrap onto rows" pair to the demo's tab context menu that flips overflow.mode live via updateOptions, with a tick marking the active mode. Wrap mode is provided by the already-imported MultiRowTabsModule.

Also drop the demo's own 'pin' menu item: pinnedTabs.enabled makes the context menu module inject Pin/Unpin at the top already, so the demo was rendering a duplicate "Pin tab".

Description

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactor / cleanup
  • Build / CI / tooling

Affected packages

  • dockview-core
  • dockview (vanilla JS)
  • dockview-react
  • dockview-vue
  • dockview-angular
  • docs

How to test

Checklist

  • yarn lint:fix passes
  • yarn format passes
  • npm run gen has been run and generated files are up to date
  • yarn test passes
  • I have added or updated tests where applicable
  • Breaking changes are documented

Add a radio-style "Overflow dropdown" / "Wrap onto rows" pair to the
demo's tab context menu that flips `overflow.mode` live via
updateOptions, with a tick marking the active mode. Wrap mode is
provided by the already-imported MultiRowTabsModule.

Also drop the demo's own 'pin' menu item: pinnedTabs.enabled makes the
context menu module inject Pin/Unpin at the top already, so the demo
was rendering a duplicate "Pin tab".

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mathuo
mathuo marked this pull request as ready for review July 17, 2026 06:37
@mathuo
mathuo merged commit 8f3d011 into v8-branch Jul 17, 2026
4 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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