fix(skills): correct references in skills (#17294)#17298
Merged
Conversation
* fix(skills): correct references in skills * chore(*): address PR comments * fix(skills): reducing the charts skill significantly * fix(skills): add sparkline type in chart ref * fix(skills): removing online docs reference in favour of MCP and adding autocomplete --------- Co-authored-by: Konstantin Dinev <kdinev@infragistics.com>
onlyexeption
approved these changes
May 28, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR corrects API references, import paths, and event/property names across the Ignite UI Angular skills documentation (used by AI assistants). It removes many > **Docs:** online links in favor of pointing agents at the igniteui-cli MCP server, and brings examples in line with the actual source for components such as Grid Lite, sorting/filtering events, pivot grid APIs, drag/drop, tooltip, snackbar, banner, tree selection, autocomplete, and charts/sparkline.
Changes:
- Corrects API names, types, event payloads, and import paths in grid and component reference files.
- Replaces inline online docs links with guidance to call
get_doc/get_api_referencevia theigniteui-cliMCP server, and adds an MCP-style overview to the grids hub. - Restructures the charts reference (significantly trimmed) and adds Sparkline coverage; adds a new Autocomplete section in form-controls.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/igniteui-angular-grids/SKILL.md | Adds an Overview section pointing agents to igniteui-cli for docs/API. |
| skills/igniteui-angular-grids/references/types.md | Updates Grid Lite snippets (templates, header note, cancel pattern) and removes docs links. |
| skills/igniteui-angular-grids/references/structure.md | Removes docs links; updates row selector template to use [readonly]. |
| skills/igniteui-angular-grids/references/state.md | Uses notifyDimensionChange() / sortDimension() public APIs for pivot ops. |
| skills/igniteui-angular-grids/references/sizing.md | Documents [width]="null" behavior and percentage-column fallback. |
| skills/igniteui-angular-grids/references/paging-remote.md | Corrects (sortingDone) payload type to ISortingExpression[]. |
| skills/igniteui-angular-grids/references/features.md | Documents cellMergeMode, grid-level mergeStrategy, DefaultMergeStrategy extension, and row-drag flow. |
| skills/igniteui-angular-grids/references/editing.md | Replaces event.cellEditArgs.newValue with event.newValue and refines cancelable event list. |
| skills/igniteui-angular-grids/references/data-operations.md | Updates sorting/filtering event payload types, custom sort strategy signature, marks filterGlobal deprecated. |
| skills/igniteui-angular-components/SKILL.md | Adds Autocomplete to the form-controls index entry. |
| skills/igniteui-angular-components/references/setup.md | Removes provideIgniteIntl() row from providers table. |
| skills/igniteui-angular-components/references/layout.md | Removes online docs links. |
| skills/igniteui-angular-components/references/layout-manager.md | Moves defineCustomElements() to app.config.ts; trims dock manager property tables. |
| skills/igniteui-angular-components/references/form-controls.md | Adds Autocomplete section; removes invalid is24HourFormat input; adds component-choice guidance. |
| skills/igniteui-angular-components/references/feedback.md | Updates Snackbar (actionText, (clicked)) and Banner (<igx-banner-actions>, close()) to current APIs. |
| skills/igniteui-angular-components/references/directives.md | Fixes Tooltip input names, drag/drop directive syntax and drop event names. |
| skills/igniteui-angular-components/references/data-display.md | Fixes list line directive, tree selection modes, card thumbnail handling, progress bar error type. |
| skills/igniteui-angular-components/references/charts.md | Significantly trims reference; adds Sparkline component and corrects color/highlight property names. |
| @@ -44,7 +44,6 @@ export const appConfig: ApplicationConfig = { | |||
| |---|---|---| | |||
| | `provideAnimations()` | `@angular/platform-browser/animations` | **All overlay and animated components** — Dialog, Combo, Select, Dropdown, Date/Time Picker, Snackbar, Toast, Banner, Navigation Drawer, Carousel, Overlay service | | |||
|
|
||
| ```typescript | ||
| import { IgxCardComponent, IgxCardHeaderComponent, IgxCardContentDirective, IgxCardActionsComponent, IgxCardMediaDirective, IgxCardHeaderTitleDirective, IgxCardHeaderSubtitleDirective, IgxCardHeaderThumbnailDirective } from 'igniteui-angular/card'; | ||
| import { IgxCardComponent, IgxCardHeaderComponent, IgxCardContentDirective, IgxCardActionsComponent, IgxCardMediaDirective, IgxCardHeaderTitleDirective, IgxCardHeaderSubtitleDirective, IgxCardThumbnailDirective } from 'igniteui-angular/card'; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fix(skills): correct references in skills
chore(*): address PR comments
fix(skills): reducing the charts skill significantly
fix(skills): add sparkline type in chart ref
fix(skills): removing online docs reference in favour of MCP and adding autocomplete
Closes #
Description
Motivation / Context
Type of Change (check all that apply):
Component(s) / Area(s) Affected:
How Has This Been Tested?
Test Configuration:
Screenshots / Recordings
Checklist:
feature/README.MDupdates for the feature docsREADME.MDCHANGELOG.MDupdates for newly added functionalityng updatemigrations for the breaking changes (migrations guidelines)