Skip to content

ref(onboarding): Use LayoutGroup for scmConnect footer position animation#111582

Closed
jaydgoss wants to merge 0 commit intojaygoss/vdy-24-scm-platform-features-step-ui-polish-and-analytics-passfrom
jaygoss/vdy-24-scm-connect-layout-animation
Closed

ref(onboarding): Use LayoutGroup for scmConnect footer position animation#111582
jaydgoss wants to merge 0 commit intojaygoss/vdy-24-scm-platform-features-step-ui-polish-and-analytics-passfrom
jaygoss/vdy-24-scm-connect-layout-animation

Conversation

@jaydgoss
Copy link
Copy Markdown
Member

Summary

  • Use LayoutGroup + layout="position" to animate the scmConnect footer position when content above changes height (e.g. benefits card appearing after repo selection)
  • Restructure content sections as direct motion siblings of the footer within the LayoutGroup so framer-motion can detect layout changes
  • Remove ScmStepContent wrapper in favor of inline MotionStack with width/maxWidth props
  • Use MotionStack wrapper around ScmStepFooter instead of motion.create(ScmStepFooter), since layout="position" needs to be on the actual DOM element

Stacked on VDY-24 platform features polish PR.

Test plan

  • Connect an SCM integration, select a repo, verify footer slides smoothly when benefits card appears
  • Verify no layout jump when switching between connected/not-connected states

@linear-code
Copy link
Copy Markdown

linear-code bot commented Mar 25, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 25, 2026
@jaydgoss
Copy link
Copy Markdown
Member Author

jaydgoss commented Mar 26, 2026

TODO: The loading and error early returns in scmConnect.tsx (lines 63-78) render outside the step layout (no header, no LayoutGroup). They should be moved into the main return inside the LayoutGroup section (lines 89-121) so the step header stays visible and the layout is consistent across all states.

@jaydgoss jaydgoss force-pushed the jaygoss/vdy-24-scm-platform-features-step-ui-polish-and-analytics-pass branch from a1f0dcf to 5cb8082 Compare March 26, 2026 13:22
@jaydgoss
Copy link
Copy Markdown
Member Author

Commit cherry-picked into #111478. This PR is no longer needed.

@jaydgoss jaydgoss closed this Mar 26, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Apr 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant