• Changed Files
  • packages/core/src/interactions/Tossable.ts

[refactored] tossable.state$ to use the aggregator anyOf
ClosedPublic

Authored by appsforartists on Sep 11 2017, 10:03 PM.

Details

Summary

I made onStart synchronous in D3325. Since it's now synchronous, we can simplify tossable.state$ to be the union of spring.state$ and draggable.state$.

Diff Detail

Repository
R13 material-motion/material-motion-js
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
Restricted Application completed building Restricted Buildable.Sep 11 2017, 10:03 PM
Restricted Application added a reviewer: O2: Material Motion. · View Herald TranscriptSep 11 2017, 10:03 PM
Restricted Application added a project: Material Motion. · View Herald Transcript
Restricted Application added a reviewer: O3: Material JavaScript platform reviewers. · View Herald Transcript
Restricted Application added a reviewer: Material Motion. · View Herald Transcript
Restricted Application failed to build Restricted Buildable!Sep 11 2017, 10:23 PM
Restricted Application completed building Restricted Buildable.Sep 13 2017, 6:27 PM
appsforartists retitled this revision from [refactored] Tossable.state to use the aggregator anyOf to [refactored] tossable.state$ to use the aggregator anyOf.Sep 19 2017, 6:52 PM
appsforartists edited the summary of this revision. (Show Details)
featherless accepted this revision.Sep 20 2017, 6:04 PM
This revision is now accepted and ready to land.Sep 20 2017, 6:04 PM
This revision was automatically updated to reflect the committed changes.