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.
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
docs: add designer handboek "breaking changes" #1151
base: main
Are you sure you want to change the base?
docs: add designer handboek "breaking changes" #1151
Changes from all commits
6c737d2
2591605
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik zou een komma toevoegen na het woord 'wordt' in de 2e zin, dus:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik zou de zin veranderen naar:
Heb er ook een interpunctie aan toegevoegd.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou van de laatste zin maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bedoel je met 'in de regel' doorgaans? Of iets code specifieks?
In het geval van 'doorgaans' zou ik dit letterlijk zo noemen. Of misschien liever 'meestal' (dit is b1).
Ik ging er namelijk vanuit dat je het over een regeltje code had.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is het een idee om voorafgaand aan deze zin een kopje te plaatsen zodat deze zin 'los' komt te staan van de tekst over 'Patch'? Suggestie:
Wat kun je met een versienummer?
Een versienummer alleen zegt niet zo veel. Maar door versienummers met elkaar te vergelijken...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Zoals' toegevoegd:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kunnen we de laatste alinea bovenaan zetten? Ik merk dat die herhaling aan het begin fijner is om de context eronder beter te snappen. Dus:
Als je deze structuur aanhoudt, kun je mijn comment hierboven over de 1e alinea negeren.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik vraag mij nu ineens af: Is een aanpassing in de waarde van een token iets wat we hier ergens willen benoemen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ja… zouden we voorbeelden van waarde veranderingen voor zowel major, minor, als patch versie bumps kunnen bedenken?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lijkt me heel goed om daar voorbeelden voor te bedenken.
Kunnen we later misschien een tabel of lijst of iets anders opstellen en deze categoriseren op basis van major, minor en patch changes?
Op die manier kunnen gebruikers snel zien welke impact verschillende updates kunnen hebben op hun code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'...de ruimte tussen het icoon en de tekst van een button'
Zouden we 'tekst' kunnen inwisselen voor 'label'? Dat is hoe de 'tekst' wordt genoemd in het button component.
Dat gezegd hebbende, we zijn bezig met het noteren van een 'Doel' per componenten. En daar hebben we het ook over 'Knop'. Dus misschien is 'de ruimte tussen het icoon en de tekst van een knop' toch wel weer beter?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou een kopje 'Design token van naam veranderen' hier handig zijn?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Het zijn maar voorbeelden, ik denk dat kopjes een eindige lijst zou suggereren en dat is het niet per se.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nieuwe* design token.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Als we uitgaan van knop i.p.v. button, dan moeten we dat woord hier ook nog aanpassen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou van de laatste zin maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Token Studio komt hier een beetje uit de lucht vallen. Misschien kunnen we linken naar een andere pagina in het Handboek voor designers? Bijvoorbeeld hierheen? Mocht iemand dan nog niks van Token Studio weten zijn ze er weer bij.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ik ga vandaag met Robbert praten want blijkbaar is Tokens Studio niet de enige plek waar waarheden over design tokens leven dus ik denk dat er nog wat meer gesleuteld moet worden.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hier kan misschien nog iets van een intro bij? Voorzet:
Jij weet wat je verandert hebt. De afnemer van de design tokens niet. Het is dus wel zo netjes om afnemers te informeren over veranderingen. We leggen je uit hoe je deze communicatie kan aanpakken.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Goed idee. Zou daarvan maken:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken:
Wanneer je een pull request hebt aangemaakt, geef je met een extra commit aan wat je precies hebt veranderd. De beschrijving die je hierbij opgeeft, wordt opgenomen in de changelog van het package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Misschien de stappen even introduceren?
Situatie A: NL Design System “themes” repository
Werk je vanuit de NL Design System “themes” repository? Volg dan onderstaande stappen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bovenaan* de pagina
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En dan hier ook?
Situatie B: Eigen repository
Werk je vanuit een eigen repository? Volg dan onderstaande stappen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dezelfde tekstuele wijzigingen als hierboven :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zou hiervan maken: