Cleanup deprecated code and Update documentation for BufferGeometry and BufferAttribute

[rafaelsc]

Why

BufferGeometry and BufferAttribute type need some cleanup to remove code that doesn't exist anymore on ThreeJS. And even marked was deprecated would cause runtime errors. And some documentation spelling and naming needed improvements.

What

• Remove the old code
• Improve documentation spelling and naming
• Update some return Types with the correct type according to the Documentation and Source code.

Checklist

• Checked the target branch (current goes master, next goes dev)
• Added myself to contributors table
• Ready to be merged

[0b5vr]

I think it's probably documentation in the era when they used to live together with Geometry.
If you are super generous and change the doc to the latest one I will appreciate it.

https://threejs.org/docs/#api/en/core/BufferGeometry

[rafaelsc]

Hello @0b5vr, I had already started working on improving TSDocs but I'm coding a way to do mostly automatic allowing update everything quicker. And this will take some time to complete.

So for now I just update the TSDocs for BufferGeometry in a very quick and dirty way. And in the near future, I will update with better TSDocs.

And a fix for all the returning types will be done in another PR.

[rafaelsc]

All Documentation is now Updated with the level of quality that I was aiming.

[rafaelsc]

Some return Types were updated with the correct type according to the Documentation and Source code, with more changes that I'm planning for other PR.

[0b5vr]

Sorry for the late response, I will check it tomorrow!

[0b5vr]

Looks... fantastic

Thank you!

[0b5vr]

@rafaelsc Seems you have to delete the line examples/jsm/utils/BufferGeometryUtils.d.ts from types/three/OTHER_FILES.txt.

You can check locally by doing yarn test-all!

[rafaelsc]

Good find. That error started after my last commit when I added the reference for BufferGeometryUtils for the updated TSDoc.