Gjør check- og kryssikonene konsistente med Material Symbols#5935
Gjør check- og kryssikonene konsistente med Material Symbols#5935
Conversation
|
Hei, @lmfaole! Takk for at du bidrar til Jøkul! 🙌
Hvis du lurer på hvordan du kommer i gang med noe av dette, er det bare å spørre oss i Jøkul-teamet om hjelp! Tagg oss i en kommentar her, eller spør oss på Teams. Warning @lmfaole, Figma Code Connect feilet. |
|
✅ Forhåndsvisning: https://jokul.fremtind.no/preview/patch/3793-checkmark-icons/ Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes. |
85f08a3 to
4e3a923
Compare
There was a problem hiding this comment.
Pull request overview
Denne PR-en gjør check- og cross-ikoner (både fargede ikoner og listemarkører) konsistente ved å gå fra håndtegnet SVG/data-URI til Material Symbols, samtidig som uttrykket med svart symbol i farget sirkel beholdes.
Changes:
- Bytter
GreenCheckIconogRedCrossIcontil å brukeIcon(Material Symbols) i stedet for inline SVG. - Oppdaterer listemarkører for check/cross til å bruke Material Symbols-fonten med farget bakgrunn.
- Legger til changeset for patch-release av
@fremtind/jokul.
Reviewed changes
Copilot reviewed 5 out of 13 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/jokul/src/components/list/styles/list.scss | Bytter listemarkør-implementasjon fra data-URI SVG til Material Symbols-basert ::before med farget sirkel. |
| packages/jokul/src/components/icon/styles/icon.scss | Oppdaterer styling for fargede statusikoner (sirkel + symbol) til å bruke tekstfarge/bakgrunn fremfor SVG-fill. |
| packages/jokul/src/components/icon/icons/RedCrossIcon.tsx | Erstatter SVG med Icon-basert Material Symbol for “close” og beholder eksisterende klassehook. |
| packages/jokul/src/components/icon/icons/GreenCheckIcon.tsx | Erstatter SVG med Icon-basert Material Symbol for “check” og beholder eksisterende klassehook. |
| .changeset/material-checkmark-icons.md | Patch-changeset for å publisere oppdateringen. |
4e3a923 to
f5790f9
Compare
e9de2fb to
06c9982
Compare
06c9982 to
4b19502
Compare
4b19502 to
911a25e
Compare
911a25e to
6dc203f
Compare
Bakgrunn
I #3793 er problemet at check- og kryssikonene ikke ser like ut på tvers av komponentene våre. De fargede ikonene og listeikonene har brukt ulike SVG-varianter, mens resten av løsningen allerede bruker Material Symbols.
Hva som er gjort
GreenCheckIconogRedCrossIconover til Material Symbols@fremtind/jokulTestet
Jeg har gått gjennom diffen og CSS-en statisk. Jeg fikk ikke kjørt
node,pnpm, tester eller build i dette miljøet fordi verktøyene ikke er installert her.Closes #3793