X Tutup
Skip to content

Add XROrigin3D scale warning#113979

Merged
akien-mga merged 1 commit intogodotengine:masterfrom
rsanchezsaez:apple/xr-node-warnings
Jan 9, 2026
Merged

Add XROrigin3D scale warning#113979
akien-mga merged 1 commit intogodotengine:masterfrom
rsanchezsaez:apple/xr-node-warnings

Conversation

@rsanchezsaez
Copy link
Contributor

@rsanchezsaez rsanchezsaez commented Dec 13, 2025

Dear Godot community,

This small PR spun off #109975

It adds a warning advising against changing the scale in XROrigin3D. I think this warning applies to all XR platforms, and is a good addition to guide developers towards best practices.

@AThousandShips AThousandShips added this to the 4.x milestone Dec 15, 2025
Copy link
Contributor

@dsnopek dsnopek left a comment

Choose a reason for hiding this comment

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

Thanks!

I agree that having warning messages for these cases makes sense! I added some suggested wording changes, but they're pretty subjective.

@rsanchezsaez rsanchezsaez force-pushed the apple/xr-node-warnings branch 2 times, most recently from db63aeb to 77c1a0a Compare December 17, 2025 23:52
@rsanchezsaez rsanchezsaez changed the title Add XROrigin3D and XRCamera3D warnings Add XROrigin3D scale warning Dec 17, 2025
@rsanchezsaez rsanchezsaez force-pushed the apple/xr-node-warnings branch from 77c1a0a to 3106b11 Compare January 9, 2026 01:05
@BastiaanOlij
Copy link
Contributor

Thanks for adding the message @rsanchezsaez , it's been a really weird one that this has always caused issues.

@akien-mga akien-mga modified the milestones: 4.x, 4.6 Jan 9, 2026
@akien-mga akien-mga merged commit f33a83c into godotengine:master Jan 9, 2026
20 checks passed
@akien-mga
Copy link
Member

Thanks!

@rsanchezsaez rsanchezsaez deleted the apple/xr-node-warnings branch February 8, 2026 00:51
rivie13 pushed a commit to rivie13/Phoenix-Agentic-Engine that referenced this pull request Feb 16, 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