Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add incubating nimble-card component #1630

Merged
merged 15 commits into from
Nov 1, 2023
Merged

Conversation

kjohn1922
Copy link
Contributor

@kjohn1922 kjohn1922 commented Oct 27, 2023

Pull Request

Initial component code for the nimble-card component.

See #296 and the nimble-card spec for more information.

🤨 Rationale

This adds the initial nimble-card component, storybook, and tests.

I will add a "title" slot to the template in a future PR. Future PRs will also add Angular and Blazor support.

The styling in the component right now is intended as a placeholder. We will get styling from our designer and update the styles before releasing the Routines UI with the card component.

👩‍💻 Implementation

  • Added myself as a code owner for the incubating component, per the contributing instructions
  • Marked the component as incubating in the component-status table
  • Added index, styles, and template files for the new component, called nimble-card

🧪 Testing

  • Added a storybook page under Incubating with an example
  • Added a spec file with a basic unit test

✅ Checklist

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

@kjohn1922 kjohn1922 changed the title Users/kelseyj/card component Add incubating nimble-card component Oct 27, 2023
@kjohn1922 kjohn1922 marked this pull request as ready for review October 30, 2023 16:15
.github/CODEOWNERS Outdated Show resolved Hide resolved
packages/nimble-components/src/card/tests/card.stories.ts Outdated Show resolved Hide resolved
@kjohn1922 kjohn1922 enabled auto-merge (squash) November 1, 2023 15:50
@kjohn1922 kjohn1922 merged commit 0049146 into main Nov 1, 2023
9 checks passed
@kjohn1922 kjohn1922 deleted the users/kelseyj/card-component branch November 1, 2023 16:10
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