[wp-trac] [WordPress Trac] #40498: TinyMCE Bug on media modification modal

[WordPress Trac]

#40498: TinyMCE Bug on media modification modal
--------------------------+-----------------------------
 Reporter:  casshern83fr  |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  TinyMCE       |    Version:  4.7.3
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 The function use setBaseAndExtent, make crash with version chrome 58,
 https://www.chromestatus.com/feature/5632500684554240

 From dev version
 tinymce\classes\dom\Selection.js

 Line 644:                               // WebKit egde case selecting
 images works better using setBaseAndExtent when the image is floated
         Line 645:                               if (!rng.collapsed &&
 rng.startContainer == rng.endContainer && sel.setBaseAndExtent && !Env.ie)
 {
         Line 650:
 sel.setBaseAndExtent(
         Line 657:
 // Since the setBaseAndExtent is fixed in more recent Blink versions we
         Line 662:
 sel.setBaseAndExtent(node, 0, node, 1);

 tinymce\js\tinymce\tinymce.js (5 hits)

 // WebKit egde case selecting images works better using setBaseAndExtent
 when the image is floated
         Line 18266:                             if (!rng.collapsed &&
 rng.startContainer == rng.endContainer && sel.setBaseAndExtent && !Env.ie)
 {
         Line 18271:
 sel.setBaseAndExtent(
         Line 18278:
 // Since the setBaseAndExtent is fixed in more recent Blink versions we
         Line 18283:
 sel.setBaseAndExtent(node, 0, node, 1);

 // WebKit egde case selecting images works better using setBaseAndExtent
 when the image is floated
         Line 18266:                             if (!rng.collapsed &&
 rng.startContainer == rng.endContainer && sel.setBaseAndExtent && !Env.ie)
 {
         Line 18271:
 sel.setBaseAndExtent(
         Line 18278:
 // Since the setBaseAndExtent is fixed in more recent Blink versions we
         Line 18283:
 sel.setBaseAndExtent(node, 0, node, 1);

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40498>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform