New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pull all of maze.js's animation logic out into dedicated animationsController #20467
Conversation
eead1e1
to
ce2f94a
Compare
59d917b
to
f430e20
Compare
ce2f94a
to
8b0e0d9
Compare
ec3d2cc
to
50dc78f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
|
||
module.exports = class AnimationsController { | ||
constructor(maze, svg) { | ||
this.maze = maze; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Long term is the plan to only pass in the options we need, vs. the entire Maze reference?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep!
…ce it doesn't actually have anything to do with animations
77b70cc
to
4f5be83
Compare
No description provided.