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

Improve layers implementation #10

Open
dmarcos opened this issue May 5, 2023 · 6 comments
Open

Improve layers implementation #10

dmarcos opened this issue May 5, 2023 · 6 comments

Comments

@dmarcos
Copy link
Member

dmarcos commented May 5, 2023

Description

We have now a simple implementation:

https://github.com/supermedium/three.js/blob/dev/src/renderers/webxr/WebXRManager.js#L373

It can be tested in

https://aframe.io/aframe/examples/showcase/comicbook/

Can switch layers by pressing thumbstick. Feature only available in immersive mode and Meta Browser on Quest.

I think there are new features available like punch through that I'm happy to incorporate.

@cabanier What do you think? Any code we can incorporate? PRs also super welcome? Thanks!

Solution

n/a

Alternatives

n/a

Additional context

No response

@cabanier
Copy link

cabanier commented May 5, 2023

This commit adds support for hole punching of Quad and Cylinder layers: mrdoob@ea1fad2

This allows layers to mix with the VR scene. The PR is stuck since January :-\

@dmarcos
Copy link
Member Author

dmarcos commented May 5, 2023

Thanks. Do you think it would be possible to move those WebGLTexture changes somewhere else. WebXRManager for instance? WebGLTexture, WebGLRenderer files have a lot of churn and it's hard to maintain in the fork so I try to avoid as much as possible.

@cabanier
Copy link

cabanier commented May 5, 2023

It's actually only a very small change (1 extra check) but for some reason it looks large in the diff because it got confused.

@dmarcos
Copy link
Member Author

dmarcos commented May 5, 2023

I see, thanks. If you have some time to open a PR against the dev branch in this repo I'll review and merge. Probably quick for you since you're familiar with the code. If busy, no worries. I'll incorporate as soon as I have some bandwidth. Thanks for getting back so quickly

@dmarcos
Copy link
Member Author

dmarcos commented May 5, 2023

The additional changes in the diff could indentation, empty space? I have similar issues sometimes since don't have a three style linter on my editor

@dmarcos
Copy link
Member Author

dmarcos commented May 5, 2023

Some editors set to replace spaces for tabs on save or viceversa

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

No branches or pull requests

2 participants