Skip to content

Commit 54b64a6

Browse files
authored
Merge branch '16.1.x' into ibarakov/feat-13524-master
2 parents d75c9ea + d33f5c0 commit 54b64a6

File tree

21 files changed

+105
-196
lines changed

21 files changed

+105
-196
lines changed

.eslintignore

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
dist/
2+
projects/igniteui-angular/migrations/
3+
projects/igniteui-angular/schematics/
4+
projects/igniteui-angular/cypress/
5+
cypress/
6+
cypress.config.ts

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@
22

33
All notable changes for each version of this project will be documented in this file.
44

5+
## 16.1.5
6+
### General
7+
- `IgxButtonGroup`:
8+
- Reverted cancellable on `selected` and `deselected` events (added in 15.1.24) as it was breaking firing order and related handling.
9+
510
## 16.1.4
611
### New Features
712
- `Themes`:

angular.json

Lines changed: 0 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -163,54 +163,6 @@
163163
}
164164
}
165165
},
166-
"igniteui-dev-demos-e2e": {
167-
"root": "e2e/",
168-
"projectType": "application",
169-
"architect": {
170-
"e2e": {
171-
"builder": "@cypress/schematic:cypress",
172-
"options": {
173-
"devServerTarget": "igniteui-dev-demos-e2e:serve",
174-
"watch": true,
175-
"headless": false
176-
},
177-
"configurations": {
178-
"production": {
179-
"devServerTarget": "igniteui-dev-demos-e2e:serve:production"
180-
}
181-
}
182-
},
183-
"lint": {
184-
"builder": "@angular-eslint/builder:lint",
185-
"options": {
186-
"lintFilePatterns": [
187-
"e2e/**/*.ts",
188-
"e2e/**/*.html"
189-
]
190-
}
191-
},
192-
"cypress-run": {
193-
"builder": "@cypress/schematic:cypress",
194-
"options": {
195-
"devServerTarget": "igniteui-dev-demos-e2e:serve",
196-
"configFile": "e2e//cypress.json"
197-
},
198-
"configurations": {
199-
"production": {
200-
"devServerTarget": "igniteui-dev-demos-e2e:serve:production"
201-
}
202-
}
203-
},
204-
"cypress-open": {
205-
"builder": "@cypress/schematic:cypress",
206-
"options": {
207-
"watch": true,
208-
"headless": false,
209-
"configFile": "e2e//cypress.json"
210-
}
211-
}
212-
}
213-
},
214166
"igniteui-angular": {
215167
"root": "projects/igniteui-angular",
216168
"sourceRoot": "projects/igniteui-angular/src",

e2e/cypress.json

Lines changed: 0 additions & 9 deletions
This file was deleted.

e2e/cypress/integration/spec.ts

Lines changed: 0 additions & 7 deletions
This file was deleted.

e2e/cypress/plugins/index.ts

Lines changed: 0 additions & 3 deletions
This file was deleted.

e2e/cypress/support/commands.ts

Lines changed: 0 additions & 43 deletions
This file was deleted.

e2e/cypress/support/index.ts

Lines changed: 0 additions & 17 deletions
This file was deleted.

e2e/cypress/tsconfig.json

Lines changed: 0 additions & 8 deletions
This file was deleted.

projects/igniteui-angular/src/lib/buttonGroup/buttonGroup.component.ts

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import { IgxRippleDirective } from '../directives/ripple/ripple.directive';
2323

2424
import { takeUntil } from 'rxjs/operators';
2525
import { DisplayDensityBase, DisplayDensityToken, IDisplayDensityOptions } from '../core/density';
26-
import { CancelableEventArgs, IBaseEventArgs } from '../core/utils';
26+
import { IBaseEventArgs } from '../core/utils';
2727
import { mkenum } from '../core/utils';
2828
import { IgxIconComponent } from '../icon/icon.component';
2929

@@ -481,33 +481,29 @@ export class IgxButtonGroupComponent extends DisplayDensityBase implements After
481481
*/
482482
public _clickHandler(index: number) {
483483
const button = this.buttons[index];
484-
const args: IButtonGroupEventArgs = { cancel: false, owner: this, button, index };
484+
const args: IButtonGroupEventArgs = { owner: this, button, index };
485485

486486
if (this.selectionMode !== 'multi') {
487487
this.buttons.forEach((b, i) => {
488488
if (i !== index && this.selectedIndexes.indexOf(i) !== -1) {
489-
this.deselected.emit({ cancel: false, owner: this, button: b, index: i });
489+
this.deselected.emit({ owner: this, button: b, index: i });
490490
}
491491
});
492492
}
493493

494494
if (this.selectedIndexes.indexOf(index) === -1) {
495+
this.selectButton(index);
495496
this.selected.emit(args);
496-
if(!args.cancel){
497-
this.selectButton(index);
498-
}
499497
} else {
500498
if (this.selectionMode !== 'singleRequired') {
499+
this.deselectButton(index);
501500
this.deselected.emit(args);
502-
if(!args.cancel){
503-
this.deselectButton(index);
504-
}
505501
}
506502
}
507503
}
508504
}
509505

510-
export interface IButtonGroupEventArgs extends IBaseEventArgs, CancelableEventArgs {
506+
export interface IButtonGroupEventArgs extends IBaseEventArgs {
511507
owner: IgxButtonGroupComponent;
512508
button: IgxButtonDirective;
513509
index: number;

0 commit comments

Comments
 (0)