[IE8] Modernizr 2.5.x causes error when opening link dialog

[Jason Woodward]

Steps to reproduce:

• Fire up IE8 on Windows XP. I used the one available from ​http://modern.ie
• Open a page which loads Modernizr 2.5.x (we tested 2.5.3).
• Click in the text.
• Click the link button.

Message: Invalid argument.
Line: 448
Char: 302
Code: 0
URI: http://ckeditor.com/apps/ckeditor/4.3/ckeditor.js?mxj8dy

After upgrading Modernizr everything works fine.

[Jason Woodward]

Screenshot of javascript error popup dialog

[Dennis1993]

update your stone-age system. You don't drink 10 years old milk!

[Jason Woodward]

Replying to Dennis1993:

update your stone-age system. You don't drink 10 years old milk!

I'm guessing you're making a joke here, but in case you're not, check out the list of "A-Grade" supported browsers at ​http://docs.ckeditor.com/#!/guide/dev_browsers which states IE8 is supported by CKEditor at the same level as more modern browsers.

Perhaps this should be a documentation bug then, with the support list being updated if the policy has changed since ​http://docs.ckeditor.com/#!/guide/dev_browsers was last updated?

[Piotrek Koszuliński]

Well... yes, error is thrown (you can ignore my screenshots). IE8 is supported and every CKEDITOR release is tested on every supported browser. But on the other hand it's impossible to test everything and fix all bugs. For example, I cannot reproduce this bug locally, so perhaps that's why we missed it.

[Piotrek Koszuliński]

Couldn't reproduce this locally, even with filebrowser enabled:

Error thrown on ​http://ckeditor.com/demo#inline. To reproduce it's enough to focus any editor and click link button:

Because of error dialog initialization is not finished (that fieldset should be hidden):

[Jakub Ś]

I only got this error on Demo page but not in my local build.

@woodwardjd could you please download editor full package put it on your server and see if you get same error?
This can be Demo specific error that can be caused by e.g. some third-party script and we call those a lot.

[Piotrek Koszuliński]

This error is ridiculous. IE fails on simple:

document.createElement( 'select' ).add( document.createElement( 'option' )

On ckeditor.com/demo#inline:

Locally:

[Jason Woodward]

Thanks for checking this out. It works fine on a dummy installation ​http://woodwardjd.github.io/cktest/

It is failing in an internal application, but since it failed on the ckeditor.com site I decided to submit with that in the bug report. I will work to isolate the configuration settings and other various bits of JS under which these errors are occurring, but it'll take me a few days.

Webpage error details

User Agent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0)
Timestamp: Mon, 9 Dec 2013 19:40:15 UTC


Message: Invalid argument.
Line: 480
Char: 302
Code: 0
URI: http://webeditor-sha-main-jdw5.test.shawebtest.cornell.edu/enki_editor/ckeditor/ckeditor.js

[Piotrek Koszuliński]

We found a reason. The bug was caused by Modernizr 2.5.3. We updated it to 2.7.1 and the error does not pop out.