Skip to content

Gjør check- og kryssikonene konsistente med Material Symbols#5935

Open
lmfaole wants to merge 1 commit intomainfrom
patch/3793-checkmark-icons
Open

Gjør check- og kryssikonene konsistente med Material Symbols#5935
lmfaole wants to merge 1 commit intomainfrom
patch/3793-checkmark-icons

Conversation

@lmfaole
Copy link
Copy Markdown
Contributor

@lmfaole lmfaole commented Apr 10, 2026

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

  • byttet GreenCheckIcon og RedCrossIcon over til Material Symbols
  • oppdatert check- og kryssmarkørene i lister til samme ikonspor
  • beholdt uttrykket med svart symbol i farget sirkel
  • lagt til changeset for @fremtind/jokul

Testet

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

Copilot AI review requested due to automatic review settings April 10, 2026 09:09
@fremtind-bot
Copy link
Copy Markdown
Collaborator

fremtind-bot commented Apr 10, 2026

Hei, @lmfaole! Takk for at du bidrar til Jøkul! 🙌
For å sørge for god dokumentasjon og brukeropplevelse er det fint om du husker å gjøre disse tingene:

  • Test endringene dine ♿️
    Sørg for å teste endringene dine for både responsivitet og universell utforming (tastaturnavigasjon, skjermleser, etc.).
    false
  • Oppdater eller lag nye stories 📚
    Hvis du har endret eller lagt til ny funksjonalitet i komponenter må du huske å oppdatere eller legge til Storybook-stories for dette i komponenten(e).
  • Oppdater informasjon i Sanity 📖
    Hvis du har endret eller lagt til ny funksjonalitet i komponenter må du huske å oppdatere informasjonen om komponenten(e) i Sanity.

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.

@fremtind-bot
Copy link
Copy Markdown
Collaborator

fremtind-bot commented Apr 10, 2026

Forhåndsvisning: https://jokul.fremtind.no/preview/patch/3793-checkmark-icons/
🔍 Commit: 1f29850

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 Apr 10, 2026
@lmfaole lmfaole changed the title fix: bruk material symbols for checkmark-ikoner Gjør check- og kryssikonene konsistente med Material Symbols Apr 10, 2026
@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from 85f08a3 to 4e3a923 Compare April 10, 2026 09:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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 GreenCheckIcon og RedCrossIcon til å bruke Icon (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.

fremtind-bot added a commit that referenced this pull request Apr 10, 2026
@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from 4e3a923 to f5790f9 Compare April 10, 2026 10:03
fremtind-bot added a commit that referenced this pull request Apr 10, 2026
Copilot AI review requested due to automatic review settings April 10, 2026 10:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

fremtind-bot added a commit that referenced this pull request Apr 10, 2026
@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from e9de2fb to 06c9982 Compare April 10, 2026 10:09
@lmfaole lmfaole marked this pull request as ready for review April 10, 2026 10:09
@lmfaole lmfaole requested a review from a team as a code owner April 10, 2026 10:09
fremtind-bot added a commit that referenced this pull request Apr 10, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 13 changed files in this pull request and generated 2 comments.

@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from 06c9982 to 4b19502 Compare April 10, 2026 10:17
fremtind-bot added a commit that referenced this pull request Apr 10, 2026
Copilot AI review requested due to automatic review settings April 10, 2026 10:25
@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from 4b19502 to 911a25e Compare April 10, 2026 10:25
fremtind-bot added a commit that referenced this pull request Apr 10, 2026
@lmfaole lmfaole force-pushed the patch/3793-checkmark-icons branch from 911a25e to 6dc203f Compare April 10, 2026 10:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 15 changed files in this pull request and generated no new comments.

fremtind-bot added a commit that referenced this pull request Apr 10, 2026
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.

Feil: Ikoner for checkmarks er ikke like

3 participants