Merge branch 'master' into pmoleri/memory-leak-fixing

Author: ChronosSF

CHANGELOG.md

@@ -1,6 +1,11 @@
 # Ignite UI for Angular Change Log
 
 All notable changes for each version of this project will be documented in this file.
+## 19.1.1
+### New Features
+- IgxListItem
+  - Added a new `selected` input property, making it easier to indicate when a list item is selected by applying styling responsible for that state.
+
 ## 19.1.0
 ### General
 - `IgxCarousel`

package-lock.json

@@ -74,7 +74,7 @@
     "@types/source-map": "0.5.2",
     "express": "^4.21.1",
     "fflate": "^0.8.1",
-    "igniteui-theming": "^15.0.0",
+    "igniteui-theming": "^15.1.1",
     "igniteui-trial-watermark": "^3.0.2",
     "lodash-es": "^4.17.21",
     "rxjs": "^7.8.0",

package.json

@@ -73,7 +73,7 @@
         "tslib": "^2.3.0",
         "igniteui-trial-watermark": "^3.0.2",
         "lodash-es": "^4.17.21",
-        "igniteui-theming": "^15.0.0",
+        "igniteui-theming": "^15.1.1",
         "@igniteui/material-icons-extended": "^3.1.0"
     },
     "peerDependencies": {

projects/igniteui-angular/package.json

@@ -390,7 +390,7 @@ describe('igxGridEditingActions #grid ', () => {
     template: `
     <igx-grid #grid [data]="data" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>
@@ -469,7 +469,7 @@ class IgxActionStripTestingComponent implements OnInit {
     template: `
     <igx-grid #grid [data]="data" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>
@@ -491,7 +491,7 @@ class IgxActionStripPinEditComponent extends IgxActionStripTestingComponent {
     template: `
     <igx-grid #grid [data]="data" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>
@@ -512,7 +512,7 @@ class IgxActionStripEditMenuComponent extends IgxActionStripTestingComponent {
     template: `
     <igx-grid #grid [data]="dataOneRow" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>
@@ -534,7 +534,7 @@ class IgxActionStripOneRowComponent extends IgxActionStripTestingComponent {
     template: `
     <igx-grid #grid [data]="dataOneRow" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>

projects/igniteui-angular/src/lib/action-strip/grid-actions/grid-editing-actions.component.spec.ts

@@ -102,7 +102,7 @@ describe('igxGridPinningActions #grid ', () => {
     template: `
     <igx-grid #grid [data]="data" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>
@@ -176,7 +176,7 @@ class IgxActionStripTestingComponent implements OnInit {
     template: `
     <igx-grid #grid [data]="data" [width]="'800px'" [height]="'500px'"
         [rowEditable]="true" [primaryKey]="'ID'">
-        @for (c of columns; track c) {
+        @for (c of columns; track c.field) {
             <igx-column [sortable]="true" [field]="c.field" [header]="c.field"
                 [width]="c.width" [pinned]='c.pinned' [hidden]='c.hidden'>
             </igx-column>

projects/igniteui-angular/src/lib/action-strip/grid-actions/grid-pinning-actions.component.spec.ts

@@ -584,7 +584,7 @@ class ButtonGroupWithSelectedButtonComponent {
 @Component({
     template: `
     <igx-buttongroup>
-        @for (item of items; track item) {
+        @for (item of items; track item.key) {
             <button igxButton [selected]="item.key === selectedValue">{{item.value}}</button>
         }
     </igx-buttongroup>

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

@@ -15,7 +15,7 @@ import { getComponentSize } from '../core/utils';
 @Component({
     template: `
         <igx-chips-area #chipsArea>
-            @for (chip of chipList; track chip) {
+            @for (chip of chipList; track chip.id) {
                 <igx-chip #chipElem class="custom"
                     [id]="chip.id" [draggable]="chip.draggable"
                     [removable]="chip.removable" [selectable]="chip.selectable"
@@ -69,7 +69,7 @@ class TestChipComponent {
 @Component({
     template: `
         <igx-chips-area>
-            @for (chip of chipList; track chip) {
+            @for (chip of chipList; track chip.id) {
                 <igx-chip>
                     <span igxLabel>label</span>
                     <span igxSuffix>suf</span>

projects/igniteui-angular/src/lib/chips/chip.spec.ts

@@ -11,7 +11,7 @@ import { IgxPrefixDirective } from './public_api';
 @Component({
     template: `
         <igx-chips-area #chipsArea class="customClass">
-            @for (chip of chipList; track chip) {
+            @for (chip of chipList; track chip.id) {
                 <igx-chip #chipElem
                     [id]="chip.id" [draggable]="chip.draggable" [removable]="chip.removable" [selectable]="chip.selectable">
                     <igx-icon igxPrefix>drag_indicator</igx-icon>
@@ -59,7 +59,7 @@ class TestChipSelectComponent extends TestChipComponent {
 @Component({
     template: `
         <igx-chips-area #chipsArea (reorder)="chipsOrderChanged($event)">
-            @for (chip of chipList; track chip) {
+            @for (chip of chipList; track chip.id) {
                 <igx-chip #chipElem [id]="chip.id" [draggable]="true"
                     [removable]="true" [selectable]="true" (remove)="chipRemoved($event)">
                     <igx-icon igxPrefix>drag_indicator</igx-icon>

projects/igniteui-angular/src/lib/chips/chips-area.spec.ts

@@ -261,7 +261,6 @@
         }
     }
 
-    %form-group-bundle:not(%form-group-bundle--disabled):hover,
     %form-group-bundle:not(%form-group-bundle--disabled):focus-within {
         %igx-combo__toggle-button {
             color: var-get($theme, 'toggle-button-foreground-focus');
@@ -274,6 +273,14 @@
         }
     }
 
+    @if $variant == 'indigo' {
+        %form-group-bundle:not(%form-group-bundle--disabled):hover {
+            %igx-combo__toggle-button {
+                color: var-get($theme, 'toggle-button-foreground-focus');
+            }
+        }
+    }
+
     .igx-input-group--filled {
         %igx-combo__toggle-button {
             color: var-get($theme, 'toggle-button-foreground-filled');
@@ -293,8 +300,10 @@
     }
 
     .igx-input-group.igx-input-group--focused:not(.igx-input-group--box) {
-        %igx-combo__toggle-button {
-            background: var-get($theme, 'toggle-button-background-focus--border');
+        @if $variant != 'fluent' {
+            %igx-combo__toggle-button {
+                background: var-get($theme, 'toggle-button-background-focus--border');
+            }
         }
     }