From 95d325fa07d83ff59e78a72470ff0f3b01ce0948 Mon Sep 17 00:00:00 2001
From: Riley Raschke <riley@rrappsdev.com>
Date: Thu, 20 Feb 2020 19:54:45 -0600
Subject: [PATCH] safari 13 has broken drag drop after first event. confirmed
 at https://github.com/SortableJS/Sortable/issues/1571

---
 src/directives.js | 35 ++++++++++++++++++++---------------
 1 file changed, 20 insertions(+), 15 deletions(-)

diff --git a/src/directives.js b/src/directives.js
index 241fce0..ae89580 100644
--- a/src/directives.js
+++ b/src/directives.js
@@ -60,27 +60,32 @@ app.directive('playerSettingsDialog', [function() {
         $(modalElement).modal(newValue ? 'show' : 'hide');
       });
       element.on('shown.bs.modal', function() {
+
         scope.$apply(function() {
           scope.model.visible = true;
+        });
 
-          // TODO: Move to it's own directive?
-          var playerOrder = document.getElementById('playerOrderList');
-          var sortablePlayers = Sortable.create(playerOrder, {
-            onEnd: function( orderEvt ){
-              if( orderEvt.newIndex == orderEvt.oldIndex ){
-                return;
-              }
-              scope.model.bjg.needsReseat = true;
-
-              tmpSeat = scope.model.bjg.game.players[orderEvt.newIndex];
-              scope.model.bjg.game.players[orderEvt.newIndex] = scope.model.bjg.game.players[orderEvt.oldIndex];
-              scope.model.bjg.game.players[orderEvt.oldIndex] = tmpSeat;
-
-              scope.model.bjg.reseat();
+        // TODO: Move to it's own directive?
+        // NOTE: iOS broken per: https://github.com/SortableJS/Sortable/issues/1571
+        //       workaround `forceFallback: true,` didn't work either... but did break firefox too..
+        var playerOrder = document.getElementById('playerOrderList');
+        var sortablePlayers = Sortable.create(playerOrder, {
+        //var sortablePlayers = new Sortable(playerOrder, {
+          //forceFallback: true,
+          onEnd: function( orderEvt ){
+            if( orderEvt.newIndex == orderEvt.oldIndex ){
+              return;
             }
-          });
+            scope.model.bjg.needsReseat = true;
+
+            tmpSeat = scope.model.bjg.game.players[orderEvt.newIndex];
+            scope.model.bjg.game.players[orderEvt.newIndex] = scope.model.bjg.game.players[orderEvt.oldIndex];
+            scope.model.bjg.game.players[orderEvt.oldIndex] = tmpSeat;
 
+            scope.model.bjg.reseat();
+          }
         });
+
       });
       element.on('hidden.bs.modal', function() {
         scope.$apply(function() {