-
Notifications
You must be signed in to change notification settings - Fork 258
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
Text input doesn't bring up virtual keyboard on mobile #29
Comments
This would be solved by #27. |
I'm afraid just upgrading to 0.12 doesn't fix it. I guess we need to port the changes that were done in emilk/egui#253 to |
Is it still possible to bring up the mobile keyboard with a WASM build? |
Still impossible, unfortunately. If anyone wants to make a PR for this, I'll gladly accept it. |
Hello, Ive been working on this on this branch (which also containts the work from #180 ). Due to winit lacking support for composition events in the web (as can be seen here) I excluded the event listeners for them as it currently sends them all out at once when the composition ends which causes unwanted behavior. I can create a pull request once #180 is done |
@oscrim did your code fix mobile keyboard issues with bevy_egui? I wasn't sure if this was an IME issue that has to be explored at a lower level. |
It was more of a work around than a proper fix. It's been a while since I looked at it |
Do you think it's something you could eventually upstream? I was still trying to understand if there was a bug in IME somewhere or if some of the code from egui needed to be adapted/bridged to bevy. (Maybe from https://github.com/emilk/egui/blob/master/crates/eframe/src/web/text_agent.rs ) I'm also curious about @Vrixyz opinion. |
I might just steal your code and maybe try to upstream it >.> @oscrim |
@v-kat please do! I don't think the ime not working is a bug, if I interpret the docs correctly there is no ime support on android/ios/web in bevy or winit https://docs.rs/winit/latest/winit/event/enum.WindowEvent.html#variant.Ime |
Implemented in bevy_egui 0.30 |
This was recently fixed in egui itself (emilk/egui#247), but doesn't currently work correctly in bevy_egui.
The text was updated successfully, but these errors were encountered: