Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22188 closed defect (bug) (fixed)

TinyMCE: Edit/Delete Buttons Missing for Captioned Images in WebKit Browsers

Reported by: designsimply's profile designsimply Owned by: azaozz's profile azaozz
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: TinyMCE Keywords:
Focuses: Cc:

Description

Image edit buttons don't appear for captioned images in Chrome 22.

If you step back to r21575, the buttons work normally. r21576 seems to be where it breaks. If you step forward to r22229, there is an "X" button at the top right of the image when you click on it, but captions don't work at all in the visual editor so you can't test the image Edit/Delete buttons with captioned images and clicking the "X" button leaves the caption shortcode text when it should remove it.

Steps to reproduce:

  1. Revert to r21576
  2. Insert an image with a caption into a post and click the image
  • Expected: to see Edit/Delete buttons appear at the top left of the image
  • Actual: no Edit/Delete buttons appear

Examples:

Image Edit/Delete buttons are missing for r21576 in Chrome 22.0.1229.79 on MacOS 10.7.4

http://f.cl.ly/items/1t2K1K3c0x2d2m342p3W/Screen%20Shot%202012-10-14%20at%201.25.32%20PM.png

Image Edit/Delete buttons work fine for r21575 in Chrome 22.0.1229.79 on MacOS 10.7.4

http://f.cl.ly/items/3w3M2I2q0r2O0p2F3j26/Screen%20Shot%202012-10-14%20at%201.25.02%20PM.png

Captions seem to be broken in the visual editor for latest WP revision as of this writing, r22229

http://f.cl.ly/items/010L0B1J113t1F2h1z37/Screen%20Shot%202012-10-14%20at%201.26.22%20PM.png

Change History (4)

#1 @designsimply
12 years ago

  • Cc sheri@… added
  • Summary changed from TinyMCE: Edit/Delete Buttons Missing for Captioned Images in Chrome to TinyMCE: Edit/Delete Buttons Missing for Captioned Images in WebKit Browsers

#2 @nacin
12 years ago

  • Milestone changed from Awaiting Review to 3.5

I imagine koopersmith's work will render this bug invalid. Moving to 3.5 for review so we can get some eyes on it, though.

#3 @azaozz
12 years ago

Yeah, this all is being replaced by the new media workflows. Patching it for consistency only.

#4 @azaozz
12 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In [22246]:

TinyMCE: fix showing popup buttons on images in WebKit, fixes #22188

Note: See TracTickets for help on using tickets.