Skip to content

LYT-1004: restore slideout and bar open/close animations - #672

Open
ashyablok-cs wants to merge 1 commit into
developfrom
LYT-1004-slideout-animation
Open

LYT-1004: restore slideout and bar open/close animations#672
ashyablok-cs wants to merge 1 commit into
developfrom
LYT-1004-slideout-animation

Conversation

@ashyablok-cs

Copy link
Copy Markdown

Transitions were dropped from .pf-widget in 5ae45dd to fix a modal CLS regression, which left slideouts and bars snapping into place.

Restore the motion using composited properties only: widgets now sit at their final offsets and are pushed off-screen with a transform, so no layout property is animated and the CLS contribution stays at zero.

  • add .slide-transition() mixin (transform/opacity/visibility)
  • slideouts: translate off-screen per position; the vertically centered left/right variants compose translateX with their translateY(-50%) so they hold their centering for the whole transition
  • bars: translateY off the top/bottom edge instead of moving top/bottom
  • rebuild dist/pathfora.css and dist/pathfora.min.css

Transitions were dropped from .pf-widget in 5ae45dd to fix a modal CLS
regression, which left slideouts and bars snapping into place.

Restore the motion using composited properties only: widgets now sit at
their final offsets and are pushed off-screen with a transform, so no
layout property is animated and the CLS contribution stays at zero.

- add .slide-transition() mixin (transform/opacity/visibility)
- slideouts: translate off-screen per position; the vertically centered
  left/right variants compose translateX with their translateY(-50%) so
  they hold their centering for the whole transition
- bars: translateY off the top/bottom edge instead of moving top/bottom
- rebuild dist/pathfora.css and dist/pathfora.min.css

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ashyablok-cs
ashyablok-cs requested a review from a team August 20, 2026 21:52
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