Skip to content

Add blazor and react wrapper for nimble chip#2925

Merged
rajsite merged 12 commits intomainfrom
users/vgleason/chips
Apr 15, 2026
Merged

Add blazor and react wrapper for nimble chip#2925
rajsite merged 12 commits intomainfrom
users/vgleason/chips

Conversation

@hellovolcano
Copy link
Copy Markdown
Contributor

Pull Request

🤨 Rationale

A blazor client application needs the nimble chip.

👩‍💻 Implementation

  • Added nimble chip to all-components.ts
  • Added Blazor and react wrappers.
  • Added Blazor tests.
  • Added chips to example apps for both Blazor and React.

🧪 Testing

Ran existing tests
Added and ran blazor tests
Manually verified

✅ Checklist

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

@hellovolcano hellovolcano marked this pull request as ready for review April 10, 2026 21:45
Comment thread packages/react-workspace/react-client-app/src/components/App.tsx Outdated
Comment thread change/@ni-nimble-blazor-3cdd2b35-fb7c-451d-b6ff-a85cb385ea22.json
@jattasNI
Copy link
Copy Markdown
Contributor

FYI @rajsite refactored the example apps in #2926 so there will be a bit of conflict resolution and new pattern adoption needed for this PR.

Comment thread packages/react-workspace/nimble-react/src/chip/index.ts
Co-authored-by: Milan Raj <rajsite@users.noreply.github.com>
@rajsite rajsite enabled auto-merge (squash) April 15, 2026 19:08
@rajsite rajsite merged commit 33a1e34 into main Apr 15, 2026
10 of 11 checks passed
@rajsite rajsite deleted the users/vgleason/chips branch April 15, 2026 19:35
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.

3 participants