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

[draft] chore(ffe-buttons-react): Forbedring av dokumentasjon på knappene is storybook. #2502

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hagenek
Copy link
Contributor

@hagenek hagenek commented Jan 27, 2025

Beskrivelse

Lagt til endel forklarende tekst og andre forbedringer.

Motivasjon og kontekst

Testing

Spinn opp storybook lokalt på branchen eller gå på azure deploymenten og sammenlign med prod.

@hagenek hagenek requested a review from a team as a code owner January 27, 2025 10:04
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-beach-0d62d0d03-2502.westeurope.2.azurestaticapps.net

Copy link
Contributor

@dagfrode dagfrode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vi bør kanskje diskutere hvordan vi vil dokumentere ting med storybook. Jeg er egentlig fan av å dokumentere komponenten mest mulig p åvanlig måte og la storybook plukke det opp, men ser og at det er enkelte ting storybook ikke plukker opp lsik ting er satt opp nå.

docs: {
description: {
component:
'ActionButton er en knapp som brukes for å trigge en handling.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jeg prøvde meg litt frem her. Egentlig så skal storybook støtte at denne ligger i docstringen til komponenten, men ser ikke ut til at det fungerer når vi bruker mdx filer.

Jeg klarer ikke å se hvor denne docstringen dukker opp i den bygde storybooken?

Det er jo en beskrivelse og i .mdx fila for actionbutton?

},
},
isLoading: {
description: 'Viser en spinner når knappen laster',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Disse beskrivelsene kan vel og legges på propnivå og storybook fanger det opp? så er det nærmere komponenten

@hagenek
Copy link
Contributor Author

hagenek commented Jan 27, 2025

Vi bør kanskje diskutere hvordan vi vil dokumentere ting med storybook. Jeg er egentlig fan av å dokumentere komponenten mest mulig p åvanlig måte og la storybook plukke det opp, men ser og at det er enkelte ting storybook ikke plukker opp lsik ting er satt opp nå.

La oss ta en sesjon sammen med denne PR-en hvor vi ser på hvordan vi kan dokumentere selve koden og få det over til storybook. Setter den som draft.

@hagenek hagenek changed the title chore(ffe-buttons-react): Forbedring av dokumentasjon på knappene is storybook. [draft] chore(ffe-buttons-react): Forbedring av dokumentasjon på knappene is storybook. Jan 27, 2025
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.

2 participants