-
Notifications
You must be signed in to change notification settings - Fork 28
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
[DRAFT] Support making hybrid apps for Pico headsets #246
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -59,9 +59,10 @@ func _process(_delta: float) -> void: | |
panel_switcher_layer.update_pointer(t.origin, -t.basis.z, pointer_pressed) | ||
|
||
func _on_right_controller_pointer_button_pressed(p_name: String) -> void: | ||
if p_name == 'trigger_click': | ||
print("button: ", p_name) | ||
if p_name == 'trigger_click' or p_name == 'select_button': | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure what's going on with this. I took a quick peek at the action map, and it seemed like I should be getting "trigger_click" for Pico 4 controllers, but instead I'm getting "select_button". Needs to be investigated more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could this be related to the issue mentioned in RC, where Pico defaults to a compat mode? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yep, I think it's the same thing, but I haven't had a chance to investigate further yet. But we'll definitely want to have the action map working correctly before taking this out of draft. |
||
pointer_pressed = true | ||
|
||
func _on_right_controller_pointer_button_released(p_name: String) -> void: | ||
if p_name == 'trigger_click': | ||
if p_name == 'trigger_click' or p_name == 'select_button': | ||
pointer_pressed = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious about this meta-data.. should its value be
vr
when we set the immersive activity as the default launcher?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might just control where the app is sorted in the library in headset. I need to experiment with that a bit.
But if that is correct, then, yeah, perhaps we should only set it when we start as panel, with the idea that the app is primarily a panel app.