Skip to content

feat: animations skill#72

Merged
B0berman merged 10 commits into
mainfrom
feat/animations-skill
May 25, 2026
Merged

feat: animations skill#72
B0berman merged 10 commits into
mainfrom
feat/animations-skill

Conversation

@B0berman
Copy link
Copy Markdown
Contributor

@B0berman B0berman commented Apr 8, 2026

Description

  • Add a new vgv-animations skill covering Flutter animation best practices using the built-in animation framework and Material 3 motion guidelines
  • Adds four reference files: animated-switcher.md, looping-animations.md, page-transitions.md, and staggered-animations.md

The skill guides developers through:

  • Choosing the right animation type — decision tree from implicit (AnimatedFoo) to explicit (AnimationController) to hero/route transitions
  • Material 3 motion tokens — enforces use of Durations and Easing constants instead of hardcoded values
  • Implicit animations — AnimatedContainer, AnimatedOpacity, TweenAnimationBuilder, etc.
  • Explicit animations — AnimationController lifecycle, TickerProviderStateMixin, chained tweens
  • Page transitions — custom PageRouteBuilder and go_router integration
  • Staggered animations — Interval-based sequencing patterns
  • Anti-patterns — common mistakes like unnecessary controllers, missing disposal, and rebuilding entire trees

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Code refactor (refactor)
  • Documentation (docs)
  • CI change (ci)
  • Chore (chore)

@B0berman B0berman requested a review from a team as a code owner April 8, 2026 12:27
Comment thread skills/animations/references/staggered-animations.md Outdated
Comment thread .claude-plugin/plugin.json
@ryzizub ryzizub self-requested a review April 15, 2026 15:11
Comment thread skills/animations/references/animated-switcher.md Outdated
Comment thread skills/animations/SKILL.md Outdated
Comment thread skills/testing/references/animation-testing.md
@ryzizub
Copy link
Copy Markdown
Contributor

ryzizub commented Apr 16, 2026

I think we need to revisit on this skill what needs to be part of it and what is already part of the model. Here we are repeating a lot what is already part of documentation and thats not ideal cause it might get outdated

@B0berman B0berman requested a review from ryzizub April 16, 2026 15:13
Comment thread skills/testing/references/animation-testing.md Outdated
@B0berman B0berman requested a review from ryzizub May 7, 2026 11:21
Copy link
Copy Markdown
Contributor

@ryzizub ryzizub left a comment

Choose a reason for hiding this comment

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

LGTM

@B0berman B0berman merged commit 70fdbc2 into main May 25, 2026
6 checks passed
@B0berman B0berman deleted the feat/animations-skill branch May 25, 2026 10:26
@vgvbot vgvbot mentioned this pull request May 20, 2026
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.

2 participants