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

Disable sRGBA format for WebXR #24222

Merged
merged 1 commit into from
Jun 13, 2022
Merged

Disable sRGBA format for WebXR #24222

merged 1 commit into from
Jun 13, 2022

Conversation

cabanier
Copy link
Contributor

The WebXR manager is creating an sRGB swapchain when using layers.
This is different from 2D or the regular WebXR workflows.

There were bugs in the Quest browser where it didn't handle sRGB gamma correction properly. However, we fixed that bug and now we see that three.js has the wrong colors when the experience is using sRGB.
With latest browser, you will see that gamma is applied twice, likely once by three and once when rgb colors are rendered into the srgb swapchain.

This contribution is funded by Meta

@cabanier cabanier closed this Jun 10, 2022
@cabanier cabanier reopened this Jun 10, 2022
@cabanier
Copy link
Contributor Author

@mrdoob Can we merge this? The current behavior will break a future build of the Quest browser.

@cabanier
Copy link
Contributor Author

also @Mugen87

@Mugen87 Mugen87 added this to the r142 milestone Jun 11, 2022
@mrdoob mrdoob merged commit e4dc2f3 into mrdoob:dev Jun 13, 2022
@mrdoob
Copy link
Owner

mrdoob commented Jun 13, 2022

Thanks!

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
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