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

Fix issue with OpenXR environment blend mode not being applied properly #81561

Merged
merged 1 commit into from Sep 14, 2023

Conversation

BastiaanOlij
Copy link
Contributor

This PR ensures environment blending is properly applied by setting the XR_COMPOSITION_LAYER_BLEND_TEXTURE_SOURCE_ALPHA_BIT flag if needed when submitting our images to OpenXR.

While fixing this I also improved some of the checking and added a project settings flag for this.

Fixes #81338

@BastiaanOlij BastiaanOlij added this to the 4.2 milestone Sep 12, 2023
@BastiaanOlij BastiaanOlij self-assigned this Sep 12, 2023
@BastiaanOlij
Copy link
Contributor Author

@hitmuri please let me know if this solves your issue properly.

@BastiaanOlij BastiaanOlij force-pushed the fix_openxr_blend_modes branch 3 times, most recently from 8fb5629 to b9dd50b Compare September 12, 2023 03:53
@hitmuri
Copy link

hitmuri commented Sep 12, 2023

@hitmuri please let me know if this solves your issue properly.

Yes it works perfectly now. Thanks !

@BastiaanOlij BastiaanOlij marked this pull request as ready for review September 13, 2023 07:43
@BastiaanOlij BastiaanOlij requested review from a team as code owners September 13, 2023 07:43
@YuriSizov YuriSizov merged commit 495b89e into godotengine:master Sep 14, 2023
15 checks passed
@YuriSizov
Copy link
Contributor

Thanks!

@BastiaanOlij BastiaanOlij deleted the fix_openxr_blend_modes branch September 14, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passthrough not working on Godot 4 OpenXR with a Varjo XR3 Headset
5 participants