X Tutup
Skip to content

Fix XR tracker name changing at runtime#110703

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
dsnopek:xr-tracker-change
Sep 23, 2025
Merged

Fix XR tracker name changing at runtime#110703
Repiteo merged 1 commit intogodotengine:masterfrom
dsnopek:xr-tracker-change

Conversation

@dsnopek
Copy link
Contributor

@dsnopek dsnopek commented Sep 19, 2025

I noticed this issue with XRHandModifier3D when working on my recent jam game.

XRHandModifier3D will work great if you set the hand_tracker to the tracker name before it's added to the scene tree, but if you change it later, it will have no effect - it'll just keep using the old tracker.

Looking at XRBodyModifier3D and XRFaceModifier3D, they appear to have the same problem as well.

@dsnopek dsnopek added this to the 4.6 milestone Sep 19, 2025
@dsnopek dsnopek requested a review from a team September 19, 2025 19:46
@dsnopek dsnopek requested a review from a team as a code owner September 19, 2025 19:46
@dsnopek dsnopek added the bug label Sep 19, 2025
@BastiaanOlij
Copy link
Contributor

I think this is something we have in a number of places. I use the is_inside_tree check all the time in XR tools.

@BastiaanOlij BastiaanOlij added the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Sep 22, 2025
@Repiteo Repiteo merged commit 5a8a3ed into godotengine:master Sep 23, 2025
39 of 40 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 23, 2025

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 4.5.2.

@akien-mga akien-mga removed the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Jan 8, 2026
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

X Tutup