Noticed an inconsistency in the use of the term "screen reader" there shouldn't be a '-'.

This is a simple patch.

CommentFileSizeAuthor
#3 proper-use-of-screen-reader-2303547-3.patch4.06 KBmgifford
proper-use-of-screen-reader-0.patch4.33 KBmgifford
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Title: Sorry, screen-reader to screen reader » Consistency - screen-reader to screen reader
Status: Active » Needs review
jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Needs work

Um, this change doesn't look correct:

- *     http://juicystudio.com/article/screen-readers-display-none.php and
+ *     http://juicystudio.com/article/screen readers-display-none.php and

That change was made twice in the patch.

mgifford’s picture

mgifford
he/him
Primary language English
CreditAttribution: mgifford commented
Status: Needs work » Needs review
FileSize
proper-use-of-screen-reader-2303547-3.patch4.06 KB

Arg.. Sorry.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community

That looks better, thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 2eac903 and pushed to 8.x. Thanks!

  • alexpott committed 2eac903 on 8.x 
    Issue #2303547 by mgifford: Fixed Consistency - screen-reader to screen...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.