Skip to content

✨ Linjer på rediger-siden viser flere farger#2344

Open
olahjacobsen wants to merge 2 commits intots6from
show-airport-express-bus-colors
Open

✨ Linjer på rediger-siden viser flere farger#2344
olahjacobsen wants to merge 2 commits intots6from
show-airport-express-bus-colors

Conversation

@olahjacobsen
Copy link
Copy Markdown
Contributor

@olahjacobsen olahjacobsen commented Mar 25, 2026

⚠️ Denne PR-en har base-branch på ts6 (#2359)

🥅 Bakgrunn

Tavla-admin har ikke alltid de samme fargene og palettene som tavla-visning har. For å skape en rød tråd, og for brukeropplevelsen burde disse være så like som mulig.

✨ Løsning

  • Legg til farger basert på transportSubmode
  • Fargene er representert i palettet globalt
  • Hvis transportmodusen starter med airport, bruk fargene for fly
  • Logikken er hentet over fra tavla-visning

Det er noen caser man burde tenke på, se bilder:

  • Vanskeligere å skille buss fra tog fra annet når det er flyfarger
  • Fargene på linjene stemmer ikke med chipen for plattformen
  • Fargene på sorteringen på toppen av siden fjerner nå ikke alle fargene som tilhører den lenger (flytog har farge for fly men er av type tog fortsatt)

📸 Bilder

Før Etter
image image
image image
image image

✅ Sjekkliste

  • Testet i Chrome, Firefox og Safari

@olahjacobsen olahjacobsen changed the title Show airport express bus colors ✨ Linjer på rediger-siden viser flere farger Mar 25, 2026
@olahjacobsen olahjacobsen marked this pull request as ready for review March 25, 2026 11:09
@olahjacobsen olahjacobsen requested a review from a team as a code owner March 25, 2026 11:09
@olahjacobsen olahjacobsen changed the base branch from main to ts6 April 8, 2026 13:43
# Conflicts:
#	tavla/app/(admin)/tavler/[id]/rediger/components/Settings/components/TransportPaletteSelect.tsx
@olahjacobsen olahjacobsen force-pushed the show-airport-express-bus-colors branch from 8c24a64 to 24f1d47 Compare April 8, 2026 13:49
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.

1 participant