[refactored] lowerBound and upperBound into single clampTo operator
ClosedPublic

Authored by appsforartists on Oct 6 2017, 11:51 PM.

Details

Summary

Continuation of https://github.com/material-motion/material-motion-js/issues/193

Since we are using named args now, it doesn't make sense to have separate upperBound and lowerBound operators. The combined operator uses _mathOperator under the hood, so it works on points as well as numbers.

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.Oct 6 2017, 11:51 PM
Restricted Application added a reviewer: O2: Material Motion. · View Herald TranscriptOct 6 2017, 11:51 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.Oct 7 2017, 12:01 AM
vietanh accepted this revision.Oct 10 2017, 7:56 PM
vietanh added a subscriber: vietanh.

cool, clamp is more easier to use than lower and upper.

This revision is now accepted and ready to land.Oct 10 2017, 7:56 PM
This revision was automatically updated to reflect the committed changes.