IgniteUI / igniteui-angular / 42029516169

94%

Build Type

push

github

Committed by web-flow

Commit Message

fix(elements): fixing elements bugs (#17830)

* test(elements, extras): adding more tests for additional coverage

* fix(tree-grid): skip autogenerate when data is set before init

Setting data on a tree grid with autoGenerate before ngOnInit threw,
since the grid API service has no grid yet. Columns are generated on
init instead, matching the grid and hierarchical grid.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(elements): apply writable props, filter factory and icon actions

- Set writable additional properties (e.g. igc-column selected) on the
  instance, since they are not inputs and setInput ignored them
- Add the deprecated getFilterFactory to igc-grid, which extends the
  base grid component instead of being it
- Only process the icon broadcast message part for its action type
- Return proxies, not result wrappers, for component arrays in events

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update test to check toggle visibility on grid scroll

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* test(elements, extras): adding more tests for additional coverage

* Update test to check toggle visibility on grid scroll

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* fix(tree-grid): skip autogenerate when data is set before init

Setting data on a tree grid with autoGenerate before ngOnInit threw,
since the grid API service has no grid yet. Columns are generated on
init instead, matching the grid and hierarchical grid.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* fix(elements): apply writable props, filter factory and icon actions

- Set writable additional properties (e.g. igc-column selected) on the
  instance, since they are not inputs and setInput ignored them
- Add the deprecated getFilterFactory to igc-grid, which extends the
  base grid component instead of being it
- Only process the icon broadcast mess... (continued)

Coverage Stats

25686 of 28027 branches covered (91.65%)

Branch coverage included in aggregate %.

58 of 58 new or added lines in 4 files covered. (100.0%)

1 existing line in 1 file now uncovered.

37416 of 39007 relevant lines covered (95.92%)

30895.92 hits per line