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

IE9-11 - setting editor to readOnly mode and pressing Delete throws unspecified error #1916

Closed
jswiderski opened this issue Apr 23, 2018 · 4 comments
Assignees
Labels
browser:ie9 The issue can only be reproduced in the Internet Explorer 9 browser. browser:ie10 The issue can only be reproduced in the Internet Explorer 10 browser. browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. regression This issue is a regression. status:confirmed An issue confirmed by the development team. support An issue reported by a commercially licensed client. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone

Comments

@jswiderski
Copy link
Contributor

jswiderski commented Apr 23, 2018

Are you reporting a feature request or a bug?

Bug

Provide detailed reproduction steps (if any)

  1. Put attached HTML file on server and run it in a IE9-11 browser.
    EDIT: You can also use this codePen if you need online example.
  2. Click inside e.g. Heading 1 and press Delete

selection_error.zip

Expected result

Nothing happens

Actual result

Many selection-fake-reset, selection-fake-reset and range-startcontainer errors are reported.

Also for IEs: Unspecified error gets thrown in selection.js in line 2202. Code throwing error is sel.addRange( nativeRange );

Other details

According to the reporter:

In IE, native selection would throw this error if nativeRange wraps around an invisible node (display: none).

It might a good tip to investigate.

Please note that problem can be reproduced from CKEditor 4.7.0. so this is rather new issue.

  • Browser: IE9-11, Edge
  • OS: Windows
  • CKEditor version: 4.7.0+
  • Installed CKEditor plugins: N/A
@jswiderski jswiderski added type:bug A bug. status:confirmed An issue confirmed by the development team. support An issue reported by a commercially licensed client. browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. labels Apr 23, 2018
@jswiderski jswiderski changed the title IE11 - setting editor to readOnly mode and pressing Delete throws unspecified error. IE9-11 - setting editor to readOnly mode and pressing Delete throws unspecified error. Apr 23, 2018
@jswiderski jswiderski added browser:ie9 The issue can only be reproduced in the Internet Explorer 9 browser. browser:ie10 The issue can only be reproduced in the Internet Explorer 10 browser. labels Apr 23, 2018
@mlewand mlewand added regression This issue is a regression. target:minor Any docs related issue that can be merged into a master or major branch. labels Apr 23, 2018
@mlewand mlewand added this to the Backlog milestone Apr 23, 2018
@mlewand mlewand changed the title IE9-11 - setting editor to readOnly mode and pressing Delete throws unspecified error. IE9-11 - setting editor to readOnly mode and pressing Delete throws unspecified error Apr 23, 2018
@jswiderski
Copy link
Contributor Author

NOTE: In IE9-11 and Edge in CKEditor 4.6.2 and below the backspace button was causing navigation to previous page. This is no longer the case starting from CKEditor 4.7.0. This is something worth taking into account as well.

@ThomasBartelsNOS
Copy link

Hello,

Is there any news on this issue? We see this is tagged as "minor"...
We are experiencing a lot of side-effects in our application caused by this issue. Did anyone already found a workaround/hotfix?

I would be happy to hear from you guys...

Kind regards,
Thomas

@jacekbogdanski jacekbogdanski self-assigned this Aug 9, 2018
@Comandeer Comandeer modified the milestones: Backlog, 4.10.1 Aug 10, 2018
@mlewand
Copy link
Contributor

mlewand commented Aug 10, 2018

@ThomasBartelsNOS it took as a little to nail this one 🐛 - but we finally have it.

A fix for this issue is ready and has been merged to our upcoming release. It will be available in 4.10.1.

However you can already see it on CKEditor preview builds.

I have forked the original codepen so that it uses preview CKE4 build here: https://codepen.io/mlewand/pen/rroRGQ

@ThomasBartelsNOS
Copy link

Great, thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser:ie9 The issue can only be reproduced in the Internet Explorer 9 browser. browser:ie10 The issue can only be reproduced in the Internet Explorer 10 browser. browser:ie11 The issue can only be reproduced in the Internet Explorer 11 browser. regression This issue is a regression. status:confirmed An issue confirmed by the development team. support An issue reported by a commercially licensed client. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

5 participants