dso-toolkit / dso-toolkit

DSO Toolkit
http://www.dso-toolkit.nl
24 stars 18 forks source link

Anchor: Ondersteuning voor `tel:` en `mailto:` #2351

Closed MYpenburg closed 10 months ago

MYpenburg commented 1 year ago

tel: link De link alleen (non-breaking) laten voorafgaan door een 'call' icoon in de kleur van de lopende tekst. Geen 'external-link' aan het eind

mailto: link De link alleen (non-breaking) laten voorafgaan door een 'email' icoon in de kleur van de lopende tekst. Geen 'external-link' aan het eind

anneke commented 1 year ago

@MYpenburg Welke tekst moeten we voorlezen als iemand een screenreader gebruikt? En zijn hier al icoontjes voor beschikbaar?

MYpenburg commented 1 year ago

De iconen staan al in de toolkit onder de naam tussen quotes. De externe link heeft natuurlijk de "opent in een nieuw venster" toevoeging, maar die hoeft hier in principe niet als mensen doorhebben dat het een nummer of een mailadres is. Misschien kunnen Grimm & Tim iets zeggen over het correcte voorleesgedrag van mailto: en tel: links? (@timveld @hansgrimm).

hansgrimm commented 1 year ago

De iconen staan al in de toolkit onder de naam tussen quotes. De externe link heeft natuurlijk de "opent in een nieuw venster" toevoeging, maar die hoeft hier in principe niet als mensen doorhebben dat het een nummer of een mailadres is. Misschien kunnen Grimm & Tim iets zeggen over het correcte voorleesgedrag van mailto: en tel: links? (@timveld @hansgrimm).

Die worden voor zover ik weet correct door de screenreaders herkend en voorgelezen.

anneke commented 1 year ago

VoiceOver op Mac met Safari leest niet voor wat voor soort link het is. (Heb het net getest). Er wordt alleen "link" voorgelezen.

tfrijsewijk commented 1 year ago

@anneke Ik zag een branch met dit issuenummer dus ben zo vrij geweest dit issue in progress in de sprint te zetten 🙏

tfrijsewijk commented 10 months ago

@hansgrimm Wil jij naar dit issue kijken? De branch is rebased op master