jquery-ui icon indicating copy to clipboard operation
jquery-ui copied to clipboard

Cursor is not properly reset with draggable and connectToSortable

Open Eydamos opened this issue 2 years ago • 4 comments

If both the sortable and draggable have the cursor option set the cursor stays at the value from the draggable after the draggable was dropped.

Example to reproduce: https://jsfiddle.net/5fxqybrs/1/ Moving the sortable elements works fine and sets the cursor to move and resets it back to auto. But moving the draggable into the sortable keeps the cursor as move and it will never go back to auto again.

The following happens:

  • draggable start event is triggered: saves body value auto and sets cursor to move on body
  • sortable start event is triggered: saves body value of move and sets cursor to move on body
  • draggable stop event is triggered: setting cursor back to stored value of auto
  • sortable stop event is triggered: setting cursor back to stored value of move

A workaround I have implemented for now is setting the stop option of the sortable to a function which sets the cursor to auto:

    $( "#sortable" ).sortable({
      revert: true,
      cursor: "move",
      stop: function() {
          $('body').css('cursor', 'auto');
      }
    });

Eydamos avatar Feb 15 '23 12:02 Eydamos

it does not work for touch screens as well

ghost avatar Feb 18 '23 11:02 ghost

Thanks for the report. Does the issue you describe exist when jQuery UI 1.12.1 is used or only with jQuery UI 1.13.0 or newer?

mgol avatar Feb 20 '23 23:02 mgol

I just tried it in another JSFiddle with jQuery UI 1.12.1 and the problem also occurs with that version

Eydamos avatar Feb 22 '23 08:02 Eydamos

Thanks for the report. Since the issue is already in 1.12, given limited team resources it's not likely to be fixed by the UI team; see the project status at https://blog.jqueryui.com/2021/10/jquery-maintainers-update-and-transition-jquery-ui-as-part-of-overall-modernization-efforts/. PRs are welcome if they're not too complex.

mgol avatar Feb 22 '23 11:02 mgol