Add Motion.@onRest

[appsforartists]

I don't intend for this to land until after I've used it and made sure we're solving the right problem, but I'm opening this now so @chenglou can play with it (and maybe add Flow if there are places I've missed).

See also #235.

[chenglou]

Thanks a lot!

[appsforartists]

My pleasure!

[threepointone]

super!

[kumarharsh]

wow! awesome!

[leopoldjoy]

@appsforartists @chenglou Can this be used for <Motion />, <StaggeredMotion />, and <TransitionMotion />?

[chenglou]

@leopoldjoy currently only implemented for Motion, which is why I waited before the imminent public release, because I wanted it on all 3 components. But after discussing with people I've noticed that they only need it for Motion?

[leopoldjoy]

Yes, I agree that it seems like it's only really relevant to Motion. I was going to open a pull request for the README.md documentation as I found it extremely useful, would this be helpful/relevant?

[chenglou]

I'll publish it officially this week or next so don't worry about it.

[leopoldjoy]

@chenglou Ok, sounds good! Thanks for your work!

[sunflowerdeath]

onRest for TransitionMotion could be also very useful.

[haraldox]

@chenglou @appsforartists Great work guys! would be awesome to have that in master at least for <Motion> :)

[AlastairTaft]

Would be really useful to have this for <StaggeredMotion />. I'm trying to animate page navigation. Each page is made up of a handful of panels, maybe 3 or 4. When the user navigates between pages I want each panel to staggered-ly spring in. I need to know when the animation is done so that I can remove the old page component.

[ollwenjones]

Echo @AlastairTaft except for TransitionMotion

[appsforartists]

I don't think @chenglou or I use this library anymore, so if you do and you want @onRest for the other components, you should open a PR. 😃

[ollwenjones]

Is there a new flavor of the month for this kind of thing we should look at instead? 😜

[appsforartists]

I still like React Motion.

I've got an experimental motion library that I use for my work, but it's not documented or published, which unfortunately makes it hard for other folks to use.

[ollwenjones]

Heh. How it goes for most of us, seems like. My use-case is a side-panel that squishes a map. Ideally, the map can be told to recenter/repaint after the squishing animation completes, but honestly I'm not sure anyone will notice it isn't happening anytime soon. If it becomes an issue that makes it to the top of my backlog, then you can look for a PR from me. ;)

[kumarharsh]

See React-Spring or Popmotion (both are in active development). I haven't tried either yet, but they are on my todo list. 😄

[nkbt]

We still use react motion in many places. Though the only component that we work with is the main Motion, so I have no context/interest in modifying any other components.
((tbh I would be quite happy to have only that one available and not add up to the bundle))

[ollwenjones]

looks like popmotion/pose is a bit higher level, which is nice.