Make WordPress Core

Opened 10 years ago

Closed 9 years ago

#28003 closed defect (bug) (fixed)

Drag and Drop of photos with Captions not working

Reported by: pvendeavors's profile pvendeavors Owned by:
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.9
Component: TinyMCE Keywords: has-patch
Focuses: ui, javascript Cc:

Description

When images have a caption in the visual editor, they cannot be repositioned in the text by dragging and dropping. The user must delete the image, move their cursor and then insert the image again.

After testing with an image that did not have a caption, the drag and drop ability worked and images could be moved anywhere in the editor.

It is possible to use the text editor and drag on drop the entire caption shortcode and image code, but you cannot in the visual editor. So it seems that the shortcode is causing this problem.

After testing in multiple browsers on both Mac and PC the problem seems to be consistent regarding the configuration of the computer.

Attachments (3)

28003.patch (1.8 KB) - added by iseulde 9 years ago.
28003.2.patch (1.5 KB) - added by iseulde 9 years ago.
28003.3.patch (1.6 KB) - added by iseulde 9 years ago.

Download all attachments as: .zip

Change History (12)

#1 @pvendeavors
10 years ago

  • Focuses ui administration added

#2 @illusio26
10 years ago

I am having the same issue. Have tested it on Firefox and Chrome for the mac. In both cases, I cannot move a photo in the editor once a caption has been added.

#3 @iseulde
10 years ago

  • Component changed from Editor to TinyMCE

#4 @iseulde
10 years ago

  • Focuses administration removed

#5 @iseulde
10 years ago

  • Focuses javascript added
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to Future Release

I added a patch to #28272 which would also fix this issue.

@iseulde
9 years ago

#6 @iseulde
9 years ago

  • Milestone changed from Future Release to 4.4

Patch above tested with Chrome, Safari, Firefox, IE8 and IE11.

@iseulde
9 years ago

@iseulde
9 years ago

#7 @iseulde
9 years ago

I'll add this now and leave the ticket open for issues/review.
Using dragend and letting the image drop seems unreliable as the selection is no longer the image.

#8 @iseulde
9 years ago

In 35199:

TinyMCE: make captioned images draggable

See #28003.

#9 @iseulde
9 years ago

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

Looking good.

Note: See TracTickets for help on using tickets.