Skip to content

Implement API for obtaining glasses friendly name - #72

Merged
patrickdown merged 1 commit into
GodotVR:mainfrom
patrickdown:add_friendly_name
Oct 6, 2025
Merged

Implement API for obtaining glasses friendly name#72
patrickdown merged 1 commit into
GodotVR:mainfrom
patrickdown:add_friendly_name

Conversation

@patrickdown

Copy link
Copy Markdown
Collaborator

This replaces PR#71

This change allows the friendly name defined in the Tilt Five control panel to be accessed.

This change also removes the default_display_name option which was producing confusion because it was badly named and was misinterpreted for the the friendly name above. This was the name that was sent to the Tilt Five control panel to display with the glasses while the application was running. But really this should just be the application name and so the plugin now just sends the application name in it's place.

The example apps were also amended to demo the feature.

@patrickdown
patrickdown requested a review from BastiaanOlij April 2, 2024 13:09
Comment thread example.csharp/addons/tiltfive/T5Manager.cs Outdated
@patrickdown
patrickdown force-pushed the add_friendly_name branch 2 times, most recently from 55fdae1 to 27bf239 Compare October 5, 2025 18:41
This change allows the friendly name defined in the Tilt Five control panel to be accessed.

This change also removes the default_display_name option which was producing  confusion because it was badly named and was misinterpreted for the the friendly name above. This was the name that was sent to the Tilt Five control panel to display with the glasses while the application was running. But really this should just be the application name and so the plugin now just sends the application name in it's place.

The example apps were also amended to demo the feature.
amended: Fixed obsolete message for GetUIDisplayName method

Co-authored-by:  Jonathan Stevens <jstevens@tiltfive.com>
@patrickdown
patrickdown merged commit 5ad1a61 into GodotVR:main Oct 6, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants