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

[Edge] Calling setData on editor initialization breaks AC in specific cases #229

Open
f1ames opened this issue Nov 18, 2016 · 0 comments
Open
Labels
Milestone

Comments

@f1ames
Copy link
Contributor

f1ames commented Nov 18, 2016

See https://github.com/cksource/new-websites/issues/252 for detailed description of the whole issue.


The problem may occur when editor with a11ychecker enabled is initialized like

var editor = CKEDITOR.replace( 'editor', config ); // Nothing special just `extraPlugins: 'a11ychecker'`
editor.on( 'instanceReady', function( evt ) {
    evt.editor.setData( 'Some sample html...' );
} );

After clicking on a11ychecker toolbar button the error is thrown
image

It happens only for built version of a11ychecker on Edge browser.

Initializing CKEditor on container with html inside does not cause the error. Also deferring setData call like

editor.on( 'instanceReady', function( evt ) {
    setTimeout( function() {
        evt.editor.setData( 'Some sample html...' );
    }, 0 );    
} );

solves the issue. Calling setData after editor instance (with html inside) is properly initialized does not break editor so it is not the issue with only setData method but probably some race condition case.

@f1ames f1ames added the bug label Nov 18, 2016
@f1ames f1ames added this to the 1.1.1 milestone Nov 18, 2016
@f1ames f1ames changed the title [Edge] Calling setData on initialization breaks AC in specific cases [Edge] Calling setData on editor initialization breaks AC in specific cases Nov 18, 2016
@mlewand mlewand modified the milestones: 1.1.1, 1.1.2 Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants