Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
The following line 39 only checks if IE and not any version. As of IE 10, the method createRange() does not exist and causes a js error:
if (CKEDITOR.env.ie) {
data.content = selection.getNative().createRange().text;
}
else {
data.content = selection.getNative().toString();
}
Related fix in One Click Upload module: https://www.drupal.org/node/1965442
Comment | File | Size | Author |
---|---|---|---|
#4 | media_wysiwyg-fix-ie10-2309233-4.patch | 1.1 KB | randallknutson |
#1 | media_wysiwyg-fix-ie10-2309233-1.patch | 710 bytes | recrit |
Comments
Comment #1
recrit CreditAttribution: recrit commentedThe attached patch adds a check for less than IE 10.
Comment #2
Devin Carlson CreditAttribution: Devin Carlson commentedLooks good.
Verified that IE11 supports the standard ECMA script objects and CKEDITOR.env.version contains the correct version number.
Committed to Media 7.x-2.x.
Comment #4
randallknutson CreditAttribution: randallknutson commentedUnfortunately this doesn't work with all combinations of IE 9 & 10 running in the various compatibility modes. We should follow the same pattern as https://www.drupal.org/node/1914904
Attaching a patch to make it work the same.
Comment #5
randallknutson CreditAttribution: randallknutson commentedComment #7
joseph.olstadIE10 is officially extinct