fix(*): add tile manager to skills - 21.1.x#17058
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the igniteui-angular-components skill documentation to include Tile Manager alongside existing Layout Manager and Dock Manager guidance, addressing the missing coverage reported in #17046.
Changes:
- Expand the Layout/Dock Manager reference to include a new Tile Manager section and TOC entry.
- Update the skill hub metadata/description to mention Tile Manager.
- Update the Task → Reference File routing table to include Tile Manager in the layout-manager reference row.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| skills/igniteui-angular-components/references/layout-manager.md | Renames the page header, adds TOC entry, and adds a full Tile Manager documentation section. |
| skills/igniteui-angular-components/SKILL.md | Updates skill description and routing table row to include Tile Manager. |
skills/igniteui-angular-components/references/layout-manager.md
Outdated
Show resolved
Hide resolved
skills/igniteui-angular-components/references/layout-manager.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| Register the Tile Manager Web Component before bootstrap: | ||
|
|
||
| ```typescript | ||
| import { defineComponents, IgcTileManagerComponent } from 'igniteui-webcomponents'; | ||
|
|
||
| defineComponents(IgcTileManagerComponent); |
There was a problem hiding this comment.
The Tile Manager setup snippet says to register the web component “before bootstrap”, but (unlike the Dock Manager section above) it doesn’t specify the concrete placement (e.g., main.ts before bootstrapApplication). Adding that call-site guidance would prevent users from registering too late and getting “unknown element” behavior.
| Register the Tile Manager Web Component before bootstrap: | |
| ```typescript | |
| import { defineComponents, IgcTileManagerComponent } from 'igniteui-webcomponents'; | |
| defineComponents(IgcTileManagerComponent); | |
| In your `main.ts`, register the Tile Manager Web Component **before** calling `bootstrapApplication`: | |
| ```typescript | |
| import { bootstrapApplication } from '@angular/platform-browser'; | |
| import { AppComponent } from './app/app.component'; | |
| import { defineComponents, IgcTileManagerComponent } from 'igniteui-webcomponents'; | |
| defineComponents(IgcTileManagerComponent); | |
| bootstrapApplication(AppComponent); |
| import { Component, CUSTOM_ELEMENTS_SCHEMA } from '@angular/core'; | ||
|
|
||
| @Component({ | ||
| selector: 'app-tile-manager', | ||
| templateUrl: './tile-manager.component.html', |
There was a problem hiding this comment.
The Tile Manager component example omits changeDetection: ChangeDetectionStrategy.OnPush, while the rest of the skill references generally default to OnPush in component examples. Consider adding OnPush here as well (and the corresponding ChangeDetectionStrategy import) so the snippet reflects the recommended Angular performance setup.
| import { Component, CUSTOM_ELEMENTS_SCHEMA } from '@angular/core'; | |
| @Component({ | |
| selector: 'app-tile-manager', | |
| templateUrl: './tile-manager.component.html', | |
| import { ChangeDetectionStrategy, Component, CUSTOM_ELEMENTS_SCHEMA } from '@angular/core'; | |
| @Component({ | |
| selector: 'app-tile-manager', | |
| templateUrl: './tile-manager.component.html', | |
| changeDetection: ChangeDetectionStrategy.OnPush, |
Closes #17046
Additional information (check all that apply):
Checklist:
feature/README.MDupdates for the feature docsREADME.MDCHANGELOG.MDupdates for newly added functionalityng updatemigrations for the breaking changes (migrations guidelines)