motion icon indicating copy to clipboard operation
motion copied to clipboard

fixing a few typos, clarity issues and stale links in the TS definitions

Open neontomo opened this issue 9 months ago • 1 comments

typos + stale links

this pull request addresses a few typos, clarity issues and stale links found in the TS definition files. the following changes have been made.

1. packages/framer-motion/src/animation/animators/AcceleratedAnimation.ts

  • Before: AnimationTimline
  • After: AnimationTimeline
  • Description: Corrected a typo in the type name.

2. packages/framer-motion/src/gestures/drag/types.ts

  • Before: "This will define a distance the named edge of the draggable component."
  • After: "This will define the distance from the specified edge of the draggable component to the boundary of its constraints."
  • Description: Revised for clarity to better explain the intended meaning.

3. packages/framer-motion/src/gestures/drag/types.ts

  • Before: See {@link https://framer.com/api/animation/#inertia | Inertia}
  • After: See {@link https://motion.dev/api/animation/docs/react-motion-component#dragtransition | Inertia}
  • Description: Updated the link to point to the correct documentation for dragTransition.

4. packages/framer-motion/src/animation/types.ts

  • Before: @param transition - Optional transtion to apply to a variant
  • After: @param transition - Optional transition to apply to a variant
  • Description: Fixed a typo in the parameter description.

5. packages/framer-motion/src/utils/interpolate.ts

  • Before: TODO Revist this approach once we've moved to data models for values
  • After: TODO: Revisit this approach once we've moved to data models for values
  • Description: Corrected the spelling of "Revist" to "Revisit" and added a colon for consistency.

thanks for making an awesome library! 🩵

neontomo avatar Mar 01 '25 20:03 neontomo

had some issues building the project with make bootstrap so i had to upgrade to latest yarn, let me know if this causes any problems for you.

CleanShot 2025-03-01 at 22 24 12@2x

neontomo avatar Mar 01 '25 21:03 neontomo

Hey sorry for just getting to this. If you can fix these conflicts I can get it merged in. AcceleratedAnimation -> NativeAnimationExtended and interpolate both now in motion-dom.

mattgperry avatar May 09 '25 07:05 mattgperry

Closing this to keep things clean but feel free to re-open with the fixed conflicts 🙏

mattgperry avatar May 15 '25 11:05 mattgperry