Skip to content

docs(AnimationsProvider): Add animations-provider.md to developer resources#4796

Closed
jenny-s51 wants to merge 1 commit into
patternfly:mainfrom
jenny-s51:animationsProviderDocs
Closed

docs(AnimationsProvider): Add animations-provider.md to developer resources#4796
jenny-s51 wants to merge 1 commit into
patternfly:mainfrom
jenny-s51:animationsProviderDocs

Conversation

@jenny-s51
Copy link
Copy Markdown
Contributor

@jenny-s51 jenny-s51 commented Sep 19, 2025

Closes #4783

Follow-up to https://github.com/patternfly/patternfly-react/pull/11990/files - DO NOT MERGE until this PR has been merged into patternfly-react.

@patternfly-build
Copy link
Copy Markdown
Collaborator

Preview: https://pf-org--pr-4796-site.surge.sh

@edonehoo
Copy link
Copy Markdown
Collaborator

edonehoo commented Sep 24, 2025

@jenny-s51 wdyt about trying to fit this into the existing motion dev guide? I also see the inclination to put this in the dev resources, but I'm hoping to try and phase out the separate dev resources section in the future, in favor of keeping related docs together

I opened #4801 to try and test that idea out, editing/condensing the info you have here (plus some light edits to the existing stuff to help condense more). it does make that guide longer, but I'm hoping it's not too disruptive? Curious for your thoughts! We can certainly find a middle ground if that idea is a bust

Copy link
Copy Markdown
Contributor Author

@jenny-s51 jenny-s51 left a comment

Choose a reason for hiding this comment

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

Perfect, thank you @edonehoo, this is a great idea - I like how you incorporated this content into the motion dev guide in #4801 and condensed some things for improved readability and length - keeping it there sounds like a good game plan.

@nicolethoen
Copy link
Copy Markdown
Collaborator

Closing this since #4801 has merged. Please feel free to reopen if that's a mistake.

@nicolethoen nicolethoen closed this Oct 9, 2025
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.

Add animations-provider.md to developer resources

4 participants