Skip to content

Commit 51a6c56

Browse files
committed
feat(Sorting): grid sorting should not affect group by state #9674
1 parent dbe2440 commit 51a6c56

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

Diff for: projects/igniteui-angular/src/lib/grids/grid-base.directive.ts

+2-7
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ import { ConnectedPositioningStrategy } from '../services/overlay/position/conne
146146
import { ContainerPositionStrategy } from '../services/overlay/position/container-position-strategy';
147147
import { AbsoluteScrollStrategy } from '../services/overlay/scroll/absolute-scroll-strategy';
148148
import { Action, StateUpdateEvent, TransactionEventOrigin } from '../services/transaction/transaction';
149-
import { ISortingExpression, SortingDirection } from '../data-operations/sorting-strategy';
149+
import { ISortingExpression } from '../data-operations/sorting-strategy';
150150
import { IGridSortingStrategy } from './common/strategy';
151151
import { IgxGridExcelStyleFilteringComponent } from './filtering/excel-style/grid.excel-style-filtering.component';
152152
import { IgxGridHeaderComponent } from './headers/grid-header.component';
@@ -4528,15 +4528,10 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
45284528

45294529
if (expression instanceof Array) {
45304530
for (const each of expression) {
4531-
if (each.dir === SortingDirection.None) {
4532-
this.gridAPI.remove_grouping_expression(each.fieldName);
4533-
}
45344531
this.gridAPI.prepare_sorting_expression([sortingState], each);
45354532
}
45364533
} else {
4537-
if (expression.dir === SortingDirection.None) {
4538-
this.gridAPI.remove_grouping_expression(expression.fieldName);
4539-
} else if (this._sortingOptions.mode === 'single') {
4534+
if (this._sortingOptions.mode === 'single') {
45404535
this.columns.forEach((col) => {
45414536
if (!(col.field === expression.fieldName)) {
45424537
this.clearSort(col.field);

0 commit comments

Comments
 (0)