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

Japanese input cause error, on iPhone SE chrome browser with CKEditor5 19.0.0. #6821

Closed
higty opened this issue May 14, 2020 · 26 comments
Closed
Labels
browser:chrome browser:ios domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). resolution:resolved This issue was already resolved (e.g. by another ticket). type:bug This issue reports a buggy (incorrect) behavior.

Comments

@higty
Copy link

higty commented May 14, 2020

πŸ“ Provide detailed reproduction steps (if any)

In iPhone chrome, I input Japanse text error occurred.

βœ”οΈ Expected result

No error occurred.

❌ Actual result

error occurred.

πŸ“ƒ Other details

  • Browser: …chrome
  • OS: …iPhone SE
  • CKEditor version: …CKEditor5 19.0.0

@higty higty added the type:bug This issue reports a buggy (incorrect) behavior. label May 14, 2020
@Mgsy
Copy link
Member

Mgsy commented May 18, 2020

Hi! Thanks for the report. Unfortunately, I'm not able to reproduce the issue. Could you please provide more details (iOS version, used keyboard), steps to reproduce with some video (if possible)?

@Mgsy Mgsy added the pending:feedback This issue is blocked by necessary feedback. label May 18, 2020
@higty
Copy link
Author

higty commented May 18, 2020

Here is a video.
Hignull_Screenshot

And you can reproduce this behavior at
https://www.hignull.com/CKEditor
with iPhone SE latest chrome browser.

And additionaly, the bug report linked below may be caused by same reason.
#6700

regards.

@higty
Copy link
Author

higty commented May 29, 2020

Is there any progress?

@Mgsy
Copy link
Member

Mgsy commented Jun 1, 2020

Hey @higty, I'm sorry for a late reply. Unfortunately, I'm still not able to reproduce your issue. It doesn't crash for me on iPhone 8 (iOS 13.4.1) and Google Chrome. Could you please check if you're able to reproduce it in our documentation example? If so, please provide exact steps to reproduce.

@higty
Copy link
Author

higty commented Jun 1, 2020

Thank you for your progress.
I try on https://ckeditor.com/docs/ckeditor5/latest/examples/builds/classic-editor.html.
And I can reproduce the bug.

1.Go to https://ckeditor.com/docs/ckeditor5/latest/examples/builds/classic-editor.html
2.Clear all text in ckeditor.
3.Key stroke "あ", "か", "さ"

Actual behavior ああかさ is inputed.
Expected behavior あかさ is inputed.

I will provide anything that will help you.

regards.

@higty
Copy link
Author

higty commented Jun 4, 2020

Hi.
I took video for reprodcut step.
https://vimeo.com/425881160/60decb6415

regards.

@Mgsy
Copy link
Member

Mgsy commented Jun 5, 2020

Thank you, now I was able to reproduce this issue in Chrome browser. However, in Safari it works properly. Could you please confirm that this issue doesn't exist in Safari?

@Mgsy Mgsy added browser:chrome browser:ios and removed pending:feedback This issue is blocked by necessary feedback. labels Jun 5, 2020
@Mgsy Mgsy added this to the backlog milestone Jun 5, 2020
@higty
Copy link
Author

higty commented Jun 5, 2020

Safari on iPhone SE works perfect!

@Mgsy
Copy link
Member

Mgsy commented Jun 5, 2020

Safari on iPhone SE works perfect!

Thanks for confirming it! :) So, the issue occurs only in mobile-Chrome browser.

@higty
Copy link
Author

higty commented Jun 5, 2020

Yes, only mobile-Chrome browser.

And FYI, it might be related to this issue.
#6700
Behavior is resemble.

regards.

@higty
Copy link
Author

higty commented Oct 17, 2020

Hi. is it any pregress?

@higty
Copy link
Author

higty commented Jan 7, 2021

Hi!!
It is very critical bug for China, Korea and Japanese language.
Is there any progress???
Is there any milestone to solve this bug?

@Mgsy
Copy link
Member

Mgsy commented Jan 8, 2021

Hi @higty! There will be a huge change in the editor engine because we plan to continue our work on beforeInput migration. Hopefully, after it's done, a major part of IME issues will be fixed. You can track the progress here - #7462.

@higty
Copy link
Author

higty commented Jan 8, 2021

Hi, @Mgsy!!!
I'm relieved to hear about the progress. And I also realized that the changes would be huge and difficult.
I'm really looking forward to getting this bug fixed. I will watch #7462.

@higty
Copy link
Author

higty commented Jun 4, 2021

Is there any progress?

@Mgsy
Copy link
Member

Mgsy commented Jun 9, 2021

@higty, unfortunately not yet. We'd love to work on it and it's high on our priorities list, however, we still have some other commitments and roadmap plans that we should handle first.

@higty
Copy link
Author

higty commented Jul 5, 2021

Hi.
I want to try to create work around of library.
Where should I look at the source code that I will try to fix?
Is there any document of architecture of entire library?

@higty
Copy link
Author

higty commented Sep 16, 2021

@Mgsy
Is there any progress?? It is really critical bug for Japanese, Chinese, Korean language.
Please consider to change the priority of this issue.

regards.

@Mgsy
Copy link
Member

Mgsy commented Sep 20, 2021

@higty, we really understand that issues related to IME are crucial for some projects, however, as I mentioned before, those issues are not trivial and require massive changes in the editor codebase, i.e. migrating to beforeInput. It's high on our priorities list, but currently, I can't promise any ETA.

@Reinmar Reinmar added the domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). label Nov 4, 2021
@pomek pomek removed this from the backlog milestone Feb 21, 2022
@higty
Copy link
Author

higty commented Mar 16, 2022

Hi.
Unfortunately, We decided to move TinyMCE.
So, you don't fix this bug with high priority.

regards.

@Reinmar
Copy link
Member

Reinmar commented Dec 12, 2022

@ckeditor/qa-team could you retest this issue on an up-to-date iOS?

@dufipl
Copy link
Contributor

dufipl commented Dec 13, 2022

@Reinmar using those reproduction steps:

1.Go to https://ckeditor.com/docs/ckeditor5/latest/examples/builds/classic-editor.html
2.Clear all text in ckeditor.
3.Key stroke "あ", "か", "さ"

Actual behavior ああかさ is inputed.
Expected behavior あかさ is inputed.

from comment #6821 (comment)
I was not able to reproduce the issue on Chrome mobile installed on iPhone with iOS 16.0.3:

RPReplay_Final1670913919.MP4

@Mgsy
Copy link
Member

Mgsy commented Dec 15, 2022

I'm closing it, as it is no longer reproducible.

@Mgsy Mgsy closed this as completed Dec 15, 2022
@Reinmar Reinmar added the resolution:resolved This issue was already resolved (e.g. by another ticket). label Dec 16, 2022
@yanghoxom
Copy link

it happens for me in Ckeditor 29.1.0 for react
I can confirm this happened fixed on the current classic editor version (look like 38.0.1)

@Witoso
Copy link
Member

Witoso commented May 25, 2023

@memsenpai could you open a new issue with the browser's details and a recording πŸ™

@yanghoxom
Copy link

@Witoso
I think it is not necessary because it did not happen on the latest version
I'm working on upgrading my project to the latest version of ckeditor5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:chrome browser:ios domain:typing/ime This issue reports a problem with standard typing & IME (typing method for CJK languages). resolution:resolved This issue was already resolved (e.g. by another ticket). type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

7 participants