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

Kalkulator for å beregne +/- 10 prosent månedsinntekt #2933

Merged
merged 10 commits into from
Oct 29, 2024

Conversation

gunnsteingarmo
Copy link
Contributor

Hvorfor er denne endringen nødvendig? ✨

Laget en enkel kalkulator som regner ut månedsinntekt +/- 10 prosent av årsinntekt og fyller inn tekst i begrunnelse.

Det var litt knot med å få til autofocus/fokus på inputen for årsinntekt. En løsning var å bruke useRef textFieldRef.current.focus() og sette fokus i en useEffect hver gang erDropdownÅpen endrer seg.

image

image

…av årsinntekt og fyller inn tekst i begrunnelse.
Copy link
Contributor

@olekvernberg olekvernberg left a comment

Choose a reason for hiding this comment

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

Kjekt! Er det avklart med saksbehandlere at de ønsker tekst rett inn i begrunnelse fremfor at tallene vises et sted uten tekst?

@gunnsteingarmo
Copy link
Contributor Author

Kjekt! Er det avklart med saksbehandlere at de ønsker tekst rett inn i begrunnelse fremfor at tallene vises et sted uten tekst?

Nei, det har jeg ikke tenkt på. Skal høre med Mirja før hun tester 👍

Copy link
Contributor

@ViktorGSolberg ViktorGSolberg left a comment

Choose a reason for hiding this comment

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

Ser bra ut! Tror saksbehandlerne vil sette pris på denne 😊

Har noen forslag til kodeendringer som jeg tror vil gjøre koden mer lesbar, se gjerne over disse.

Et forslag:
Når man klikker på beregnknappen så burde kalkulatoren også lukkes

gunnsteingarmo and others added 4 commits October 28, 2024 12:42
…gnetInntektTekstIBegrunnelse. Kan åpne kalkulator med 'cmd + k'. Viser feilmelding hvis det ikke er fylt inn noe tall. Lukker dropdown når 'enter' er trykket.
@gunnsteingarmo
Copy link
Contributor Author

Ser bra ut! Tror saksbehandlerne vil sette pris på denne 😊

Har noen forslag til kodeendringer som jeg tror vil gjøre koden mer lesbar, se gjerne over disse.

Et forslag: Når man klikker på beregnknappen så burde kalkulatoren også lukkes

Gjort endringer 👍 Har også gjort slik at "kalkulatoren" lukkes

@gunnsteingarmo gunnsteingarmo merged commit 4243e75 into main Oct 29, 2024
1 check passed
@gunnsteingarmo gunnsteingarmo deleted the beregnet-inntekt-kalkulator branch October 29, 2024 07:25
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.

3 participants