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

refactor: add images from new assets branch #191

Merged
merged 2 commits into from
May 23, 2024

Conversation

Yolijn
Copy link
Member

@Yolijn Yolijn commented May 22, 2024

Voor het toevoegen van plaatjes op een plek waar we controle over hebben gebruiken we bij nldesignsystem.nl een losse assets branch.
Met de GitHub UI kun je daar plaatjes uploaden die je vervolgens kan gebruiken in markdown. Hieronder de beschrijving van hoe dit werkt, zoals deze ook toegevoegd is in de README.md

Screenshot 2024-05-23 at 01 12 18

Plaatjes toevoegen

Plaatjes zijn heel groot om via GitHub heen en weer te sturen. Daarom gebruiken we de assets branch om plaatjes te uploaden. Deze kunnen vervolgens als plaatje in markdown worden gezet, zoals bijvoorbeeld:

![Plaatje van een browser met waar de Google Translate pop-up getoond wordt om de pagina te gaan vertalen.](https://raw.githubusercontent.com/nl-design-system/gebruikersonderzoeken/assets/utrecht-google-translate.png)

Voor de naam van het bestand gebruik je <naam van het markdown bestand>__<plek in de markdown> bijvoorbeeld:

![Een lijst van contactmomenten met de datum van het contactmoment, de beschrijving van het type contactmoment en titel](https://raw.githubusercontent.com/nl-design-system/gebruikersonderzoeken/assets/denhaag-mijn-omgeving__contactmomenten.png)

Copy link

vercel bot commented May 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gebruikersonderzoeken 🛑 Canceled (Inspect) May 22, 2024 11:09pm

Copy link
Contributor

@JeroenduC JeroenduC left a comment

Choose a reason for hiding this comment

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

Super, fijne beschrijving in de Readme.md, ik ga het op deze manier proberen!

@JeroenduC JeroenduC merged commit f76dfb2 into main May 23, 2024
6 checks passed
@JeroenduC JeroenduC deleted the refactor/add-images-from-assets-branch branch May 23, 2024 07:05
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.

2 participants