Draft
Conversation
jiromaykin
requested changes
Apr 23, 2026
Collaborator
jiromaykin
left a comment
There was a problem hiding this comment.
ToDo:
- geen pixelwaarden gebruiken maar alleen Rem waarden met Token Studio https://docs.tokens.studio/manage-tokens/token-types/dimension misschien kun je REMs niet meteen invullen maar moet je er een procent teken achteraan invullen zodat Style Dictionary er CSS van kan maken
- uitzoeken of 'Lato' vervangen kan worden door 'Body,' en 'Ubuntu' door 'Heading', waarschijnlijk kan dat niet tenzij je met Token Studio er variabelen van kunt maken, en dan bijvoorbeeld 'Body' kunt laten doorverwijzen naar de waarde 'Lato' zodat het er in Figma toch nog uit ziet als of er een lettertype gekozen is, misschien d.m.v. de "alias": https://docs.tokens.studio/manage-tokens/token-types/typography/font-family
- uitzoeken waarom de
"components/button/utrecht"tokens er nog in staan nu de NL-Button candidate is - marges baseren op wat we nu hebben in T-shirt sizes https://github.com/maykinmedia/open-inwoner-design-tokens/blob/main/src/brand/openinwoner/layout.tokens.json
- kijk strakker naar de line-heights etc. in open-inwoner zelf: https://github.com/maykinmedia/open-inwoner/blob/develop/src/open_inwoner/scss/views/App.scss als een tekst font-size: 16px heeft en line-height: 150%, dan is de regelafstand 16px × 1.5 = 24px. Misschien kan Style-Dictionary niet werken met 'unitless' waarden.
- beslis welke grijstinten we weg gooien of juist meer willen gebruiken etc. https://github.com/maykinmedia/open-inwoner/blob/a402c98314bc63f64da3c328f610d2926ad55eab/src/open_inwoner/scss/views/App.scss#L32
- de max inline size van OIP is 1024 dus niet degene die NLDS gebruikt:
"max-inline-size": {
"$type": "dimension",
"$value": "1440px"
},
| "default": { | ||
| "$type": "fontFamilies", | ||
| "$value": "IBM Plex Sans, sans-serif" | ||
| "$value": "Lato" |
Collaborator
There was a problem hiding this comment.
ToDo: zoek uit of hier een waarde in kan worden ingevuld die in FIgma geen bestaand font is. Open Inwoner gebruikt hier het woord "Body" in plaats van Lato - omdat we geen font vast mogen zetten, zodat gemeente hun eigen fonts kunnen uploaden.
| }, | ||
| "md": { | ||
| "$type": "lineHeights", | ||
| "$value": "150%" |
Collaborator
There was a problem hiding this comment.
Moet line-height in percentages? In CSS moet dat in vaste "unitless" notatie, dus 1,5 in plaats van 150%. Maar misschien kan style-dictionary dat niet verwerken.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
changing the 'Voorbeeld' theme to OIP theme.