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

OpenXR: Allow GDExtensions to implement _on_pre_draw_viewport() and _on_post_draw_viewport() #98831

Conversation

dsnopek
Copy link
Contributor

@dsnopek dsnopek commented Nov 4, 2024

For some reason, OpenXRExtensionWrapperExtension doesn't expose _on_pre_draw_viewport() and _on_post_draw_viewport(), despite exposing all other virtual methods.

I don't think there's any reason for this, I think it was just an accidental omission.

@dsnopek
Copy link
Contributor Author

dsnopek commented Nov 4, 2024

Ah, it looks like PR #97151 also exposes _on_post_draw_viewport() because it needed it. To me, that's a further indication that both of these should be exposed.

Copy link
Member

@fire fire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the XR extension code needs this to work, it should be exposed.

@m4gr3d m4gr3d modified the milestones: 4.x, 4.4 Nov 5, 2024
@dsnopek dsnopek force-pushed the openxr-gdextension-pre-post-draw-viewport branch from 0bfd676 to c72eb04 Compare November 5, 2024 14:16
@Repiteo Repiteo merged commit 1559645 into godotengine:master Nov 6, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 6, 2024

Thanks!

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.

6 participants