Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: Removed CurveUtils, added Interpolations, and redirected links #12040

Merged
merged 1 commit into from
Aug 24, 2017

Conversation

moraxy
Copy link
Contributor

@moraxy moraxy commented Aug 24, 2017

src/extras/CurveUtils.js was removed in fe98988, but not from docs/list.js. I removed both the file and the extra entry. Since CurveUtils.interpolate was moved to Interpolations.js, which wasn't yet in docs/list.js either, I also added an entry and a rough page for it.

Also, as far as I can tell CurveUtils.interpolate was last used only once and internally(in src/extras/curves/SplineCurve.js) - does that mean no entries in docs/api/deprecated/DeprecatedList.html and src/Three.Legacy.js are needed because it wasn't really public API?

@mrdoob mrdoob merged commit eb49209 into mrdoob:dev Aug 24, 2017
@mrdoob
Copy link
Owner

mrdoob commented Aug 24, 2017

Thanks!

@moraxy moraxy deleted the docs_remove_curveutils branch October 16, 2017 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants