Flere feilrettinger og forbedringer i Storybook #886
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.