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 formulierenrichtlijnen: links in een formulier #375

Merged
merged 3 commits into from
Nov 24, 2023

Conversation

rianrietveld
Copy link
Contributor

No description provided.

Copy link

vercel bot commented Nov 17, 2023

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

Name Status Preview Comments Updated (UTC)
documentatie ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 24, 2023 9:35am

@rianrietveld
Copy link
Contributor Author

rianrietveld commented Nov 17, 2023

@hidde
Hierbij de links binnen een formulier pagina.
Om even wat te proberen heb ik blokjes visual/code toegevoegd met inline style, zonder een React component.
Het enige wat we dat hoeven doen is wat CSS toevoegen aan het hoofd stylesheet.
Het staat nu in inline code, voordat ik de CSS ga aanpassen (het moet natuurlijk in een class) en ik wil ook je mening weten of dit zo een goede oplossing is.
Screenshot codevoorbeeld

Ik ben geen designer, maar wellicht kan het design met CSS wat mooier worden.
Ook heb ik een icoontje als SVG code toegevoegd, dat kan ook beter.

Maar anyhow. Ik denk dat dit een goede middenweg is om te voorkomen dat we React nodig hebben in de snippits.

Ook had ik nog de vraag hoe we afbeeldingen toevoegen, ik heb er nu een screenshot in die de formulierenndirectory staat.

@rianrietveld
Copy link
Contributor Author

rianrietveld commented Nov 22, 2023

@hidde De eerste richtlijn in de nieuwe structuur :-)
Ik heb bij het menu item (beta) gezet en een verhaaltje op de overzichtspagina gezet.

Ik zag nog de volgende zaken aan de guidelines component:

  • CSS .nlds-guideline__badge--dont en .nlds-guideline__badge--dont gaat niet goed, kleur wordt niet overnenomen. utrecht-paragraph overschrijf deze css
  • er staat geen aria-hidden op de icoontjes
  • de tekst op de toggle verberg code verdwijnt
    Apart issue van maken?

@hidde
Copy link
Member

hidde commented Nov 22, 2023

Puntje:

Gisteren leerde ik dat we afbeeldingen plaatsen in de assets branch, je kunt ze hier (klik hier!!) uploaden en dan pad in de guideline aanpassen naar https://raw.githubusercontent.com/nl-design-system/documentatie/assets/BESTANDSNAAM, bv https://raw.githubusercontent.com/nl-design-system/documentatie/assets/_img_links-vo-link-in-label.png

Copy link
Member

@Robbert Robbert left a comment

Choose a reason for hiding this comment

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

Pakt heel mooi uit zo! Ik wordt echt heel blij van dit soort documentatie, bedankt! 🫶


<Guideline appearance="do" title="Zichtbaar icoontje plus onzichtbare, maar uitgesproken tekst voor screenreaders.">
<Canvas language="html">
<a href="#" target="_blank" class="icon-ext">
Copy link
Member

Choose a reason for hiding this comment

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

"icon-ext" zou ik liever niet doen, ik zou <svg>...</svg> adviseren vanwege beter resultaat in forced colors mode.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Robbert Dat is niet handig voor plugins of CMSssen. Dan moet de content manager handmatig de SVG code invoeren.
Nu kan het zelfs eventueel met de CSS a[target="_blank"]

@@ -0,0 +1,40 @@
---
title: Links in een formulier | Richtlijnen NL Design System
Copy link
Member

Choose a reason for hiding this comment

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

hier zou een | tussenmoeten voor extra consistentie

Suggested change
title: Links in een formulier | Richtlijnen NL Design System
title: Links in een formulier | Richtlijnen | NL Design System

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@hidde 👍

@rianrietveld
Copy link
Contributor Author

@hidde @Robbert
Ik heb onderaan de pagina tekst met een link geplaatst:
Deze richtlijnen worden onderhouden door het NL Design System. Heb je aanvullingen of opmerkingen? Open een issue op GitHub met je suggesties.

Ik kan geen Issue template aanmaken, wellicht dat een van juillie dat kan verzorgen.

@Robbert Robbert self-requested a review November 24, 2023 09:41
@Robbert Robbert merged commit 70f2994 into main Nov 24, 2023
6 checks passed
@Robbert Robbert deleted the add-links-in-form branch November 24, 2023 09:46
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.

3 participants