Skip to content
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

Composition event on web #2173

Closed
wants to merge 50 commits into from
Closed

Composition event on web #2173

wants to merge 50 commits into from

Conversation

KentaTheBugMaker
Copy link

@KentaTheBugMaker KentaTheBugMaker commented Jan 30, 2022

  • Tested on all platforms changed
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality
  • Updated feature matrix, if new features were added or implemented

current tested platform

  • Firefox 96.0.2(windows)
  • Firefox 96.0.2(ubuntu)
  • MS edge 97.0(windows)
  • Epiphany(Ubuntu 20.04)
  • Firefox Nightly 98.0a1(Android 9)
  • Android WebView 97.04(Android 9)

buggy platforms

  • Chrome on android 9 with Microsoft SwiftKey.

expected preferable behavior Firefox with MS Swiftkey
compositionstart: compositionupdate: ん compositionupdate: に compositionupdate: にh compositionupdate: にほ compositionupdate: にほん compositionupdate: にほんg compositionupdate: にほんご compositionupdate: にほんごん compositionupdate: にほんごんy compositionupdate: にほんごにゅ compositionupdate: にほんごにゅう compositionupdate: にほんごにゅうr compositionupdate: にほんごにゅうry compositionupdate: にほんごにゅうりょ compositionupdate: にほんごにゅうりょk compositionupdate: にほんごにゅうりょく compositionupdate: 日本語入力 compositionend: 日本語入力

buggy behavior Chrome with MS Swiftkey
compositionstart: compositionupdate: compositionstart: compositionupdate: ん compositionend: ん compositionstart: compositionupdate: ん compositionupdate: に compositionend: に compositionstart: compositionupdate: にh compositionupdate: にほ compositionend: にほ compositionstart: compositionupdate: にほ compositionupdate: にほん compositionend: にほん compositionstart: compositionupdate: にほんg compositionupdate: にほんご compositionend: にほんご compositionstart: compositionupdate: にほんご compositionupdate: にほんごん compositionend: にほんごん compositionstart: compositionupdate: にほんごんy compositionupdate: にほんごにゅ compositionend: にほんごにゅ compositionstart: compositionupdate: にほんごにゅ compositionupdate: にほんごにゅう compositionend: にほんごにゅう compositionstart: compositionupdate: にほんごにゅうry compositionupdate: にほんごにゅうりょ compositionend: にほんごにゅうりょ compositionstart: compositionupdate: にほんごにゅうりょk compositionupdate: にほんごにゅうりょく compositionend: にほんごにゅうりょく compositionstart: compositionupdate: にほんごにゅうりょく compositionupdate: nihongonyuuryoku compositionend: nihongonyuuryoku

How to test

Open Web demo using egui and enter text to TextEdit.
if you want to try it on mobile devices please read this QR code.
image

This PR also support mobile text input.
I don't have macos/ios computer.please test it.

@madsmtm madsmtm added DS - web C - waiting on maintainer A maintainer must review this code labels Feb 4, 2022
KentaTheBugMaker and others added 5 commits April 9, 2022 18:13
This commit brings new IME event to account for preedit state of input
method, also adding `Window::set_ime_allowed` to toggle IME input on
the particular window.

This commit implements API as designed in #1497 for desktop platforms.

Co-authored-by: Artur Kovacs <kovacs.artur.barnabas@gmail.com>
Co-authored-by: Murarth <murarth@gmail.com>
Co-authored-by: Yusuke Kominami <yukke.konan@gmail.com>
Co-authored-by: moko256 <koutaro.mo@gmail.com>
KentaTheBugMaker and others added 6 commits April 15, 2022 18:40
…/winit into rw-composition-event

# Conflicts:
#	CHANGELOG.md
#	src/event.rs
#	src/platform_impl/linux/wayland/window/shim.rs
#	src/window.rs
This commit brings new IME event to account for preedit state of input
method, also adding `Window::set_ime_allowed` to toggle IME input on
the particular window.

This commit implements API as designed in #1497 for desktop platforms.

Co-authored-by: Artur Kovacs <kovacs.artur.barnabas@gmail.com>
Co-authored-by: Murarth <murarth@gmail.com>
Co-authored-by: Yusuke Kominami <yukke.konan@gmail.com>
Co-authored-by: moko256 <koutaro.mo@gmail.com>
@kchibisov kchibisov force-pushed the composition-event branch 4 times, most recently from 2a0f171 to 98540a6 Compare April 17, 2022 19:21
@kchibisov kchibisov force-pushed the composition-event branch 2 times, most recently from 86c5257 to 0f30ac6 Compare April 24, 2022 20:38
@KentaTheBugMaker
Copy link
Author

i don't have enough time to work on this PR.
feel free to fork or implement it originally.

@kchibisov kchibisov force-pushed the composition-event branch from f699224 to aa9c7be Compare May 7, 2022 01:33
@madsmtm madsmtm deleted the branch rust-windowing:master June 10, 2022 14:54
@madsmtm madsmtm closed this Jun 10, 2022
@madsmtm madsmtm reopened this Jun 11, 2022
@madsmtm madsmtm changed the base branch from composition-event to master June 11, 2022 02:32
@KentaTheBugMaker KentaTheBugMaker closed this by deleting the head repository Nov 6, 2022
@daxpedda daxpedda mentioned this pull request May 5, 2023
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C - waiting on maintainer A maintainer must review this code DS - web
Development

Successfully merging this pull request may close these issues.

6 participants