31

I'm trying to make a list of elements which can be repositioned by dragging and dropping. The first element, Box 1, works just fine 100% of the time. Sometimes the second box works, but none of the others work as expected. They seem to fire off all the drag events at once as soon as you start dragging them.

I'm using the latest Chrome (v23) if that matters.

var $questionItems = $('.question-item');

$questionItems
  .on('dragstart', startDrag)
  .on('dragend', removeDropSpaces)
  .on('dragenter', toggleDropStyles)
  .on('dragleave', toggleDropStyles);


function startDrag(){
  console.log('dragging...');
  addDropSpaces();
}

function toggleDropStyles(){
  $(this).toggleClass('drag-over');
  console.log(this);
}


function addDropSpaces(){
  $questionItems.after('<div class="empty-drop-target"></div>');
  console.log('drop spaces added');
}

function removeDropSpaces(){
  $('.empty-drop-target').remove();
  console.log('drop spaces removed');
}

Here's the HTML:

<div class="question-item" draggable="true">Box 1: Milk was a bad choice.</div>
<div class="question-item" draggable="true">Box 2: I'm Ron Burgundy?</div>
<div class="question-item" draggable="true">Box 3: You ate the entire wheel of cheese?</div>
<div class="question-item" draggable="true">Box 4: Scotch scotch scotch</div>

Here is a jsFiddle of my code: http://jsfiddle.net/zGSpP/5/

3
  • Are you using jquery draggables? where are you initializing the elements as draggable via element.draggable();?
    – jbabey
    Jan 7, 2013 at 20:53
  • It is really wird. You should use JQueryUI widget events as workaround : jqueryui.com/draggable
    – sdespont
    Jan 7, 2013 at 21:10
  • jbabey: The HTML5 attribute 'draggable=true'
    – Bill
    Jan 7, 2013 at 21:18

6 Answers 6

40

Had this problem just now - it is a Chrome bug, you must not manipulate the DOM in dragStart event, or else dragEnd fires immediately. The solution for me turned out to be - use setTimeout() and manipulate the DOM inside the function you timeout.

3
  • This worked for me. Also I had the same code working for me on one page variant but not the other and the only difference between them was that parent element for the manipulated element wasn't rendering (was hidden) in the case when it was failing. Mar 13, 2017 at 17:18
  • generally just works well, but sometime must works like this answer..(this worked for me). i don't know what is wrong...
    – JaeIL Ryu
    May 31, 2022 at 9:01
  • For users of modern web frameworks such as Angular, Vue, and React: also consider data-binding mechanisms that can make these DOM manipulations a bit harder to detect.
    – David Brem
    Mar 16, 2023 at 11:52
12

It seems to be a Chrome bug : https://groups.google.com/a/chromium.org/forum/?fromgroups=#!msg/chromium-bugs/YHs3orFC8Dc/ryT25b7J-NwJ

7
  • 2
    This bug still appears in Chrome 32.
    – Akkuma
    Jan 27, 2014 at 20:34
  • 2
    Still seem to be experiencing this problem in Chrome 47.
    – Puppy
    Dec 14, 2015 at 9:21
  • 1
    Still there in Chrome 55.0.2883.87 Jan 11, 2017 at 10:29
  • 3
    And still in Chrome 62 :/ Dec 1, 2017 at 19:47
  • 1
    The linked bug is labelled WontFix for being unreproducible since 2015. Might be a good idea to submit a new bug with a reproduction in a CodePen or jsFiddle. Feb 20, 2018 at 16:08
9

Not sure whether it is a bug or not, but I think the problem comes in when you change the DOM in the dragstart handler. The reason the first box works I presume has something to do with the fact that its position is before the other boxes in the DOM and when you change the DOM, the position (?) of the first one isn't affected and the dragdrop events fire reliably.

You will need to refactor your code somewhat, and add the dropSpaces in the dragenter event handler. You would need to change your addDropSpaces method to accommodate the fact that it will be called multiple times, though.

1

Life is easier with jQuery. This is exactly what you described you wanted, using jQueryUI.

http://jqueryui.com/draggable/#sortable

