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

IE10-IE9 support #175

Closed
jacekbogdanski opened this issue Sep 2, 2020 · 7 comments
Closed

IE10-IE9 support #175

jacekbogdanski opened this issue Sep 2, 2020 · 7 comments
Assignees
Milestone

Comments

@jacekbogdanski
Copy link
Contributor

It seems like spell checking stopped working for older browsers. There is already a reported issue for IE8 - #161, but currently, it also relates to IE9 and IE10.

Reproduction steps are almost identical:

  1. Use system with Internet Explorer 9/10:
  2. Open sample http://cdn.ckeditor.com/4.14.1/full-all/samples/
  3. Try to enable SCAYT

scayt

The only option which works is WSC dialog, there is no console error or anything like that.

If these browsers are no longer supported, it would be great if you could show some information about it or disable spell checking button. Otherwise, it seems like the feature is broken.

@jshaptala
Copy link
Member

Discussed the details in the separate chat, Our team will try to reproduce the issue and get back with the status.

@jshaptala
Copy link
Member

Just to let you know that the problem with IE9-10 has been reproduced and the root problem has also been found. It will be fixed in our upcoming release – mid September. No changes to the plugin are needed. The problem is in the core app.

@jshaptala jshaptala added this to the 5.4.0 milestone Sep 4, 2020
@jacekbogdanski
Copy link
Contributor Author

Thanks for the quick response, sounds great!

@jshaptala jshaptala changed the title IE10-IE8 support IE10-IE9 support Sep 18, 2020
@jshaptala
Copy link
Member

Hi @jacekbogdanski, glad to inform you that yesterday we rolled out release v5.6.4 that restored the support of IE9-10 by SCAYT plugin.

Please note that I've renamed the issue, so now it is about support for IE9-10. There is a separate issue open for IE8 and it was already commented on. We are not going to fix it or provide support for IE8 further.

@f1ames
Copy link
Contributor

f1ames commented Sep 21, 2020

Thanks @jshaptala! 👍 🎉

@jacekbogdanski could you check on our side if it works fine in IE9-10? If so, I guess we could close this ticket.

@jshaptala
Copy link
Member

@f1ames, we've validated the fix on our side. Would you please confirm if this works for you as well?

@jacekbogdanski
Copy link
Contributor Author

jacekbogdanski commented Sep 21, 2020

@jshaptala fix works fine, we've tested it on IE9-10. Thanks for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants