-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Fixes the extension check required for OIT #10417
Conversation
Thanks for the pull request @sanjeetsuhag!
Reviewers, don't forget to make sure that:
|
@@ -18,6 +18,7 @@ | |||
|
|||
##### Fixes :wrench: | |||
|
|||
- Fixed the rendering issues related to order-independent translucency on iOS devices. [#10417](https://github.com/CesiumGS/cesium/pull/10417) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this is how we should phrase it or if it needs to go into breaking changes (since the workaround initially went in breaking changes)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would definitely add a bullet mentioning that the workaround has been removed. That new bullet probably belongs in breaking changes.
I can confirm this fixes the behavior on my iOS device which could reproduce the initial material issue. |
@sanjeetsuhag I think you have one eslint error to clean up. @lilleyse Could you confirm the fix? |
Looks good |
See #9827 for more discussion.
Based on the information in this WebKit thread, this PR adds a check for
EXT_float_blend
when enabling OIT. It also removes the restriction on OIT for iOS devices added as a workaround in #10337.Tested on an iPhone XR running iOS 15.5 using WebGL via Metal.