improved styling and layout on recruitmentapplicationformpage #1614
Conversation
snorrekr
requested review from
Mathias-a,
robines and
Snorre98
as code owners
| @@ -157,8 +158,6 @@ export function RecruitmentApplicationFormPage() { | |||
| </Link> | |||
| </h2> | |||
| <p className={styles.text}>{dbT(recruitmentPosition, 'long_description')}</p> | |||
| <h2 className={styles.sub_header}>{t(KEY.recruitment_applyfor)}</h2> | |||
| <p className={styles.text}>{t(KEY.recruitment_applyforhelp)}</p> | |||
| </div> | |||
| <div className={styles.other_positions}> | |||
There was a problem hiding this comment.
Hadde vært nice om om div fra og med linje 162 til og med linje 187 rendrer under søknadsformet hvis man er på mobil. Altså, under "Send søkand" knappe.
There was a problem hiding this comment.
Glemte å si det når jeg forklarte issuet muntlig. For å conditionaly rendre den nevnte diven kan du bruke useMobile hooken vår. Søk "isDesktop" eller "isMobile" i hele prosjekte, så finner du sikkert plasser hvor det er brukt.
Formålet er at man skal kunne skrive en søknad og trykke "Send søknad" uten å måtte scrolle på mobil.
| @@ -207,7 +206,12 @@ export function RecruitmentApplicationFormPage() { | |||
| submitText={submitText} | |||
There was a problem hiding this comment.
Kan du erstatte Samf-form på denne siden med React hook form? Står ikke i issuet, men greit å gjøre siden det er så lite som skal gjøres her.
There was a problem hiding this comment.
Blir veldig likt hvordan Simen har gjort det i denne PR-en: #1568
Men i din PR her skal det gjøres post/put request når søkeren trykker på knappen selvfølgelig.
No description provided.