fix(stark-ui): table - Multisort - fix order issue when adding/removi…

…ng sort criteria ISSUES CLOSED: #1478
NationalBankBelgium · Jan 14, 2020 · d113d05 · d113d05
1 parent b157a68
commit d113d05
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/stark-ui/src/modules/table/components/column.component.html b/packages/stark-ui/src/modules/table/components/column.component.html
@@ -10,7 +10,7 @@
 					<mat-icon *ngSwitchCase="'desc'" svgIcon="arrow-down" class="stark-small-icon"></mat-icon>
 					<mat-icon class="order-tip stark-small-icon" *ngSwitchDefault svgIcon="arrow-up"></mat-icon>
 				</ng-container>
-				<span *ngIf="sortPriority < 1000 && sortDirection" class="priority">{{ sortPriority }}</span>
+				<span *ngIf="sortPriority < 100 && sortDirection" class="priority">{{ sortPriority }}</span>
 			</div>
 			<button *ngIf="filterable" class="button-filter" [matMenuTriggerFor]="filterMenu" mat-icon-button>
 				<mat-icon class="stark-small-icon" svgIcon="filter" [matTooltip]="'STARK.TABLE.COLUMN_FILTER' | translate"></mat-icon>

diff --git a/packages/stark-ui/src/modules/table/components/column.component.ts b/packages/stark-ui/src/modules/table/components/column.component.ts
@@ -158,7 +158,7 @@ export class StarkTableColumnComponent extends AbstractStarkUiComponent implemen
 	 * Priority of the column.
 	 */
 	@Input()
-	public sortPriority = 0;
+	public sortPriority = 100;
 
 	/**
 	 * Whether the column is visible or not. Default: true
@@ -253,6 +253,10 @@ export class StarkTableColumnComponent extends AbstractStarkUiComponent implemen
 		) {
 			this._filterFormCtrl.setValue(this.filterValue);
 		}
+
+		if (simpleChanges["sortPriority"] && typeof simpleChanges["sortPriority"].currentValue === "undefined") {
+			this.sortPriority = 100;
+		}
 	}
 
 	/**