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

Implement OnyxCard component #2383

Open
11 of 18 tasks
larsrickert opened this issue Dec 19, 2024 · 0 comments
Open
11 of 18 tasks

Implement OnyxCard component #2383

larsrickert opened this issue Dec 19, 2024 · 0 comments
Assignees
Labels
dev Requires technical expertise
Milestone

Comments

@larsrickert
Copy link
Collaborator

larsrickert commented Dec 19, 2024

Why?

We want to offer a card component.

Open questions / TODOs

No response

Depends on

Design

Figma

Acceptance criteria

  • the whole card content is provided by the user via the default slot
  • the card has a pre-defined padding which can optionally be overridden/removed by the user via custom CSS
  • Storybook examples are implemented for the following content (see Figma):
    • Headline + text
    • Small image + headline + text
    • large/full-width image + headline + text
    • Headline with right aligned content (e.g. a system button or badge/tag) + text
    • Headline + KPI (e.g. "5.00%)
  • optionally, the card can be made clickable. It is documented for the property that clickable cards must not contain any interactive content

Implementation details

  • if the card is clickable, we should use a <button> tag for the card
  • the card does not have a skeleton on its own, only the content can have skeleton (e.g. buttons)

Reference implementations

No response

Applicable ARIA Pattern

No response

Definition of Done

  • The following component requirements are implemented:

    • density
    • example usage added to apps/demo-app/src/views/HomeView.vue
  • Should be covered by tests:

    • visual tests (Playwright screenshots)
  • Make sure, that

    • follow-up tickets were created if necessary
    • updated version + documentation is deployed
    • Storybook can show the feature
    • Storybook code snippet of new/changed examples are checked that they are generated correctly
    • Namings are aligned with Figma

Approval

Storybook
Documentation

  • Checked and approved by
    • designer
    • dev
@larsrickert larsrickert added 0-refinement All issues that can or need to be estimated in our next refinement dev Requires technical expertise labels Dec 19, 2024
@larsrickert larsrickert self-assigned this Dec 19, 2024
@larsrickert larsrickert added this to onyx Dec 19, 2024
@github-project-automation github-project-automation bot moved this to New in onyx Dec 19, 2024
@larsrickert larsrickert moved this from New to Backlog in onyx Dec 19, 2024
@mj-hof mj-hof added this to the Cards & tiles milestone Jan 8, 2025
@larsrickert larsrickert moved this from Backlog to In Progress in onyx Jan 13, 2025
@larsrickert larsrickert removed the 0-refinement All issues that can or need to be estimated in our next refinement label Jan 13, 2025
github-merge-queue bot pushed a commit that referenced this issue Jan 13, 2025
Relates to #2383

Implement basic card component and examples

## Checklist

- [x] The added / edited code has been documented with
[JSDoc](https://jsdoc.app/about-getting-started)
- [ ] All changes are documented in the documentation app (folder
`apps/docs`)
- [x] If a new component is added, at least one [Playwright screenshot
test](https://github.com/SchwarzIT/onyx/actions/workflows/playwright-screenshots.yml)
is added
- [x] A changeset is added with `npx changeset add` if your changes
should be released as npm package (because they affect the library
usage)

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Requires technical expertise
Projects
Status: In Progress
Development

No branches or pull requests

2 participants