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

Flere feilrettinger og forbedringer i Storybook #886

Merged
merged 5 commits into from
Oct 18, 2024
Merged

Conversation

chrhein
Copy link
Contributor

@chrhein chrhein commented Oct 18, 2024

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:

  • Bilder som lastet inn på korrekt måte lokalt dukket ikke opp etter deploy til Github Pages. Flytter alt av bilder i assets-mappen tilbake i hovedmappen public og dermed skal bildene laste inn på riktig måte.
  • Dokumentasjonen for 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 av AspectRatio, 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 å bruke Link 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 å bruke justifyContent 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.

@chrhein chrhein requested a review from a team as a code owner October 18, 2024 09:19
Copy link

changeset-bot bot commented Oct 18, 2024

⚠️ No Changeset found

Latest commit: 19269cc

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

Copy link
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link
Contributor

github-actions bot commented Oct 18, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-18 09:23 UTC

@chrhein chrhein changed the title Flere feilrettinger i Storybook Flere feilrettinger og forbedringer i Storybook Oct 18, 2024
@chrhein chrhein merged commit 50366af into master Oct 18, 2024
8 checks passed
@chrhein chrhein deleted the innspill-fra-ws branch October 18, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant