fix: spacing resourcelist#2918
Conversation
|
Er nok fremdeles for stort. På https://www.figma.com/design/E4Y9nwHcDVTETrEQFxf3bD/Components?node-id=22471-17631&t=9MQrJaFPA1lLVzDO-0 viser det at størrelsen er 24px. |
There was a problem hiding this comment.
Putt heller BatchSelectOptionsWrapper inn for å erstatte <>.
Co-authored-by: Copilot <[email protected]>
| transitionProperty: "opacity", | ||
| transitionTimingFunction: "ease-in-out", | ||
| minHeight: "3xlarge", | ||
| minHeight: "3xsmall", |
There was a problem hiding this comment.
Antar du kan fjerne heile minHeight siden linja tar plassen til knappene inni.
| }, | ||
| }); | ||
|
|
||
| const ControlsWrapper = styled("div", { |
There was a problem hiding this comment.
Ser ikkje heilt kva denne gjør fra eller til. Om eg fjerner den fra return-metoden så ser det identisk ut for meg. Veldig mulig eg tar feil altså.
…sWrapper Co-authored-by: Copilot <[email protected]>
Jonas-C
left a comment
Jonas-C
left a comment
There was a problem hiding this comment.
Litt sent ute på dette ser jeg. Dette er sånn ca det vi hadde før, og som Hedvig ville ha endret. Vil hun revertere tilbake til gammel oppførsel, eller vil hun at vi skal gjøre dette på en annen måte?
| )} | ||
| </BatchSelectOptionsWrapper> | ||
| {!!selectedResources.length && ( | ||
| <BatchSelectOptionsWrapper visible> |
There was a problem hiding this comment.
Hvis denne alltid er visible kan vi fjerne stylingen og propen.
Kanskje ønsket var at ved klikk på Velg flere, så skulle plass til knappene vises. Eg mistforsto i tilfelle det. Kan spørre på trello. |
3 participants
Fixes /NDLANO/Issues/issues/4537