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

Deprecate legacy BufferGeometry nomenclature for built-in geometries #24352

Merged
merged 3 commits into from
Aug 12, 2022

Conversation

WestLangley
Copy link
Collaborator

As proposed in #24349 (comment).

Does this look like the right approach? If so, I'll add the remaining geometries.

@LeviPesin
Copy link
Contributor

I think the right approach would be to do this in BoxGeometry.js instead of Three.Legacy.js.

@Mugen87
Copy link
Collaborator

Mugen87 commented Jul 16, 2022

We only put deprecated methods or properties into non-deprecated classes. Legacy classes should go into Three.Legacy.js.

Does this look like the right approach?

Looking good.

@WestLangley WestLangley force-pushed the dev-buffer_geometries branch from 285ade7 to 9abfc1d Compare July 16, 2022 21:07
@WestLangley WestLangley marked this pull request as ready for review July 16, 2022 21:36
@WestLangley WestLangley added this to the r143 milestone Jul 16, 2022
@WestLangley
Copy link
Collaborator Author

Let's merge before too many conflicts ensue. New build required.

@Mugen87
Copy link
Collaborator

Mugen87 commented Jul 18, 2022

TBH, I want that @mrdoob merges this PR (and #24346). I'm not sure about his preferences regarding the geometry generator aliases.

@WestLangley
Copy link
Collaborator Author

TBH, I want that @mrdoob merges this PR...

@mrdoob What do you think? 😇

@mrdoob mrdoob modified the milestones: r143, r144 Jul 28, 2022
@mrdoob
Copy link
Owner

mrdoob commented Aug 11, 2022

What if we just removed these aliases? 🤔

@WestLangley
Copy link
Collaborator Author

What if we just removed these aliases?

@Mugen87 What do you prefer?

@Mugen87
Copy link
Collaborator

Mugen87 commented Aug 11, 2022

I've thought the policy was to first deprecated things and then make the removal after 10 releases. I personally would prefer to keep doing this especially in this case since I frequently see the usage of BoxBufferGeometry or PlaneBufferGeometry in user code.

@mrdoob
Copy link
Owner

mrdoob commented Aug 12, 2022

@Mugen87 Sounds good!

@mrdoob mrdoob merged commit 5acd8ce into mrdoob:dev Aug 12, 2022
@mrdoob
Copy link
Owner

mrdoob commented Aug 12, 2022

Thanks!

@WestLangley WestLangley deleted the dev-buffer_geometries branch August 12, 2022 17:00
abernier pushed a commit to abernier/three.js that referenced this pull request Sep 16, 2022
snagy pushed a commit to snagy/three.js-1 that referenced this pull request Sep 21, 2022
wiledal pushed a commit to pmndrs/react-three-rapier that referenced this pull request Oct 4, 2022
usefulthink added a commit to usefulthink/js-three that referenced this pull request Feb 1, 2023
Last year, [three.js deprecated the `XxxBufferGeometry`
classes](mrdoob/three.js#24352), this change
adjust example code to no longer use them and use the new versions
instead.
amuramoto pushed a commit to googlemaps/js-three that referenced this pull request Feb 2, 2023
Last year, [three.js deprecated the `XxxBufferGeometry`
classes](mrdoob/three.js#24352), this change
adjust example code to no longer use them and use the new versions
instead.
github-actions bot pushed a commit to googlemaps/js-three that referenced this pull request Feb 2, 2023
Last year, [three.js deprecated the `XxxBufferGeometry`
classes](mrdoob/three.js#24352), this change
adjust example code to no longer use them and use the new versions
instead. e488ed2
github-actions bot pushed a commit to ehtick/js-three that referenced this pull request Feb 2, 2023
Last year, [three.js deprecated the `XxxBufferGeometry`
classes](mrdoob/three.js#24352), this change
adjust example code to no longer use them and use the new versions
instead. e488ed2
kui added a commit to kui/7dtd-map that referenced this pull request Jun 11, 2023
Mercury21th000 pushed a commit to Mercury21th000/react-three-rapier that referenced this pull request Nov 17, 2024
naotokato111 added a commit to naotokato111/react-three-rapier that referenced this pull request Nov 20, 2024
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.

4 participants