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

fix timestamp affecting row height #256

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

malinnsnieske
Copy link
Collaborator

Background

Når man legger til svar eller kommentar, øker timestampen høyden på raden som ser rart ut

Solution

La til padding på DataTableCell-komponenten slik at det alltid er plass til timestampen

Resolves #issue-this-pr-resolves
#248

Copy link
Collaborator

@etse etse left a comment

Choose a reason for hiding this comment

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

Jeg kan ikke helt se at dette løser helt problemet. Det er nå mye mer luft i cellen - men den vokser på samme måte når teksen dukker opp. I tillegg ville jeg nok kun brukt marginBottom og ikke alle retningene

Jeg tror at det kanskje hadde vært bedre å legge koden for å legge til luft i bunnen inn i AnwerCell og Comment, og kun hvis man ikke har updated som skal vises.

F.eks. noe som denne på <Flex-komponenten som wrapper innholdet i Comment-cell.

marginBottom={updated == null ? "2.2rem" : "0"}

(juster størrelse så det passer med hvor mye den vokser)

@perzonas
Copy link
Collaborator

Som stefen er inne på løser dette ikke problemet. Du kan gjøre som steffen føreslår eller sette en fixed minHeight på elementet alternativt legge til en placeholder component når vi ikke viser noe som har samme height som "updated at".

Uansett hade jeg gåt lenger ned i stacken og lagt på propsen på den spesifike komponenten eller parent komponenten og ikke på grand ... grand parent (har ikke sjekket hvor langt opp vi er) komponenten.

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.

Fiks at rader endrer høyde når tidspunkt settes for første gang
3 participants