markdown render idea working #1713
Conversation
...pplicantAdminPage/components/RecruitmentInterviewNotesForm/RecruitmentInterviewNotesForm.tsx
Show resolved
Hide resolved
...pplicantAdminPage/components/RecruitmentInterviewNotesForm/RecruitmentInterviewNotesForm.tsx
Show resolved
Hide resolved
| field.onBlur(); // Call the default onBlur handler from react-hook-form | ||
| handleNotesChange(newNotes.target.value); // Call your custom function on blur |
There was a problem hiding this comment.
Dette er nok ikke helt din skyld, men tror jeg hadde foretrukket (om mulig) at vi ikke forholdt oss til "newNotes", men heller bare forholdt oss til verdien til feltet i formet. Vet ikke om det funker med conditional rendering?
| .markdownBox { | ||
| min-height: 550px; | ||
| max-height: 550px; /* Add max-height to enable scrolling */ | ||
| margin: 10px; |
There was a problem hiding this comment.
Nå har jeg ikke sett så nøye på dette, men dette kan nok løses med padding og/eller gap. Tror det er en bedre løsning. Forklaring på hvorfor jeg foretrekker det finner du her
| .markdownBox { | ||
| cursor: pointer; /* Add cursor pointer to indicate it's clickable */ | ||
| } |
There was a problem hiding this comment.
Dette er vel ikke nødvendig nå som det er en button?
| width: 100%; | ||
| box-sizing: border-box; |
There was a problem hiding this comment.
Jeg tror box-sizing: border-box er default i hele prosjektet, så det skal nok ikke ha noe å si her. Må innrømme at jeg ikke er 100% sikker, så ville testa det ut 😅
| color: #333333; | ||
| background-color: #ffffff; | ||
| border: 1px solid #cccccc; |
There was a problem hiding this comment.
Vi pleier å prøve å bruke farger fra 'constants' filen. Bare så fargene skal være konsekvente. Det gjør du ved å skrive:
// øvers i filen
@import 'src/constants'
//bruk av farger, f. eks.
color: $grey;
Se i "constants.scss" filen for å se alle fargene
| font-size: 16px; | ||
| line-height: 1.5; |
There was a problem hiding this comment.
Er dette forskjellig fra default?
| display: block; | ||
| font-family: inherit; |
There was a problem hiding this comment.
Er ikke begge disse default?
| resize: vertical; | ||
| box-sizing: border-box; |
There was a problem hiding this comment.
Mulig box-sizing er default i prosjektet (Er ikke det generelt i css, men burde vært det). Da trenger du ikke definere det på nytt her
No description provided.