Skip to content

feat(chip): update chip xplat topic - #903

Open
IMinchev64 wants to merge 9 commits into
vnextfrom
iminchev/update-chip-topic
Open

IMinchev64 wants to merge 9 commits into
vnextfrom
iminchev/update-chip-topic

Conversation

@IMinchev64

@IMinchev64 IMinchev64 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Closes #701

React samples: IgniteUI/igniteui-react-examples#1118
WC samples: IgniteUI/igniteui-wc-examples#1294
Blazor samples: IgniteUI/igniteui-blazor-examples#1321
Angular samples: IgniteUI/igniteui-angular-samples#4050

Checklist:

  • check topic's TOC/menu and paragraph headings
  • Include TOC topic labels in the topic content when it has a valuable update, is new, or is considered preview / beta
  • link to other topics using ./page.mdx or ../relative/path.mdx (.mdx extension required)
  • at the References section at the end of the topic add links to topics, samples, etc
  • reference API documentation instead of adding a section with API

  • use valid component names - [Data] Grid, IgxSelectComponent, <igx-combo>
  • use spell checker tool (VS Code, Grammarly, Microsoft Editor)
  • add inline code blocks for the names of classes / tags / properties
  • add language descriptor for the code blocks
  • check if links function by running the check-api/mdx npm commands
  • check if sample is working and fully visible in the topic
  • check if sample is working and fully visible in the StackBlitz
  • check if code blocks match the code in StackBlitz demo
  • add or update meaningful llms.description metadata and run npm run check:llms-metadata


  • do not resolve requested changes (leave that to the reviewer)

IMinchev64 and others added 2 commits October 7, 2026 17:30
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@IMinchev64 IMinchev64 added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. ❌ status: awaiting-test PRs awaiting manual verification and removed status: awaiting-sample topic is missing a sample ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. labels Oct 8, 2026
@DiLazarova

Copy link
Copy Markdown
Contributor

Test and Review for Chip - React / Chrome

1. Live Demo

Please use the Live Demo design from Figma File : Figma Design

Screenshot 2026-10-12 at 14 17 15

2. Layout

Please, use for Indigo theme the Indigo Arrow Icon - indigo-arrow-upward. This is relevant for ALL EXAMPLES with indigo theme for both light and dark mode.
Screenshot 2026-10-12 at 16 06 44

3. Outlined Chip

Please use the outline variants for this example
Screenshot 2026-10-12 at 16 14 18

4. Size

Please double check the Chip background color for Material theme - Light Mode. Should be #E0E0E0
Screenshot 2026-10-12 at 16 32 53

5. Section "Remove Chip" to be "Read only" with no interactions

No need for "restore chip" button
Screenshot 2026-10-12 at 17 35 15

Screenshot 2026-10-12 at 17 35 23

<PlatformBlock for="WebComponents">

- The `outlined` property is available in {PackageWebComponents} 7.3.0 and later.
- The `action` and `remove` parts are available in 7.4.0 and later. In earlier versions, the `base` part is the selection button and the remove icon renders inside it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The action and remove parts and the documented start/end slots shipped in 7.3.1, not 7.4.0. See the 7.3.1 CHANGELOG entry (#2362), or compare components/chip/chip.js and chip.d.ts in the 7.3.0 and 7.3.1 packages. Also, 7.3.0 doesn't mark start/end as deprecated. It doesn't list them at all, and the deprecation note comes from a much older release.

<div class="container sample vertical">
<IgbChip>Chip</IgbChip>
</div>
Use the `IgniteUI.Blazor` package. In `IgniteUI.Blazor` 26.1, `IgbChip` doesn't expose the `Outlined`, `Locale`, or `ResourceStrings` properties.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a good place to say that Blazor 26.1 ships the older chip. Besides the missing Outlined, Locale, and ResourceStrings: base is the selection button, there are no action/remove parts, and the focus styles read --ig-chip-focus-outline-color / --ig-chip-focus-selected-outline-color. You can check it in staticwebassets/igniteui-webcomponents.*.bundle.js of the IgniteUI.Blazor 26.1.98 NuGet package.


</PlatformBlock>

Leave <ApiLink pkg="core" type="Chip" member="variant" label="Variant" /> unset for the default type, or set it to <PlatformBlock for="WebComponents, React, Angular">`primary`</PlatformBlock><PlatformBlock for="Blazor">`StyleVariant.Primary`</PlatformBlock> for the primary type. The variant also accepts <PlatformBlock for="WebComponents, React, Angular">`info`, `success`, `warning`, and `danger`</PlatformBlock><PlatformBlock for="Blazor">`StyleVariant.Info`, `StyleVariant.Success`, `StyleVariant.Warning`, and `StyleVariant.Danger`</PlatformBlock> for status colors.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: ApiLink labels in shared prose render as-is on every platform, so WC/React/Angular readers see Variant (also Outlined L388, Disabled L472, Selectable/Selected L522, Removable L612) while the property is variant. The tables already use the right casing per platform.


Keyboard users reach each chip with <kbd>Tab</kbd>, followed by its remove control when the chip is removable. Disabled chips are skipped.

<PlatformBlock for="WebComponents, React, Blazor">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the chip that Blazor bundles, the remove key binding falls back to the host when the chip isn't removable. So on a selectable chip without Removable, Space doesn't toggle the selection, and Enter toggles it and also raises Remove. 7.3.1 fixed this ("Space did not toggle the selection, and igcRemove fired on chips that are not removable"). For Blazor, this table and the 2.1.1 row (L1173) hold only when the chip is also removable.


The {Platform} Chip exposes its state through native semantics and ARIA attributes, and takes its accessible name from its content.

<PlatformBlock for="WebComponents, React, Blazor">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not true for Blazor yet. The default select icon has aria-label="select chip", so it isn't hidden. The remove icon renders inside the selection button, so it's a nested control, not a separate one. Because of that, the accessible name includes the icon labels ("Chip remove chip"), which 7.3.1 fixed. Suggest WC/React-only bullets here, plus Blazor bullets or a Blazor known exception.


<PlatformBlock for="WebComponents, React, Blazor">

### Why did my `::part(base)` styles change after an upgrade?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

::part(base): change 7.4.0 to 7.3.1. Neither question applies to Blazor yet. Blazor still has the old part structure and still uses the outline variables, so the focus question (L1246) tells Blazor readers to rename to variables that don't exist. Suggest for="WebComponents, React" for both questions.

@Hristo313 Hristo313 added 💥 status: in-test PRs currently being tested and removed ❌ status: awaiting-test PRs awaiting manual verification labels Oct 19, 2026
@Hristo313 Hristo313 added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Oct 21, 2026
@ChronosSF
ChronosSF self-requested a review October 21, 2026 16:13

@ChronosSF ChronosSF left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is sample-related but writing it here because it spans all platforms.

Can we have the remove button on the Remove Chip section actually do something. I clicked on it and was surprised it doesn't do anything. I know removing a chip is an application scenario but it doesn't mean we can't implement it for this sample.

Have it remove the chip and add a button in its place that lets you restore it or something simple liek that. You can also just restore it after 3 seconds with a toast counting down or something like that, doesn't have to be super complicated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chip doc-redesign status: ✔️ by-design ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

inputs/chip.mdx

4 participants