feat(layout): trigger open animation when moving column or window between outputs #2379
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.
When moving a window or a whole column to another output, the open animation now plays on the destination. This makes it easier to see what actually moved, especially if there are multiple similar windows (e.g. terminals) on the target output. It just reuses
start_open_animation_for_window(..)
so it respects whatever animation config the user already set.Details
Why
Before, windows/columns just appeared on the other output, which can be hard to spot in busy layouts. The animation makes the move obvious without adding any new config or custom effect.
Before:
before.mp4
After:
after.mp4