Make WordPress Core

Opened 12 years ago

Closed 11 years ago

#28003 closed defect (bug) (fixed)

Drag and Drop of photos with Captions not working

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

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

Download all attachments as: .zip

Change History (12)

#1 @pvendeavors
12 years ago

  • Focuses ui administration added

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

  • Component EditorTinyMCE

#4 @iseulde
12 years ago

  • Focuses administration removed

#5 @iseulde
12 years ago

  • Focuses javascript added
  • Keywords has-patch added
  • Milestone Awaiting ReviewFuture Release

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

@iseulde
11 years ago

#6 @iseulde
11 years ago

  • Milestone Future Release4.4

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

@iseulde
11 years ago

@iseulde
11 years ago

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

In 35199:

TinyMCE: make captioned images draggable

See #28003.

#9 @iseulde
11 years ago

  • Resolutionfixed
  • Status newclosed

Looking good.

Note: See TracTickets for help on using tickets.