[wp-trac] [WordPress Trac] #20545: TinyMCE - Edit/Delete Image Buttons Show For Non-editable Items

[WordPress Trac]

#20545: TinyMCE - Edit/Delete Image Buttons Show For Non-editable Items
--------------------------+-----------------------------
 Reporter:  Master Jake   |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  TinyMCE       |    Version:  3.3.1
 Severity:  normal        |   Keywords:
--------------------------+-----------------------------
 Forgive me if this has already been reported and/or fixed in 3.4. I've yet
 to have a chance to try out the BETA.

 The wpEditImage plugin creates Edit and Delete buttons for every image on
 selection. There are several instances where images need to be used as
 content placeholders like in the instance of the <!-- more --> tag and
 [gallery] shortcode. The plugin provides a special class "mceItem" which
 prevents these image editing buttons from appearing on any image with the
 class.

 However, the class apparently isn't checked in all the places it needs to
 be because you can make these image editing buttons appear even when
 they're not supposed to. These even occurs in the case of the [gallery]
 shortcode which has it's own Edit/Delete buttons. If you use one of these
 hacks, you can actually make the Edit/Delete "Image" buttons appear over
 the gallery-specific ones.

 Here are a few examples and attached are some screenshots:

 1. '''Firefox 11''' Even if an image uses the mceItemNoResize, the resize
 handles will still appear on hover. If you try to drag these handles, the
 image doesn't resize (as it shouldn't) but the Edit/Delete image buttons
 do appear. ''See Firefox Screenshot Suite''

 2. '''IE 9''' Insert a [gallery] and hover your mouse just below an image
 until the "text editor cursor" appears. Simply click and the Edit/Delete
 image buttons will appear. You can differentiate these buttons from the
 Edit/Delete gallery buttons by hovering your mouse over them to read their
 title text. Unfortunately, I could not get a good screenshot of this.

 Note: I could not get this error to occur on Safari or Opera.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/20545>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software