Fix list reorder for new list header

azavea · Oct 10, 2019 · cead4ab · cead4ab
1 parent 4c9e0fa
commit cead4ab
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/app/scripts/cac/control/cac-control-tour-list.js b/src/app/scripts/cac/control/cac-control-tour-list.js
@@ -82,7 +82,7 @@ CAC.Control.TourList = (function (_, $, MapTemplates) {
             var $sortableList = $destinationList.sortable({
                 animation: 150,
                 direction: 'vertical',
-                draggable: '.tour-place-card',
+                draggable: options.selectors.destinationItem,
                 // Allow scrolling while dragging by not using native HTML5
                 // See: https://github.com/SortableJS/Sortable/issues/935
                 forceFallback: true,
@@ -95,7 +95,8 @@ CAC.Control.TourList = (function (_, $, MapTemplates) {
     // Called when Sortable list of destinations gets updated
     function onDestinationListReordered(e) {
         var kids = e.to.children;
-        for (var i = 0; i < kids.length; i++) {
+        // First list item is the header; skip it
+        for (var i = 1; i < kids.length; i++) {
             var k = kids[i];
             var originalIndex = k.getAttribute('data-tour-place-index');
             // assign property with new order