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

IS-1900: Aksel DS in dialogmøte-referat-deltakere components #1128

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

andersrognstad
Copy link
Contributor

Hva har blitt lagt til✨🌈

Endret til å bruke Aksel DS + mer tailwind i Referat-deltakere-komponenter.

Screenshots 📸✨

image

@andersrognstad andersrognstad requested a review from a team as a code owner December 11, 2023 13:10
@andersrognstad andersrognstad changed the title Aksel DS in dialogmøte-referat-deltakere components IS-1900: Aksel DS in dialogmøte-referat-deltakere components Dec 11, 2023
Comment on lines +54 to 55
<div className="mt-4">
<FieldArray<NewDialogmotedeltakerAnnenDTO> name={"andreDeltakere"}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Kan stylingen bare ligge på FieldArray eller blir det krøll da? 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Funket ikke 😞 FieldArray kommer vel fra form-biblioteket og ser ikke ut til å ha className-prop.

Copy link
Contributor

Choose a reason for hiding this comment

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

Aha! Skjønner 😊

<FlexRow topPadding={PaddingSize.SM}>
<Normaltekst>{texts.behandlerTekst}</Normaltekst>
</FlexRow>
<FlexRow topPadding={PaddingSize.SM}>{texts.behandlerTekst}</FlexRow>
Copy link
Contributor

Choose a reason for hiding this comment

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

Trenger ikke å gjøre det i denne PR'en, men er FlexRow noe vi egentlig vil fase ut? Jeg synes at en slik komponent hadde vært enklere å lese om den bare stod i klartekst med tailwind classes😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, litt enig i det. Har erstattet det noen steder, men ble litt mye i denne PRen så lot det ligge enn så lenge.

Copy link
Contributor

@vetlesolgaard vetlesolgaard left a comment

Choose a reason for hiding this comment

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

Bare noen småkommentarer. Bra jobba! 🙌

@andersrognstad andersrognstad merged commit c8ccab6 into master Dec 12, 2023
@andersrognstad andersrognstad deleted the IS-1900-deltakere-expandable branch December 12, 2023 09:29
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