Skip to content

fix(skills): removed misleading listing of bar as category chart type#17261

Open
ivanvpetrov wants to merge 4 commits into
masterfrom
ipetrov/bar-chart-skill-clarification
Open

fix(skills): removed misleading listing of bar as category chart type#17261
ivanvpetrov wants to merge 4 commits into
masterfrom
ipetrov/bar-chart-skill-clarification

Conversation

@ivanvpetrov
Copy link
Copy Markdown
Contributor

@ivanvpetrov ivanvpetrov commented May 7, 2026

Description

The IgxCategoryChartComponent sections in the charts.md skills incorrectly listed "Bar" among the supported chart types.
This leads to falsely generating category charts like this:

<igx-category-chart chartType="Bar"
    ...
</igx-category-chart>

Changes

  • Updated the Related note under Column Chart to clarify that the horizontal bar equivalent requires switching to IgxDataChartComponent with IgxBarSeriesComponent
  • Removed "Bar" from the IgxCategoryChartComponent API section header; replaced with accurate types: Area, Column, Line, Point, Spline, Step, Waterfall

Copilot AI review requested due to automatic review settings May 7, 2026 11:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the charts skills reference to stop implying that IgxCategoryChartComponent supports “Bar” charts, and clarifies that horizontal bar charts require using IgxDataChartComponent with IgxBarSeriesComponent.

Changes:

  • Updated the main “Category Chart” description/usage guidance to remove “Bar” and list supported category chart types.
  • Clarified the Column Chart “Related” note to direct users to IgxDataChartComponent + IgxBarSeriesComponent for horizontal bars.
  • Updated the Category Chart API and data requirements headings to remove “Bar”.

Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Comment thread skills/igniteui-angular-components/references/charts.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants