You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
agreed, it can be improved a lot and there is stuff like the animationRef that can be removed. not sure about classes, if you still want to implement IE then best we can do are prototypes or modules.
If IE isnt an issue then the only thing i have to comment on would be that i think the specific animation functions wont fit well as a class but maybe we can abstract it to a set of instructions like the svgblueprint
Activity
hampterhub commentedon Oct 9, 2018
agreed, it can be improved a lot and there is stuff like the
animationRefthat can be removed. not sure about classes, if you still want to implement IE then best we can do are prototypes or modules.If IE isnt an issue then the only thing i have to comment on would be that i think the specific animation functions wont fit well as a class but maybe we can abstract it to a set of instructions like the svgblueprint
thumbnail commentedon Oct 9, 2018
Since we add browserify / babelify in #14, it gets transpiled to es2015 anyway, so we can use (most of) the features afaik.