Conversation
mtsvyatkova
added
20.2.x
❌ status: awaiting-test
There was a problem hiding this comment.
Pull request overview
Updates the igniteui-angular-components skill documentation to include Tile Manager alongside the existing Layout Manager and Dock Manager references, addressing the missing skill coverage noted in #17046.
Changes:
- Extend the layout manager reference doc to add a new “Tile Manager” section with installation/setup/usage guidance.
- Update the skill hub description and task → reference mapping to mention Tile Manager and route users to the correct reference file.
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 | Adds Tile Manager documentation and links within the existing Layout/Dock Manager reference. |
| skills/igniteui-angular-components/SKILL.md | Updates skill metadata and routing table to include Tile Manager. |
| import { defineComponents, IgcTileManagerComponent } from 'igniteui-webcomponents'; | ||
|
|
||
| defineComponents(IgcTileManagerComponent); |
There was a problem hiding this comment.
<igc-tile-manager> usage in the examples includes <igc-tile> children, but the setup only registers IgcTileManagerComponent. In this repo’s web components samples, child elements are registered explicitly (e.g. tabs registers both IgcTabsComponent and IgcTabComponent), so this likely leaves <igc-tile> undefined at runtime. Import and register the tile component as well, and update the “Key Rules” item to match.
| import { defineComponents, IgcTileManagerComponent } from 'igniteui-webcomponents'; | |
| defineComponents(IgcTileManagerComponent); | |
| import { defineComponents, IgcTileManagerComponent, IgcTileComponent } from 'igniteui-webcomponents'; | |
| defineComponents(IgcTileManagerComponent, IgcTileComponent); |
| ### Installation | ||
|
|
||
| ```bash | ||
| npm install igniteui-webcomponents --save |
There was a problem hiding this comment.
The install snippet uses npm install igniteui-webcomponents --save. --save is redundant with modern npm and is inconsistent with the other install snippets in this skill hub (e.g. Dock Manager uses npm install igniteui-dockmanager). Consider removing --save for consistency.
| npm install igniteui-webcomponents --save | |
| npm install igniteui-webcomponents |
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)