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

Added Lines Segments Example #28380

Closed
wants to merge 2 commits into from

Conversation

eduardfossas
Copy link

@eduardfossas eduardfossas commented May 14, 2024

Description

Hello, I don't know how useful this will be to anyone, but I was doing some generative art the other day and I put together a few lines of code trying to use LineSegments and I realised there were no examples in the Three.js documentation site, so I decided to create one.

Let me know if you want me to change something or contribute in another way.

Warm regards.

webgl_lines_segments

@Mugen87
Copy link
Collaborator

Mugen87 commented May 14, 2024

Thanks for filing the PR. But since there are already multiple examples using LineSegments, it would better to share your code in the forum under the resources category.

Existing examples that could be added to the doc page of LineSegments are:

https://threejs.org/examples/webgl_buffergeometry_drawrange
https://threejs.org/examples/webgl_buffergeometry_lines_indexed
https://threejs.org/examples/webgl_buffergeometry_selective_draw

It would be great if you could add them with a new PR.

@Mugen87 Mugen87 closed this May 14, 2024
@eduardfossas
Copy link
Author

Hello @Mugen87. That was exactly my point! If I went to the LineSegments documentation page I could not find any examples, so I am happy to do a new PR adding the examples that you shared.

Thank you!

@Mugen87 Mugen87 added this to the r165 milestone May 17, 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.

None yet

2 participants