Opened 9 years ago
Closed 9 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 9 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 9 years ago by
| Description: | modified (diff) |
|---|
comment:3 Changed 9 years ago by
| Milestone: | CKEditor 4.6.1 → CKEditor 4.6.2 |
|---|
comment:4 Changed 9 years ago by
| Owner: | set to Marek Lewandowski |
|---|---|
| Status: | confirmed → assigned |
comment:5 Changed 9 years ago by
| Status: | assigned → review |
|---|
comment:6 Changed 9 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