fixed leading tabs when copying multiple selections (#1562)

* fixed leading tabs when copying multiple selections

* add tabs in the end as well

* remove space from tab

src/controllers/queryRunner.ts

@@ -419,6 +419,9 @@ export default class QueryRunner {
 
         // Go through all rows and get selections for them
         let allRowIds = rowIdToRowMap.keys();
+        const endColumns = this.getSelectionEndColumns(rowIdToRowMap, rowIdToSelectionMap);
+        const firstColumn = endColumns[0];
+        const lastColumn = endColumns[1];
         for (let rowId of allRowIds) {
             let row = rowIdToRowMap.get(rowId);
             const rowSelections = rowIdToSelectionMap.get(rowId);
@@ -428,20 +431,25 @@ export default class QueryRunner {
                 return ((a.fromCell < b.fromCell) ? -1 : (a.fromCell > b.fromCell) ? 1 : 0);
             });
 
-            // start copy paste from left-most column
-            const firstColumn = rowSelections[0].fromCell;
-
             for (let i = 0; i < rowSelections.length; i++) {
                 let rowSelection = rowSelections[i];
+
+                // Add tabs starting from the first column of the selection
                 for (let j = firstColumn; j < rowSelection.fromCell; j++) {
-                    copyString += ' \t';
+                    copyString += '\t';
                 }
                 let cellObjects = row.slice(rowSelection.fromCell, (rowSelection.toCell + 1));
+
                 // Remove newlines if requested
                 let cells = this.shouldRemoveNewLines()
                 ? cellObjects.map(x => this.removeNewLines(x.displayValue))
                 : cellObjects.map(x => x.displayValue);
                 copyString += cells.join('\t');
+
+                // Add tabs until the end column of the selection
+                for (let k = rowSelection.toCell; k < lastColumn; k++) {
+                    copyString += '\t';
+                }
             }
             copyString += os.EOL;
         }
@@ -518,6 +526,27 @@ export default class QueryRunner {
         }
     }
 
+    /**
+     * Gets the first and last column of a selection: [first, last]
+     */
+    private getSelectionEndColumns(rowIdToRowMap: Map<number, DbCellValue[]>, rowIdToSelectionMap: Map<number, ISlickRange[]>): number[] {
+        let allRowIds = rowIdToRowMap.keys();
+        let firstColumn = -1;
+        let lastColumn = -1;
+        for (let rowId of allRowIds) {
+            const rowSelections = rowIdToSelectionMap.get(rowId);
+            for (let i = 0; i < rowSelections.length; i++) {
+                if (firstColumn === -1 || rowSelections[i].fromCell < firstColumn) {
+                    firstColumn = rowSelections[i].fromCell;
+                }
+                if (lastColumn === -1 || rowSelections[i].toCell > lastColumn) {
+                    lastColumn = rowSelections[i].toCell;
+                }
+            }
+        }
+        return [firstColumn, lastColumn];
+    }
+
     /**
      * Sets a selection range in the editor for this query
      * @param selection The selection range to select