[wp-trac] [WordPress Trac] #24409: Edit Image in TinyMCE editor revamp

[WordPress Trac]

#24409: Edit Image in TinyMCE editor revamp
----------------------------+-----------------------------
 Reporter:  eablokker       |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  3.9
Component:  Media           |     Version:  3.5.1
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:  ui, javascript
----------------------------+-----------------------------

Comment (by gcorne):

 > I also noticed that when using the resize handles to reduce the size of
 the image that often on {{{mouseup}}} the modal opens, which is not
 expected. I tried to disable opening the modal by taking advantage of the
 TinyMCE {{{ObjectResizeStart}}} event, but it doesn't seem to be firing
 when expected.

 This comment was actually invalid because [attachment:24409.05.patch
 24409.05.patch] actually doesn't actually open the modal when a clicking a
 second time on a selected image. Chalk it up to getting a little mixed up
 when merging my changes.

 > One drawback of using adding a div that sits above the entire selected
 image is that it inhibits the user from dragging the image to a new
 location, which is not disabled if the image doesn't have a caption.
 Something to keep in mind as we figure out the design/ux.

 To address this for the time being, I removed the overlay in
 [attachment:24409-06.patch 24409-06.patch]. I improved the UX a little by
 removing the toolbar when starting to drag and when cutting or copying a
 selected image to avoid ending up with an orphaned toolbar when dragging
 an image to a new location.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/24409#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform