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

iOS Safari typing text scrolls the ckeditor #498

Open
razyon opened this issue Jun 9, 2017 · 20 comments
Open

iOS Safari typing text scrolls the ckeditor #498

razyon opened this issue Jun 9, 2017 · 20 comments
Labels
browser:ios The issue can only be reproduced on a mobile device with iOS. status:confirmed An issue confirmed by the development team. type:feature A feature request.
Milestone

Comments

@razyon
Copy link

razyon commented Jun 9, 2017

Type of report

Feature request

Provide description of the new feature

This issue occurs due to upstream Safari #184178 bug.

On the main page of ckeditor.com add a few lines to the end of the first ckEditor instantiated (Article Editor) then place your cursor to the top of the article and start typing.

The CKEditor will scroll down and you will not see what you are typing.

Known workarounds

This editor happens only with the classic editor, so using divarea plugin or an inline editor will mitigate the problem.

@msamsel msamsel self-assigned this Jun 28, 2017
@msamsel
Copy link
Contributor

msamsel commented Jun 28, 2017

iOS 10.3.2
iPhone SE
Safari

  1. Open ckeditor.com, or sample (tested on 4.7.0)
  2. Add text in editor, to have scrolled down text inside editor
  3. Unselect editor
  4. Scroll up text in editor to the beginning
  5. Put caret in first line and start to write
  6. Content will scroll down and caret will be above the editor

Picture how it looks, with visible caret.
img_0665

@msamsel msamsel added browser:ios The issue can only be reproduced on a mobile device with iOS. status:confirmed An issue confirmed by the development team. type:bug A bug. labels Jun 28, 2017
@msamsel msamsel removed their assignment Jul 5, 2017
@buguil
Copy link

buguil commented Sep 5, 2017

Is there any update on this issue ?
Why assignment removed on july 5th ? Difficult to fix it ?

@benfarhner
Copy link

iPhone 7 Plus
iOS 10.3.3
Safari & Chrome

I'm able to reproduce this issue as well on my iPhone with the latest version of iOS (10.3.3) in both Safari and Chrome (both up to date as well). This is affecting a major customer of ours who uses iPads for entering and editing long notes with CKEditor. Are there any workarounds or solutions we can use in the meantime until this gets fixed in the main library?

@mlewand
Copy link
Contributor

mlewand commented Sep 11, 2017

@benfarhner We don't have yet any verified workarounds. If you're willing to tinker a bit, you could check the #328 PR - it might help mitigate/resolve this issue.

@peopleplaces
Copy link

Update: PR #328 does not resolve the issue. Using iOS 9.3.5. Also confirmed issue still exists on iOS 10.3.3

@youchenlee
Copy link

This issue exists on my iPhone (on both Safari and Google Chrome) and my Android phone (with Google Chrome), other browsers not been tested yet.

I think this is not only an iOS/Safari issue.

@Mgsy
Copy link
Member

Mgsy commented Dec 4, 2017

I just want to make a small update about this issue. This bug still occurs in iOS (tested with iPad iOS 11.1.2) and CKEditor 4.8.0.

Update: PR #328 does not resolve the issue. Using iOS 9.3.5.

I can confirm that this PR doesn't fix problem with scrolling, but I'm not able to reproduce it in iPod iOS 9.3.5 Safari.

This issue exists on my iPhone (on both Safari and Google Chrome) and my Android phone (with Google Chrome), other browsers not been tested yet.

I've checked it in Google Chrome Android 7.0 and I'm not able to reproduce this issue.

It looks like this one occurs in iOS 10+ only.

@mlewand mlewand added this to the Backlog milestone Dec 12, 2017
@youchenlee
Copy link

@Mgsy thank you for the update.
I will try on my Google Chrome on Android again and update the result here.

@evgeny-belyaev
Copy link

I found workaround for this issue: with autogrow plugin enabled it works just fine for me (iPhone iOS 11.1.1 safari and chrome).

@KupnoH
Copy link

KupnoH commented Jan 23, 2018

Hi everyone!
I'm also experiencing the problem with editor on iphone and ipad. I'm using drupal and by suggestion of evgeny-belyaev on the comment above installed plugin autogrow, but, unfortunately, nothing happens. It still have problems.

@heyflo
Copy link

heyflo commented Jan 24, 2018

Tested on iOS 10+ and I'm reproducing this error with CK Editor module on Drupal 7.
Also tested it on Chrome Android (last version), it is working properly.

I don't know if I should try the autogrow plugin but if we do, I'll keep you updated weither it's working or not.

@engineering-this
Copy link
Contributor

engineering-this commented Apr 3, 2018

Hi there.

This issue is upstream as I could reproduce it without CKEditor.

Webkit issue

I understand that it makes our editor very hard to use on iOS we will try to implement some workaround.

Note: This bug is not occuring when using the divarea plugin.

Edit: PR with proposed workaround: #1861

@mlewand
Copy link
Contributor

mlewand commented Apr 30, 2018

Just to provide an update on this issue: we were trying to tackle this issue, however there is no easy way for us to solve this issue. Just see pull request #1861.

This is an upstream issue so the best would be to fix the issue in Safari itself, so it's a good idea to show your need for that by ccing/commenting https://bugs.webkit.org/show_bug.cgi?id=184178

@benfarhner
Copy link

Does this issue exist in CKEditor 5 since it's not using contentEditable?

@engineering-this
Copy link
Contributor

Does this issue exist in CKEditor 5 since it's not using contentEditable?

This issue shouldn't be reproducible in CKEditor 5 because it doesn't use iframe. CKEditor 4 with divarea works good on iPad too.

@jonasva
Copy link

jonasva commented Jul 4, 2018

CKEditor 4.7 + divarea plugin solved the issue for me (tested on iPhone SE iOS 11.3)

@mlewand
Copy link
Contributor

mlewand commented Jul 27, 2018

Since this is an upstream bug, this issue is now tracked as a feature request to get around a Safari bug.

@mlewand mlewand added the type:feature A feature request. label Jul 27, 2018
@mlewand mlewand added target:major Any docs related issue that should be merged into a major branch. and removed type:bug A bug. labels Jul 27, 2018
@Mgsy
Copy link
Member

Mgsy commented Aug 6, 2018

There is another scenario with a similar effect.

Steps to reproduce

  1. Open a sample with scrollable content.
  2. Focus the editor.
  3. Scroll to the bottom of the editor.
  4. Put the selection in some text.
  5. Scroll up a little.
  6. Put the caret in some text.
  7. Scroll slowly to the top of the editor.
  8. Type something.

Current result

  • After step 7, the caret moves down out of the editor.
  • The editor has been scrolled to the bottom and there is a visible caret over a toolbar (even while typing).

@shram2k
Copy link

shram2k commented Aug 7, 2018

CKEditor 4.7 + divarea resolved this issue on ipad portrait mode but issue still exists in landscape mode. Any workaround?
https://stackoverflow.com/questions/51473653/ckeditor-4-5-11-cursor-moves-to-top-on-typing-in-ipad-v-11-3-1

@mlewand
Copy link
Contributor

mlewand commented Aug 14, 2018

@shram2k none that we're aware of. We'd really appreciate bumping Safari upstream bug, the best solution here is to get upstream fix in Safari.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:ios The issue can only be reproduced on a mobile device with iOS. status:confirmed An issue confirmed by the development team. type:feature A feature request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.