Skip to content

Add OK search-input#2918

Draft
fredvisser wants to merge 3 commits intomainfrom
users/fvisser/ok-lib-angular-search-input
Draft

Add OK search-input#2918
fredvisser wants to merge 3 commits intomainfrom
users/fvisser/ok-lib-angular-search-input

Conversation

@fredvisser
Copy link
Copy Markdown
Contributor

@fredvisser fredvisser commented Apr 4, 2026

Pull Request

🤨 Rationale

This PR adds the OK search-input component, including the OK web component, Angular wrapper, Storybook coverage, and a small example-client-app integration.

👩‍💻 Implementation

The implementation adds the OK search-input web component in the OK components package, including its template, styles, appearance types, and browser tests.

It adds the matching Angular wrapper entry point for ok-angular/search-input, registers the component in ok-components all-components, and adds a focused example-client-app demo block.

It also adds Storybook stories, matrix coverage, and docs for the new OK search-input component.

Important files:

  • packages/ok-components/src/search-input/index.ts
  • packages/ok-components/src/search-input/template.ts
  • packages/ok-components/src/search-input/styles.ts
  • packages/angular-workspace/ok-angular/search-input/ok-search-input.directive.ts
  • packages/storybook/src/ok/search-input/search-input.stories.ts

🧪 Testing

Automated validation run locally:

  • npm run build:components -w @ni/ok-components
  • npm run lint -w @ni/ok-components
  • npm run test-chrome -w @ni/ok-components
  • npm run build:ok -w @ni-private/angular-workspace
  • npx ng lint @ni/ok-angular
  • npm run test-ok -w @ni-private/angular-workspace
  • npm run build:application -w @ni-private/angular-workspace
  • npx ng lint example-client-app
  • npm run lint -w @ni-private/storybook

✅ Checklist

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

@fredvisser fredvisser requested a review from jattasNI April 4, 2026 03:51
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.

1 participant