[refactored] ReactiveProperty to be an IndefiniteSubject subclass
ClosedPublic

Authored by appsforartists on May 15 2017, 9:12 PM.

Details

Summary

Benefits:

  • It passes isObservable(), and can thus be used as an argument to _reactiveMap and friends
  • It exposes next(value) publicly, which makes it possible to do otherStream.subscribe(property)

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.
appsforartists created this revision.May 15 2017, 9:12 PM
Restricted Application added a reviewer: O2: Material Motion. · View Herald TranscriptMay 15 2017, 9:12 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 completed building Restricted Buildable.May 15 2017, 9:15 PM
featherless accepted this revision.May 16 2017, 12:01 PM
This revision is now accepted and ready to land.May 16 2017, 12:01 PM
This revision was automatically updated to reflect the committed changes.