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: skrive om flex komponent #4258

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

Fix: skrive om flex komponent #4258

wants to merge 13 commits into from

Conversation

zenabii
Copy link
Contributor

@zenabii zenabii commented Nov 6, 2024

Fremdeles work in progress!
Ikke klar for review enda :)

Notater for dokumentasjon til Flex:

PROPS:
Layout
Angir layout for elementer per rad. Layouter med lik bredde er navngitt for å korrelere med antall elementer du ønsker per rad, da dette gir en konsistent opplevelse når du hekker. Layouter med ujevn bredde er navngitt basert på et 12-kolonne rutenett for å korrelere med hvordan designere jobber i Figma.

Godtar både en enkelt verdi layout="2" eller et objekt for å spesifisere forskjellige verdier for forskjellige bruddpunkter (alle bruddpunkter er valgfrie): layout={{ xs: 1, md: 2, lg: 4 }}.

Gap
Gap kan være en av forhåndsdefinerte konstanter. Godtar også to verdier atskilt med mellomrom for å spesifisere forskjellige row- og column gap. For eksempel vil gap="lg md" forårsake lg row gap og md column gap.

Avstandsstørrelsene er xs (unit-10 / 8px), sm (unit-20 / 16px), md (unit-30 / 24px), lg (unit-40 / 32px), xl (unit-50 / 40px), xxl (enhet-100 / 80px). (disse verdiene må tilpasses etter Jøkul sin praksis - her har jeg bare lagt inn midlertidige verdier)

Aksepterer både en enkelt verdi gap="md" eller et objekt for å spesifisere forskjellige verdier for forskjellige bruddpunkter (alle bruddpunkter er valgfrie): gap={{ xs: 'none', md: 'lg' }}.

Alt du kan gjøre med display: flex, du kan gjøre med < Flex >. Props til < Flex > er kjent for CSS flex-spesifikasjonen. Dette betyr at vi kan skrive mindre CSS når vi bare trenger å plassere elementer - både i rutenett, men også på mikronivå, som Buttons ved siden av hverandre. Det betyr også; jo mer du kjenner til CSS flex-standarden, jo mer kan du låse opp kreftene til < Flex >. Vi anbefaler disse CSS flex guidene og spillene:
https://css-tricks.com/snippets/css/a-guide-to-flexbox/
https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Flexbox
https://flexboxfroggy.com

Som standard gjengis alle Flex som vanlige

-tagger. Du kan endre dette ved å sende et hvilket som helst HTML-tagnavn eller React-komponent til as prop.

🎯 Sjekkliste

@zenabii
Copy link
Contributor Author

zenabii commented Nov 14, 2024

Har begynt å såvidt samle info og skrible litt for dokumentasjon til flex, legger det øverst i selve PR'en :)

@zenabii zenabii force-pushed the fix--flex-component branch from 4983998 to 5b454a3 Compare January 7, 2025 11:42
@fremtind-bot
Copy link
Collaborator

fremtind-bot commented Jan 7, 2025

Forhåndsvisning: https://jokul.fremtind.no/preview/fix--flex-component/
🔍 Commit: 24d23a6

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request Jan 7, 2025
) {
const Tag = as || "div";
const gaps = toObj(gap).flatMap(([breakpoint, gap]) => {
const [row, col = row] = gap.trim().split(" ");
Copy link
Contributor

Choose a reason for hiding this comment

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

Her smeller det i eksempelet til Expander (det er derfor visuell regresjonstest feiler) fordi gap er sent inn som et tall og tall har ingen trim() funksjon.

Har også kanskje lyst til å foreslå at toObj får et bedre navn for den ser ikke ut til å lage noe objekt men et array. Det var litt vanskelig å resonere seg fram til hvorfor det krasjer.

fremtind-bot added a commit that referenced this pull request Jan 9, 2025
fremtind-bot added a commit that referenced this pull request Jan 9, 2025
@zenabii
Copy link
Contributor Author

zenabii commented Jan 9, 2025

Lagt inn ignore prettier på selve style filen for Flex og ryddet opp, dette reduserer linjer kode i style.scss fra over 1000+ til ca 300 linjer med styling 👍

fremtind-bot added a commit that referenced this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
3 participants