0% feriepenger! #1229
Merged
0% feriepenger! #1229
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For enkelte yrkesgrupper er det ikke noen feriepengesats, så derfor burde det være mulig å definere 0% feriepenger i løsningen. Ettersom vi lager enda et valg vil radiogruppen bli for stor, og derfor bytter vi ut radio med en dropdown-select (lik feks arbeidsgiveravgift)
Når teksten som indikerer hva som mangler er for kort, vil varselteksten ende opp på samme linje som "tittelen", som ser veldig rart ut. Denne endringen flytter varselboksen på linjen under. I tillegg endres sjekken på om noe mangler til en "isNil"-sjekk, slik at vi ikke behøver å "hacke" til løsningen når et felt feks er tallet 0.
Dersom man taster inn tallet 0 i ValutaInput så vil tallet byttes ut med tom string når man refresher siden, selv om tallet da har blitt satt til 0. Løses ved at parsingen av tallet returnerer undefined dersom det ikke er et tall, og så sjekke om verdien er undefined, i stedet for å gjøre en "falsey"-sjekk. For å gjenbruke "isNil"-predikatet opprettes også en predicates- modul, og isNil flyttes dit og blir omdøpt til erNil.
<p>-elementer kan ikke nøstes! Men her hadde vi et tilfelle hvor en <BodyShort> inneholdt en <p>, som i praksis betyr at vi nøster <p>'er. Det er ingen spesiell grunn til at dette gjøres, så vi bare endrer til et fragment. Kan også vurderes å bruke en <span>, men spans har oppførsel i motsetning til fragments.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Før
Etter