IE8 and MathJax

[Piotr Jasiun]

There is a problem with MathJax in IE8.

I think that we should replace MathJax with placeholder there.

[Piotr Jasiun]

IE8 use different frame wrapper which shows only TeX instead of MathJax. Changes in ​t/11085 and corresponding test branch.

[Piotrek Koszuliński]

1. JSLint issues.
2. JSDuck issues.
3. JSDuck comments do not conform the style guide for documentation.
4. I think that it would be nice if MathJax preview (both in dialog and in editable) could be disabled also on other than IE8 browsers. For example on those on which MathJax works very slowly. So this could be configurable.

[Piotr Jasiun]

Replying to Reinmar:

1. JSLint issues.
2. JSDuck issues.
3. JSDuck comments do not conform the style guide for documentation.

I fixed JSLint and JSDuck issues.

4. I think that it would be nice if MathJax preview (both in dialog and in editable) could be disabled also on other than IE8 browsers. For example on those on which MathJax works very slowly. So this could be configurable.

IE8 placeholder, which is displayed instead of preview, is created only for IE8 and tested only with it. If we want to give users option to use placeholder or preview in any browser we should create more tests and spend some time on this and now we have other important issues to work on.

[Olek Nowodziński]

Instead of

if( CKEDITOR.env.ie && CKEDITOR.env.version == 8 )
    CKTESTER.test( {} );

and wrapping all tests in conditional statement, the following seems much cleaner:

'async:init' : function() {
    if ( CKEDITOR.env.ie && CKEDITOR.env.version == 8 )
        assert.ignore();

    ...
}

Also the following in regressions.js wouldn't work anyway (notation requires .html#test_name):

'/dt/plugins/mathjax/mathjax.html': 'env.ie && env.version == 8',

Anyway, what piece of logic controls the width of the placeholder?

[Piotrek Koszuliński]

I fixed the broken tests, which were ignored from 2 places at the same time.

Question which remains is:

Anyway, what piece of logic controls the width of the placeholder?

[Piotr Jasiun]

Question which remains is:

Anyway, what piece of logic controls the width of the placeholder?

This: ​https://github.com/cksource/ckeditor-dev/blob/t/11085/plugins/mathjax/plugin.js#L414

[Piotrek Koszuliński]

Fixed on master with ​git:4cdcded on dev and e48344b on tests.