Skip to content

[6.x] Fix sidebar not showing after quick resize#13969

Merged
jasonvarga merged 4 commits into6.xfrom
publish-tabs-resizing
Feb 17, 2026
Merged

[6.x] Fix sidebar not showing after quick resize#13969
jasonvarga merged 4 commits into6.xfrom
publish-tabs-resizing

Conversation

@duncanmcclean
Copy link
Member

This pull request fixes an issue where resizing a publish form quickly could lead to the sidebar not being "expanded" correctly.

The ElementContainer.vue used in Publish/Tabs.vue uses a throttled ResizeObserver, but the throttle would only fire once immediately, then ignore all calls for 200ms. If resizing finished within that window, the final width was never reported.

Fixes #13915

@jasonvarga
Copy link
Member

AI pointed out a harmless issue, but I've resolved it anyway.

No cleanup/cancellation mechanism. The ElementContainer creates a ResizeObserver in mounted but never disconnects it or cancels pending timeouts on unmount. This is a pre-existing
issue (not introduced by this PR), but the trailing setTimeout does make it slightly more relevant — a timeout could fire after the component is destroyed. That said, it would just
set this.width on a destroyed component, which is harmless in practice. Not a blocker.

I also added tests.

@jasonvarga jasonvarga merged commit 2e80a26 into 6.x Feb 17, 2026
11 checks passed
@jasonvarga jasonvarga deleted the publish-tabs-resizing branch February 17, 2026 14:45
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.

Break points aren't always triggered

2 participants

Comments