Skip to content

Research: use AMS Header?#387

Draft
TessaViergever wants to merge 4 commits intomainfrom
refactor/header-ams
Draft

Research: use AMS Header?#387
TessaViergever wants to merge 4 commits intomainfrom
refactor/header-ams

Conversation

@TessaViergever
Copy link
Copy Markdown
Contributor

@TessaViergever TessaViergever commented Mar 24, 2026

⚠️ Do not merge - research purposes only atm

In this PR

  • replace Utrecht Header with AMS Header component in Layout.tsx
  • use NL Link, NL Button, Utrecht Icon, Utrecht Textbox to create visual Header items needed for MijnServices
  • add some missing Header tokens to voorbeeld theme (other themes may follow but just one for POC)
  • adjust Logo to match AMS Header use

Related issue

#255

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mijn-services Ready Ready Preview Mar 24, 2026 10:41am

Request Review

@TessaViergever TessaViergever requested a review from bddjong March 24, 2026 10:41
@bddjong
Copy link
Copy Markdown
Contributor

bddjong commented Apr 15, 2026

@TessaViergever dit ligt voor nu even bij Francesca om te kijken of zij vanuit figma ook uit de voeten kan met het community component. Als dat kan kunnen we deze afronden door 'm gelijk te trekken met figma

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.

2 participants