Closed chrhein closed 2 weeks ago
Latest commit: 19269ccec2ec35d3d2ef5126ff6305f60c7fe8b0
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.
Package | Version | Score | Details |
---|
PR Preview Action v1.4.8 :---: Preview removed because the pull request was closed. 2024-10-18 09:23 UTC
Etter testefesten i design-laug denne uken ble det avdekket en rekke mindre og større feil her og der i dokumentasjonen på Storybook. Har begynt å rette opp i noe av dette, og denne PR-en fikser følgende:
assets
-mappen tilbake i hovedmappenpublic
og dermed skal bildene laste inn på riktig måte.Progress
viste eksemplene på en måte hvor komponentene manglet høyde og bredde og at de dermed ikke ble vist. Dette gjaldt også dokumentasjonen avAspectRatio
, så for begge disse er det nå lagt inn faste bredder slik at komponentene i hvert fall tegner opp noe på skjermen.Feedback
-komponenten som oppfordrer til å komme med ris og ros inneholdt lenker som har satt opp som<Button as="a" ... />
istedenfor å brukeLink
som vi tilbyr selv. Denne er nå endret til å bruke våre egne komponenter på den måten vi selv mener er best practice.Header
-dokumentasjonen hadde et eksempel hvor den samme komponenten ble repetert tre ganger uten at det var noe forskjell på disse. Tanken har nok opprinnelig vært å vise tre forskjellige måter å brukejustifyContent
på, men foreløpig var dette ikke brukt og jeg fjerner duplikatene for nå.Fortsatt mye som gjenstår fra testefesten, så resten får komme etterhvert.