feat: initial setup of button classes #48
Conversation
|
fabiangaukler
force-pushed
the
feat/buttons
branch
from
009cad2
to
d97c9a1
Compare
| stories: ["../src/**/*.mdx"], | ||
| addons: ["@storybook/addon-links", "@storybook/addon-essentials"], | ||
| stories: ["../src/**/*.mdx", "../src/**/*.stories.ts"], | ||
| addons: ["@storybook/addon-essentials", "storybook-addon-pseudo-states"], |
There was a problem hiding this comment.
ist addon-links in den essentials enthalten, oder weshalb ist es rausgeflogen?
There was a problem hiding this comment.
Die Links verwenden wir momentan nicht, soweit ich weiß
| }; | ||
|
|
||
| // Hovered (Button Hovered) | ||
| export const HoveredStory: Story = { |
There was a problem hiding this comment.
Brauchen wir die Stories für die einzelnen Zustände wirklich? Ist im Storybook nicht eigentlich spannender, wie die Buttons (oder andere Komponenten) aussehen und verhalten, wenn man sie in einer Anwendung einbaut?
Meine Auffassung einer einzelnen Story wäre genrell "das ist eine Möglichkeit, wie du diese Komponente nutzen kannst". In dem Moment wird dann auch der "view code" button richtig spannend für poentielle Nutzer des design systems.
| font-size: var(--hap-typography-font-size-body-small); | ||
| font-weight: var(--hap-typography-font-weight-medium); | ||
| letter-spacing: var(--hap-typography-letter-spacing-lg); | ||
| /* has to be calculated because it's only accepted as a value of percentage */ |
There was a problem hiding this comment.
There was a problem hiding this comment.
Ich hatte da mal Probleme mit, aber nach deinem Kommentar hab ich es wieder probiert und es funktioniert mit px
No description provided.