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

Feat/utrecht thema #374

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Feat/utrecht thema #374

wants to merge 3 commits into from

Conversation

Marwaxhello
Copy link
Collaborator

@Marwaxhello Marwaxhello commented Jan 13, 2025

Utrecht Theme gebruiken voor Signalen

To-do's | Styling

(is dit gemaakt obv een oude versie van Purmerend Signalen? - ik kom veel issues tegen die al opgelost waren)

Op alle/meerdere pagina's

  • max-inline-size voor page-header content en page-footer content (zodat header en footer gecentreerd zijn)
  • geen marge onderin de pagina
  • focus ring heeft geen markeer kleur (hoort dit zo bij Utrecht?)
  • progress bar: op geen enkele form stap
  • marge tussen label/description verschilt van marge tussen legend/description (bijv. op stap 1)
  • map niet zichtbaar (stap 2, mapDialog, stap 4 - juiste env variable/dependency ontbreekt misschien?)
  • icon buttons: icon positie
  • darkmode?
  • forced colors?

Stap 1

  • upload buttons: geen border

Stap 2

  • font-family van adressSelect (combobox)

MapDialog

  • font-family van "Zoek op adres" label in dialog
  • uitlijning/size van de icon in de close button
  • contrast buttons op de kaart onvoldoende? (lichtgrijs op wit)
  • font-family van "mijn locatie" (geen toestemming) error message
  • combobox: styling van listbox
    • H5 icm paragraph tekst looks awkward (h5 kleiner dan p)
    • hover "Hoe gebruik ik de kaart" (combobox?) niet voldoende contrast (?)

Stap 3

  • font-size/weight van onderdelen van "Mogen we uw melding doorsturen" looks awkward
  • marge tussen onderdelen van "Mogen we uw melding doorsturen"
  • marge tussen H2 "Mogen we u bellen" en de paragraph eronder (dicht op elkaar)

Stap 4

  • font-family van "Wijzig .." links
  • font-size H2 en H3 (even groot)

Succespagina
x

To'do's | Overig

  • wijzigingen in config.json om teksten en URLs aan te passen naar Utrecht
  • favicon van Utrecht gebruiken ipv Purmerend

Copy link

vercel bot commented Jan 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
signalen-frontend-wcag ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 13, 2025 3:47pm

@Marwaxhello Marwaxhello requested a review from Robbert January 13, 2025 15:48
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