Skip to content

Add config.yaml defaults, examples, and enum values for handsfree_pointer - #583

Open
rockobonaparte wants to merge 1 commit into
wayvr-org:mainfrom
rockobonaparte:dev/rockobonaparte/handsfree_pointer_yaml
Open

Add config.yaml defaults, examples, and enum values for handsfree_pointer#583
rockobonaparte wants to merge 1 commit into
wayvr-org:mainfrom
rockobonaparte:dev/rockobonaparte/handsfree_pointer_yaml

Conversation

@rockobonaparte

Copy link
Copy Markdown

The configuration system already processed that setting with a default of 'hmd': Head-Mounted Display, Pinch Mode, and VR controllers. This makes that overt while allow it to be changed without needing to use the watch menu in each session.

  • Created serde defaults for the hands-free pointer values
  • Added an explanation of the field and values to the default config.yaml

…nter

* Created serde defaults for the hands-free pointer values
* Added an explanation of the field and values to the default config.yaml

The configuration system already processed that setting with a default of
'hmd': Head-Mounted Display, Pinch Mode, and VR controllers. This makes
that overt while allow it to be changed without needing to use the watch
menu in each session.
@galister

galister commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

I think i want to stop including keys that are configured from the UI in the example config file.

People will either:

  • make a config that's alphabetically before the zz-saved-config.json5 and then it won't take effect
  • make a config that's after saved config and then the UI setting will not work

@galister

galister commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

what do you think @oo8dev

@oo8dev

oo8dev commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

There could be a new "extras" example config file with tweakables not present in the dashboard yet

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.

3 participants