Amsterdam Page Header ondersteunt geen custom Logo

[eslook]

TLDR

Dit component is al Community maar voldoet eigenlijk nog niet aan de standaarden, omdat dit component niet buiten Amsterdam gebruikt kan worden. Dit omdat de Logo component gebruikt in de Page Header component alleen preset logos van Amsterdam toestaat.

Oplossingsrichting

Issue aangemaakt in Amsterdam:

TLDR

Context I am using the Page Header (Storybook, Code) in NL Design System (themes Storybook, available to entire community).

Problem I can only use one of the preset logos in the Page Header (code) which are all Amsterdam. This disallows the use of the Logo and components which use the logo (ie the Page Header) outside of Amsterdam.

Solution Since Page Header makes use of Logo (Storybook, Code) I want to request allowing for a custom LogoConfigItem to be passed as brand. I have included a solution which I tested locally, see below.

... Code Snippets ...

Bron: Amsterdam/design-system#2311

Actiepunten

• Onderzoek doen in Amsterdam Design System hoe dit opgelost kan worden.
• Issue aanmaken met duidelijke context en potentiele oplossing.
• Issue aankaarten via Slack bij Amsterdam developers.
• Afwachten op reactie en overige actiepunten.
• ...
• Controleren of probleem nu is opgelost in Code en in Figma.

Acceptatiecriteria

• Versie van de Amsterdam Page Header in themes repository staat toe om een custom Logo te gebruiken. Dit kan worden geshowcased in de voorbeeld thema preview van de component.