✨ Copybutton-komponent #1982
✨ Copybutton-komponent #1982
Conversation
🦋 Changeset detectedLatest commit: 2b4a425 The changes in this PR will be included in the next version bump. This PR includes changesets to release 10 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
There was a problem hiding this comment.
story for bruk av CopyButton med breadcrumbs?
There was a problem hiding this comment.
follow-up Q: burde vi mirrore alle Examples i DS med examples i Storybook? 1:1 ?
There was a problem hiding this comment.
Breadcrumbs != Component? (egentlig urelatert)
There was a problem hiding this comment.
Tenkte den som et mer praktisk eksempel i bruk, men er kanskje litt forvirrende ja siden den ikke har noe med komponenten å gjøre. Kanskje hvis jeg renamer den til Used in breadcrumbs?
Selv ser jeg på storybook mer som en testplatform for chromatic, men blir gjerne utfordret på det
- aksel.nav.no: Mer praktiske eksempler, showcase av muligheter
- storybook: Teste alle varianter, størrelser, komposisjoner etc
|
Burde vi alltids sjekke ut kode + yarn boot + yarn storybook når man reviewer en PR? (har sjekket chromatic og lest kode her). Selv tenker jeg det er jo best å sjekke det ut og kjøre lokalt, men mye av runtime testing kan man få ut av chromatic sin storybook. Thoughts? |
Tenker det meste blir testet automatisk ved PR slik at man slipper å gå gjennom den prosessen
Er alltids sikkert lurt å ta en dobbelsjekk lokalt hvis det er snakk om litt større endringer, men til nå har disse sjekkene plukket opp det aller meste som kan gå feil 🤞 |
<CopyToClipboard />fra@navikt/ds-react-internal