NHSRplotthedots icon indicating copy to clipboard operation
NHSRplotthedots copied to clipboard

Published 20 hours ago • verified publisher icon nhs-r-community

Accessibility - change "here" to more descriptive link

Open Lextuga007 opened this issue 2 years ago • 1 comments

Following the thread https://twitter.com/Em1lySm/status/1516741011422199810 it's advisable to not just use "here" as a link for accessibility.

https://github.com/nhs-r-community/NHSRplotthedots/blob/5a4af03caff10a7ea6b8027ca72feae4b1003584/README.md https://github.com/nhs-r-community/NHSRplotthedots/blob/ca67eae34ae3aa228415d6ba0ef861da272cf840/vignettes/intro.Rmd - this says [mdc] but preferably should be written out in full to Making Data Count as mdc is not a recognised universal acronym (yet!).

Lextuga007 avatar Apr 21 '22 08:04 Lextuga007

I would suggest those instances are converted to

['Making Data Count' programme][mdc].

tomjemmett avatar Apr 21 '22 11:04 tomjemmett

@Lextuga007 - can this be closed?

tomjemmett avatar Jul 14 '23 09:07 tomjemmett

Looks like we still have a "Please see here" https://github.com/nhs-r-community/NHSRplotthedots/blob/bcf5f8cdb9fc74f3c2b30fb489973fce1fa58e71/vignettes/intro.Rmd#L17 so I'll fix that before closing.

Lextuga007 avatar Jul 14 '23 09:07 Lextuga007

oh, before changing that, wait for #176 to merge

tomjemmett avatar Jul 14 '23 09:07 tomjemmett

or, at least start your PR off of #176 to begin with. Otherwise, you are going to be in merge conflict hell :-)

tomjemmett avatar Jul 14 '23 09:07 tomjemmett