Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

Commit 9e6b8ac

Browse files
committed
Merge pull request #1503 from HDinger/fix/drag-drop-IE
fix(uiSelectSortDirective): fix drag'n'drop in internet explorer This fixes a bug in IE where drag and drop was not working. This had two reasons: First there was 'text' expected, instead of 'text/plain'. Second the other parameter should be a String. This fix works now for all browsers.
2 parents 86fc336 + b0a4a3d commit 9e6b8ac

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/uiSelectSortDirective.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ uis.directive('uiSelectSort', ['$timeout', 'uiSelectConfig', 'uiSelectMinErr', f
3131
element.on('dragstart', function(event) {
3232
element.addClass(draggingClassName);
3333

34-
(event.dataTransfer || event.originalEvent.dataTransfer).setData('text/plain', scope.$index);
34+
(event.dataTransfer || event.originalEvent.dataTransfer).setData('text', scope.$index.toString());
3535
});
3636

3737
element.on('dragend', function() {
@@ -63,7 +63,7 @@ uis.directive('uiSelectSort', ['$timeout', 'uiSelectConfig', 'uiSelectMinErr', f
6363
var dropHandler = function(event) {
6464
event.preventDefault();
6565

66-
var droppedItemIndex = parseInt((event.dataTransfer || event.originalEvent.dataTransfer).getData('text/plain'), 10);
66+
var droppedItemIndex = parseInt((event.dataTransfer || event.originalEvent.dataTransfer).getData('text'), 10);
6767

6868
// prevent event firing multiple times in firefox
6969
$timeout.cancel(dropTimeout);

0 commit comments

Comments
 (0)