Merge branch '15.1.x' into bpachilova/fix-13392-15.1.x

Author: teodosiah

package-lock.json

@@ -61,7 +61,7 @@
     "@angular/platform-browser": "^15.2.0",
     "@angular/platform-browser-dynamic": "^15.2.0",
     "@angular/router": "^15.2.0",
-    "@igniteui/material-icons-extended": "^2.10.0",
+    "@igniteui/material-icons-extended": "^3.0.1",
     "@types/hammerjs": "^2.0.40",
     "@types/source-map": "0.5.2",
     "fflate": "^0.7.3",

package.json

@@ -4,6 +4,7 @@
   "description": "IgniteUI for Angular localization resources package",
   "main": "./index.js",
   "types": "./index.d.ts",
+  "sideEffects": false,
   "repository": {
     "type": "git",
     "url": "https://github.com/IgniteUI/igniteui-angular.git",

projects/igniteui-angular-i18n/package.json

@@ -77,7 +77,7 @@
         "lodash-es": "^4.17.21",
         "uuid": "^8.3.0",
         "igniteui-theming": "^1.4.1",
-        "@igniteui/material-icons-extended": "^2.10.0"
+        "@igniteui/material-icons-extended": "^3.0.0"
     },
     "peerDependencies": {
         "@angular/common": "^15.0.0",

projects/igniteui-angular/package.json

@@ -45,7 +45,7 @@ class Button {
 
 describe('IgxButtonGroup', () => {
     configureTestSuite();
-   beforeAll(waitForAsync(() => {
+    beforeAll(waitForAsync(() => {
         TestBed.configureTestingModule({
             declarations: [
                 InitButtonGroupComponent,
@@ -194,8 +194,8 @@ describe('IgxButtonGroup', () => {
         UIInteractions.simulateClickEvent(buttongroup.buttons[0].nativeElement);
         UIInteractions.simulateClickEvent(buttongroup.buttons[1].nativeElement);
         expect(buttongroup.selectedButtons.length).toBe(0);
-        UIInteractions.simulateClickEvent(buttongroup.buttons[0].nativeElement);
-        UIInteractions.simulateClickEvent(buttongroup.buttons[3].nativeElement);
+        buttongroup.buttons[0].nativeElement.click();
+        buttongroup.buttons[3].nativeElement.click();
         // Button 3 is disabled, and it should not be selected with mouse click
         expect(buttongroup.selectedButtons.length).toBe(1);
     });

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

@@ -2114,7 +2114,8 @@ export class IgxColumnComponent implements AfterContentInit, OnDestroy, ColumnTy
         }
 
         if (hasIndex) {
-            grid._moveColumns(this, targetColumn);
+            index === grid._pinnedColumns.length - 1 ? 
+            grid._moveColumns(this, targetColumn, DropPosition.AfterDropTarget) : grid._moveColumns(this, targetColumn, DropPosition.BeforeDropTarget);
         }
 
         if (this.columnGroup) {

projects/igniteui-angular/src/lib/grids/columns/column.component.ts

@@ -5092,7 +5092,7 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
      * @param columnName
      * @param index
      */
-    public pinColumn(columnName: string | IgxColumnComponent, index?): boolean {
+    public pinColumn(columnName: string | IgxColumnComponent, index?: number): boolean {
         const col = columnName instanceof IgxColumnComponent ? columnName : this.getColumnByName(columnName);
         return col.pin(index);
     }
@@ -5107,7 +5107,7 @@ export abstract class IgxGridBaseDirective extends DisplayDensityBase implements
      * @param columnName
      * @param index
      */
-    public unpinColumn(columnName: string | IgxColumnComponent, index?): boolean {
+    public unpinColumn(columnName: string | IgxColumnComponent, index?: number): boolean {
         const col = columnName instanceof IgxColumnComponent ? columnName : this.getColumnByName(columnName);
         return col.unpin(index);
     }

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

@@ -188,6 +188,17 @@ describe('IgxGrid - Column Pinning #grid', () => {
 
             });
 
+            it('should pin the column on the last position if the index for the last position is provided', () => {
+                grid.pinColumn('CompanyName');
+                fix.detectChanges();
+
+                grid.pinColumn('City', 2);
+                fix.detectChanges();
+
+                expect(grid.pinnedColumns.length).toEqual(3);
+                expect(grid.pinnedColumns[2].field).toEqual('City');
+            });
+
             it('should correctly initialize pinned columns z-index values.', () => {
 
                 const headers = GridFunctions.getColumnHeaders(fix);