godot icon indicating copy to clipboard operation
godot copied to clipboard

[Web, 3.x] Add IME input support.

Open bruvzg opened this issue 1 year ago • 3 comments

Backport of https://github.com/godotengine/godot/pull/79362 (also include #91695 fixes).

bruvzg avatar May 06 '24 08:05 bruvzg

Thank you for your backport. It seems to work fine on Firefox.

https://github.com/godotengine/godot/assets/640587/e9e0da5b-ee17-41a1-96df-aa65da9e8b8a

However, upon confirming the Japanese input with the Enter key, a new line is inserted simultaneously on Safari on macOS.

https://github.com/godotengine/godot/assets/640587/96ef22f6-f341-48b2-bc9f-5d57a2c6f658

This could be caused by Safari or Godot; it's not clear which one.

jtakakura avatar May 07 '24 02:05 jtakakura

This could be caused by Safari or Godot; it's not clear which one.

Seems to be something Safari specific, I can't reproduce it in Chrome or Firefox. Also reproducible with Safari and 4.x.

bruvzg avatar May 09 '24 06:05 bruvzg

How can I check if it works?

JunShiozawa avatar May 14 '24 07:05 JunShiozawa

@JunShiozawa First, you need to check out this PR. After that, build Godot and its web template. See the following documents for more details.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally?platform=windows https://docs.godotengine.org/en/3.6/development/compiling/introduction_to_the_buildsystem.html https://docs.godotengine.org/en/3.6/development/compiling/compiling_for_web.html

jtakakura avatar May 15 '24 03:05 jtakakura

@akien-mga Why don't you review this PR?

JunShiozawa avatar May 25 '24 08:05 JunShiozawa

@akien-mga Why don't you review this PR?

I would kindly ask you to stop lobbying maintainers to work on your personal priorities. We are all busy and doing what we can with the time we have.

If you absolutely need this PR today, you can merge it in your own branch and use it, following the documentation that was linked to you above.

If you want to help getting this merged, actually testing it and reporting whether it works on all your target browsers would be very helpful.

akien-mga avatar May 25 '24 08:05 akien-mga

@akien-mga I am sorry.

JunShiozawa avatar May 25 '24 11:05 JunShiozawa

@JunShiozawa Did you check if it works? I already explained how to check it. Will this PR meet your expectations if merged as it is?

jtakakura avatar May 25 '24 13:05 jtakakura

Where is Export templates? I can not export HTML5.

JunShiozawa avatar Jun 25 '24 01:06 JunShiozawa

This for Godot 4.x Version is Chrome for Android Japanese input OK. Firefox for Android Japanese input OK. Safari for iPhone Japanese input OK. Firefox for Linux Japanese input OK. Firefox for Windows Japanese input OK. Probably it will work.

JunShiozawa avatar Jun 25 '24 03:06 JunShiozawa

@JunShiozawa Thanks for testing and confirming it works on multiple platforms and browsers.

akien-mga avatar Jun 25 '24 06:06 akien-mga

Thanks!

akien-mga avatar Jun 25 '24 06:06 akien-mga