Skip to content

fix: edge group collapse/expand button stuck after two clicks (#1241)#1250

Merged
mathuo merged 1 commit intomasterfrom
fix/edge-group-collapse-1241
May 8, 2026
Merged

fix: edge group collapse/expand button stuck after two clicks (#1241)#1250
mathuo merged 1 commit intomasterfrom
fix/edge-group-collapse-1241

Conversation

@mathuo
Copy link
Copy Markdown
Owner

@mathuo mathuo commented May 8, 2026

  • demo: read event.isCollapsed in onDidCollapsedChange handler instead of treating the event object as a boolean (kept React state truthy forever).
  • core: route auto-collapse-when-empty through setEdgeGroupCollapsed so onDidCollapsedChange fires for listeners.
  • core: skip the splitview resize on no-op state changes so spaced themes don't accumulate gap rounding drift on repeated expand() calls.

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

- demo: read event.isCollapsed in onDidCollapsedChange handler instead of treating the event object as a boolean (kept React state truthy forever).
- core: route auto-collapse-when-empty through setEdgeGroupCollapsed so onDidCollapsedChange fires for listeners.
- core: skip the splitview resize on no-op state changes so spaced themes don't accumulate gap rounding drift on repeated expand() calls.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mathuo mathuo linked an issue May 8, 2026 that may be closed by this pull request
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

@mathuo mathuo merged commit d36f525 into master May 8, 2026
9 checks passed
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.

Unexpected behavior for the edge group's expand/collapse button

1 participant