Opened 8 years ago
Closed 8 years ago
#16728 closed Bug (fixed)
[QM] Editor is broken once Copy Formatting is added
Reported by: | Marek Lewandowski | Owned by: | Marek Lewandowski |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.6.2 |
Component: | General | Version: | 4.6.0 |
Keywords: | Cc: |
Description (last modified by )
- With IE in Quirks Mode open an editor with copy formatting, e.g. simply samples page.
Expected:
Editor gets loaded.
Actual:
An exception is thrown Object doesn't support this property or method
.
The problem is caused by this line. Simply there's no underlying querySelectorAll method in QM.
It can be avoided by simply changing function to the following:
_getScreenReaderContainer: function() { if ( CKEDITOR.env.ie6Compat ) { // findOne is not supported on Quirks. return null; } return CKEDITOR.document.getBody().findOne( '.cke_copyformatting_notification div[aria-live]' ); }
Change History (6)
comment:1 Changed 8 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 8 years ago by
Milestone: | CKEditor 4.6.1 → CKEditor 4.6.2 |
---|
comment:4 Changed 8 years ago by
Owner: | set to Marek Lewandowski |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 8 years ago by
Status: | assigned → review |
---|
comment:6 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review → closed |
LGTM. Fixed with git:05987f5.
Pushed to branch:t/16728 - didn't add a manual test since this bug can be reproduced virtually anywhere :D