Skip to content

Add Angular support for nimble-chip component#2822

Merged
rajsite merged 7 commits into
mainfrom
users/vivin/add-directive-for-nimble-chip
Jan 23, 2026
Merged

Add Angular support for nimble-chip component#2822
rajsite merged 7 commits into
mainfrom
users/vivin/add-directive-for-nimble-chip

Conversation

@vivinkrishna-ni

Copy link
Copy Markdown
Contributor

Pull Request

🤨 Rationale

There is a requirement to use a chip‑like UI element in the SLE, which is an Angular application. Therefore, add an Angular wrapper for the nimble-chip component as part of #413.

See the nimble-chip component spec for details.

👩‍💻 Implementation

  • Add and export a new Angular directive and module for the nimble-chip component
  • Update the component status table to "ready" for Angular support

🧪 Testing

  • Added unit tests for the directive.
  • Manually tested in the example Angular app
    image

✅ Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed.

@vivinkrishna-ni vivinkrishna-ni marked this pull request as ready for review January 22, 2026 14:21
Comment thread packages/angular-workspace/nimble-angular/chip/nimble-chip.directive.ts Outdated
Comment thread packages/angular-workspace/nimble-angular/chip/testing/public-api.ts Outdated
@rajsite

rajsite commented Jan 23, 2026

Copy link
Copy Markdown
Member

lgtm! bypassing @mollykreis for straightforward change

@rajsite rajsite merged commit fed6c36 into main Jan 23, 2026
9 checks passed
@rajsite rajsite deleted the users/vivin/add-directive-for-nimble-chip branch January 23, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants