feat/theme component missing #3154
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
veslav3
force-pushed
the
feat/theme-component-missing
branch
from
105a599 to
46a99e8
Compare
|
eslook
left a comment
eslook
left a comment
There was a problem hiding this comment.
Ziet er wat mij betreft goed uit, maar gezien het een nieuwe pagina is, nog ff check door Robbert of Yolijn
| Wat gaaf, je hebt een component gevonden die nog niet in het thema verwerkt is! | ||
| Er zijn meerdere stappen mogelijk om dit op te lossen. | ||
|
|
||
| ## 1. Praat met de designer |
There was a problem hiding this comment.
Guidance wat je met de designer bespreekt:
Een korte checklist:
- welke component
- welke states ontbreken
- welke tokens nodig zijn
- of het component al in Figma bestaat
|
|
||
| _Gebruik deze optie als je direct verder wilt, zonder te wachten op de permanente oplossing._ | ||
|
|
||
| Je kunt de ontbrekende tokens lokaal overschrijven met de waarden die je nodig hebt: |
There was a problem hiding this comment.
Er wordt niet uitgelegd wat de risicos zijn van deze aanpak: dat dit kan afwijken van toekomstige updates
|
|
||
| Deze wijzigingen worden gedaan in de [themes-repository](https://github.com/nl-design-system/themes). | ||
|
|
||
| ### 3.1. Maak een issue aan |
There was a problem hiding this comment.
Wat maakt een goed issue? DOD / welk component / link naar documentatie of Figma
| Hiermee maak je inzichtelijk waarom de wijziging nodig is en voorkom je dat anderen hetzelfde werk doen. | ||
| Elk issue krijgt automatisch een nummer, handig om die later te gebruiken bij het maken van de pull-request om de issue automatisch weer te sluiten. | ||
|
|
||
| ### 3.2. Pas de tokens aan in het thema |
There was a problem hiding this comment.
hoe tokens zijn opgebouwd? wat wel/niet aangepast mag worden?
closes #2913