Make WordPress Core

Opened 11 years ago

Closed 10 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 10 years ago.
28003.2.patch (1.5 KB) - added by iseulde 10 years ago.
28003.3.patch (1.6 KB) - added by iseulde 10 years ago.

Download all attachments as: .zip

Change History (12)

#1 @pvendeavors
11 years ago

  • Focuses ui administration added

#2 @illusio26
11 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
11 years ago

  • Component changed from Editor to TinyMCE

#4 @iseulde
11 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
10 years ago

#6 @iseulde
10 years ago

  • Milestone changed from Future Release to 4.4

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

@iseulde
10 years ago

@iseulde
10 years ago

#7 @iseulde
10 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
10 years ago

In 35199:

TinyMCE: make captioned images draggable

See #28003.

#9 @iseulde
10 years ago

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

Looking good.

Note: See TracTickets for help on using tickets.