GridController transform fix + test

Author: dhilt

src/ui-scroll-grid.js

@@ -220,26 +220,15 @@ angular.module('ui.scroll.grid', [])
 
       function transform (row) {
         let parent = row[0].parent();
-        let last = row[row.length-1].next();
         let visible = [];
 
         row.forEach((cell, index) => {
           columns[index].applyCss(cell);
           visible[columns[index].mapTo] = row[index];
+          row[index].detach();
         });
 
-        let current = visible.shift();
-        current.detach();
-        if (last.length)
-          last.before(current);
-        else
-          parent.append(current);
-
-        visible.forEach((cell) => {
-          cell.detach();
-          current.after(cell);
-          current = cell;
-        });
+        visible.forEach(cell => parent.append(cell));
       }
 
     }

test/GridTestsSpec.js

@@ -391,8 +391,8 @@ describe('uiScroll', function () {
 					applyOrderLayout(scope, [1, 3, 2, 0]);
 					expectLastRowContents(body, ['col3', 'col0', 'col2', 'col1']);
 
-					applyOrderLayout(scope, [0, 2, 3, 1]); // this is broken !!
-					//expectLastRowContents(body, ['col0', 'col3', 'col1', 'col2']);
+					applyOrderLayout(scope, [0, 2, 3, 1]);
+					expectLastRowContents(body, ['col0', 'col3', 'col1', 'col2']);
 				}
 			);
 		});