which allows sorting. The code:

<!doctype html>
<html lang="en">
<head>
    <meta charset="utf-8" />
    <title>jQuery UI Draggable + Sortable</title>
    <link rel="stylesheet" href="http://code.jquery.com/ui/1.9.2/themes/base/jquery-ui.css" />
    <script src="http://code.jquery.com/jquery-1.8.3.js"></script>
    <script src="http://code.jquery.com/ui/1.9.2/jquery-ui.js"></script>
    <link rel="stylesheet" href="/resources/demos/style.css" />
    <style>
    ul { list-style-type: none; margin: 0; padding: 0; margin-bottom: 10px; }
    li { margin: 5px; padding: 5px; width: 150px; }
    </style>
    <script>
    $(function() {
        $( "#sortable" ).sortable({
            revert: true
        });
        $( "#draggable" ).draggable({
            connectToSortable: "#sortable",
            helper: "clone",
            revert: "invalid"
        });
        $( "ul, li" ).disableSelection();
    });
    </script>
</head>
<body> 
<ul>
    <li id="draggable" class="ui-state-highlight">Drag me down</li>
</ul>

<ul id="sortable">
    <li class="ui-state-default">Item 1</li>
    <li class="ui-state-default">Item 2</li>
    <li class="ui-state-default">Item 3</li>
    <li class="ui-state-default">Item 4</li>
    <li class="ui-state-default">Item 5</li>
</ul>

</body>
</html> 

also see: http://jqueryui.com/sortable/

Try adding the events manually, the long way, something like :

$questionItems[0].on('dragstart', startDrag); 
$questionItems[0].on('dragend', removeDropSpaces) 
$questionItems[0].on('dragenter', toggleDropStyles) 
$questionItems[0].on('dragleave', toggleDropStyles); 
$questionItems[1].on('dragstart', startDrag); 
$questionItems[1].on('dragend', removeDropSpaces) 
$questionItems[1].on('dragenter', toggleDropStyles)     
$questionItems[1].on('dragleave', toggleDropStyles); 

etc. just to see the effect?

5
  • I really need to be able to do more than sort the list, such as when an item is dragged on top of another item they become grouped.
    – Bill
    Jan 7, 2013 at 22:35
  • you still can, you just do it through jQuery for events. $( ".selector" ).on( "drag", function( event, ui ) {} ); and $( ".selector" ).on( "dragstart", function( event, ui ) {} ); And it has callbacks so you can still use the functions you already built. Jan 7, 2013 at 22:42
  • Also, have you tried to add each .on() to each single box item manually? Just to see the effect? Jan 7, 2013 at 22:47
  • Something like $questionItems[0].on('dragstart', startDrag); $questionItems[0].on('dragend', removeDropSpaces) $questionItems[0].on('dragenter', toggleDropStyles) $questionItems[0].on('dragleave', toggleDropStyles); $questionItems[1].on('dragstart', startDrag); $questionItems[1].on('dragend', removeDropSpaces) $questionItems[1].on('dragenter', toggleDropStyles) $questionItems[1].on('dragleave', toggleDropStyles); etc. just to see the effect? Jan 7, 2013 at 22:49
  • Yeah I'm going to give jQ UI's sortable a go. At this point I just want it to work. Good to know I can tap into the events too. Thanks.
    – Bill
    Jan 8, 2013 at 15:40
0

You need to prevent the "event" from bubbling up so it doesn't fire up other events. See https://developer.mozilla.org/en-US/docs/DOM/event.stopPropagation

For startDrag, toggleDropStyles, removeDropSpaces functions, you'd write something like this:

function startDrag(ev) {
   ev.stopPropagation();
   ... your code here ...
}
1
-1

change this function

function addDropSpaces()
{
  $(this).after('<div class="empty-drop-target"></div>');
}

in your version you are adding that div after each of the $questionItems.

2
  • The target is to add spaces after each boxes
    – sdespont
    Jan 7, 2013 at 21:00
  • It seems related to the bug, but I do indeed want the div injected after every .question-item, not just the current one.
    – Bill
    Jan 7, 2013 at 21:29

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.