Skip to content

Conversation

@gggritso
Copy link
Member

@gggritso gggritso commented Nov 7, 2025

Adds a new motion type called "spring".

Unlike the current motion tokens, this one is physics-based, so its definition is not as a Bezier curve, but as a Framer Motion animation. It's mostly meant to be used with motion.div element, however I'm also generating a CSS linear() function, so it can be used in CSS.

Immediately uses the token in the place where I extracted it from, which is the SlideOverPanel component.

@gggritso gggritso requested a review from natemoo-re November 7, 2025 20:31
@gggritso gggritso requested a review from a team as a code owner November 7, 2025 20:31
@linear
Copy link

linear bot commented Nov 7, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 7, 2025
Copy link
Member

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for adding this!

@gggritso gggritso merged commit 71f2546 into master Nov 12, 2025
66 of 70 checks passed
@gggritso gggritso deleted the georgegritsouk/de-537-add-a-design-token-for-spring-animation branch November 12, 2025 16:44
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
Adds a new motion type called "spring".

Unlike the current motion tokens, this one is physics-based, so its
definition is not as a Bezier curve, but as a Framer Motion animation.
It's mostly meant to be used with `motion.div` element, _however_ I'm
also generating a CSS `linear()` function, so it can be used in CSS.

Immediately uses the token in the place where I extracted it from, which
is the `SlideOverPanel` component.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants