fix: remove selectstart event handler (#63)

* fix: remove selectstart event handler * fix: less hacky way for legacy IEs
jasonslyvia · Aug 9, 2016 · 9e57941 · 9e57941
1 parent c867850
commit 9e57941
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/SortableItemMixin.js b/src/SortableItemMixin.js
@@ -24,13 +24,16 @@ function handleSortableItemReadyToMove(e) {
 function handleComponentDidMount() {
   const node = ReactDOM.findDOMNode(this);
 
-  on(node, 'selectstart', (e) => {
-    if (e.preventDefault) {
-      e.preventDefault();
-    } else {
-      e.returnValue = false;
-    }
-  });
+  // Prevent odd behaviour in legacy IE when dragging
+  if (window.attachEvent && !document.body.style['-ms-user-select']) {
+    on(node, 'selectstart', (e) => {
+      if (e.preventDefault) {
+        e.preventDefault();
+      } else {
+        e.returnValue = false;
+      }
+    });
+  }
 
   if (isFunction(this.props.onSortableItemMount)) {
     this.props.onSortableItemMount(position(node),