Skip to content

Conversation

@veslav3
Copy link
Contributor

@veslav3 veslav3 commented Nov 19, 2025

No description provided.

@veslav3 veslav3 requested a review from a team as a code owner November 19, 2025 13:16
@changeset-bot
Copy link

changeset-bot bot commented Nov 19, 2025

⚠️ No Changeset found

Latest commit: bdec6e3

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented Nov 19, 2025

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

Project Deployment Review Updated (UTC)
documentatie Ready Ready Preview, Comment Dec 17, 2025 1:59pm

…ntsource installation via NPM and best practices for font usage.
@sonarqubecloud
Copy link

<link rel="stylesheet" href="<jouw-thema>.css" />
```

Mocht je nog geen eigen thema hebben, dan kun je een thema maken op basis van het [start-thema](https://nldesignsystem.nl/handboek/huisstijl/themas/start-thema/). Het [voorbeeld-thema](https://nldesignsystem.nl/handboek/huisstijl/themas/voorbeeld-thema/#voorbeeld-thema) is hierop gebaseerd en dient als voorbeeld hoe je dit ook zou kunnen bereiken met je eigen thema.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goede verwijzing naar documentatie, maar het blijft abstract wanneer je welk thema kiest (start vs voorbeeld). Gebruik het start thema als je vanaf nul begint, en het voorbeeld thema als je een compleet uitgewerkt referentiepunt wilt


Daarom is het beter om fonts lokaal te hosten of via een NPM package te installeren, mits je NPM gebruikt.

Soms worden fonts bijgewerkt, daar moet je ook op voorbereid zijn.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verbind dit expliciet aan versioning of testing, bijvoorbeeld: “Door versiebeheer (zoals via NPM) voorkom je onverwachte wijzigingen in productie.”

import "@fontsource/fira-sans/700.css"; // Bold
```

3. Gebruik het font in je CSS variabelen (in je thema configuratie):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Geef aan of --font-family-base een NLDS token is of een eigen variabele, en hoe dit doorwerkt in componenten.

- **Google Translate**: Als een gebruiker de pagina vertaalt, kunnen er karakters nodig zijn die niet in je standaard subset zitten.
- **Internationale tekens**: Voor talen als Japans worden karakters vaak pas geladen als ze nodig zijn. Door de juiste definities te hebben, zorg je voor een goede weergave in alle scenario's.

## Heb je feedback?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Misschien nog uitleg over token naming & overrides?

- **Google Translate**: Als een gebruiker de pagina vertaalt, kunnen er karakters nodig zijn die niet in je standaard subset zitten.
- **Internationale tekens**: Voor talen als Japans worden karakters vaak pas geladen als ze nodig zijn. Door de juiste definities te hebben, zorg je voor een goede weergave in alle scenario's.

## Heb je feedback?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Het document zegt wat je moet laden, maar niet in welke volgorde en waarom dat belangrijk is?

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.

Eigen thema gebruiken en font installeren documentatie voor developers

3 participants