-
-
Notifications
You must be signed in to change notification settings - Fork 442
OpenXR - BT/OTG support and hardcode native resolution #671
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
Conversation
| if (XrActivity.isEnabled()) { | ||
| width = 1024; | ||
| height = 1024; | ||
| } |
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.
Probably it will be better to use some 16:9 or at least 4:3 resolution. 1:1 resolution are not the best for human eye perception.
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 made it 4:3 in 27b52aa
| // Bluetooth/OTG keyboards work properly with the standard flow | ||
| if (hasFocus) { | ||
| return super.dispatchKeyEvent(event); | ||
| } |
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.
There is hasWindowFocus function, no need to store current window focus value in separate variable.
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 fixed that in 27b52aa
| } | ||
|
|
||
| @Override | ||
| public boolean dispatchKeyEvent(KeyEvent event) { |
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.
Does LorieView's dispatchKeyEventPreIme work in the case if Activity's dispatchKeyEvent is overridden?
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.
Software keyboard sends randomly event with KEYCODE_MEDIA_PLAY into dispatchKeyEventPreIme.
Hardware keyboard works sends correct events into dispatchKeyEventPreIme.
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 am sorry, I do not understand how is your question related to the code.
What change do you require here?
|
There are pending reviews. |
|
@lvonasek can you please read pending reviews and respond? |
|
I responded to not answered reviews (although I thought this was clear, I am sorry for the misunderstanding). About the answer from @natebragg:
I think the user reacted to ctrl+space issue which isn't expected to work in the XR mode neither (the same issue like #665). |
There must be some issue with GitHub. These comments aren't visible for me. But let me clarify it here: These events are not passing into I personally prefer to avoid "hacky" code if it is not necessary. |
|
What about the last one? |
Yes, |
|
@lvonasek Is it possible to add support for other OpenXR-Compatible devices using the Monado runtime for Android? Monado has drivers for Nreal Air/Pro and Rokid glasses. I enjoy using OpenXR mode for Termux-X11 on the Quest, but being able to use it on my glasses would be great! |
|
This addresses XR issues mentioned in #665.
AndroidManifest.xml)LorieView.java)XrActivity.java)