material-motion / material-motion-android

Reactive motion for Android. Deprecated; please use the Motion library in Material Components for Android instead: https://material.io/develop/android/theming/motion/.
Apache License 2.0
444 stars 26 forks source link

Rename MaterialSpring to Spring #47

Open jverkoey opened 7 years ago

pingpongboss commented 7 years ago

The reason this was named MaterialSpring is because Rebound's library already defines a Spring class. If I rename this to Spring, devs' IDEs will ask them to disambiguate. This is not a big deal, except if devs use both material-motion's Spring and rebound's Spring. In that case, they will be forced to use the full package prefix when referring to one of them (dev's choice which one).

jverkoey commented 7 years ago

Discussed in sprint planning: it's not super likely that people will be using Rebound and Material Motion's spring in the same file, especially if we drop the Rebound dependency.