Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentatie: Changelogs voor designers #1076

Open
2 of 3 tasks
Robbert opened this issue Jul 2, 2024 · 0 comments
Open
2 of 3 tasks

Documentatie: Changelogs voor designers #1076

Robbert opened this issue Jul 2, 2024 · 0 comments
Assignees

Comments

@Robbert
Copy link
Member

Robbert commented Jul 2, 2024

Specificatie

  • Misschien kan de template in de documentatie staan als codeblock
  • Misschien kan de template in een .changesets/major.md.template staan, met wat inline uitleg over de change ✅
  • Uitzoeken: Gaan comments in een changeset markdown mee met de changelog? Indien niet, dan kunnen we comments in de template plaatsen ❌ .

Acceptatie-criteria

  • Er is documentatie op nldesignsystem.nl onder de "Handboek" ➝ Designers categorie
  • Uitleg over breaking changes voor design tokens naamgeving
  • Is een design beslissing wijzigen een breaking change, of een feature?
  • Tip: geef informatie hoe designers kunnen migreren, bijvoorbeeld: token X is nu token Y geworden, rename
  • Tip geven over backwards compatibility van een theme. Bijvoorbeeld: example.button.icon.space wordt example.button.column-gap. Dan kun je dit maken:
{
  example: {
    button: {
      'column-gap': { value: '8px' },
      icon: {
        space: { value: '{example.button.column-gap}' },
      },
    },
  },
}
  • Uitleg dat de versie niet automatisch wordt gepubliceerd, je moet nog de automatisch aangemaakte PR nog mergen.
  • Uitleg wat het effect is als oude software een nieuwe theme gebruikt terwijl er breaking changes zijn. Bijvoorbeeld: de button verliest de ontworpen icon space, die krijgt de default waarde voor de icon space (als die er is)
  • We bieden een template .md aan voor major, minor en patch changes.
  • Designers moeten kunnen weten welke npm package ze gebruiken.
  • README.md in .changesets/ van themes repo met in elk geval een linkje naar de documentatie op de website
  • Met de designers even een plan maken voor specifieke aanwijzingen voor wijzigen van brand tokens en common tokens.
    • Common tokens en brand tokens worden in de praktijk vaak gebruikt door front-end developers in de CSS, voor custom CSS. Bijvoorbeeld, om een margin in een bepaalde situatie in te stellen. Daarom is het handig om als migratie-stap aan te bieden dat front-end developers bepaalde CSS variables moeten renamen. Bijvoorbeeld: "vervang de volgende tokens: var(--example-space-block-elephant)var(--example-space-block-xl2).
  • Misschien ook leuk: een screenshot van GitHub code search voor design tokens.

Testen

  • @jeffreylauwers en @Rozerinay hebben het document toegepast om een changelog entry te maken in de themes repository, en ze zijn tevreden!
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

No branches or pull requests

2 participants