After some reading about accessibility I would like to recommend removing aria-hidden=true, because it is meant for elements with multiple states (like collapsing items).
|
$output .= '</textarea><textarea id="comment" aria-hidden="true" name="comment" autocomplete="nope" style="padding:0;clip:rect(1px, 1px, 1px, 1px);position:absolute !important;white-space:nowrap;height:1px;width:1px;overflow:hidden;" tabindex="-1"></textarea>'; |
If we want to hide forever it seems that display: none; is sufficient. Maybe we could add hidden as this has good support in modern browsers:
https://caniuse.com/#search=hidden
But it seems that this is necessary to hide content from screen readers:
https://web.wsu.edu/accessibility-guides/hidden-content/
https://developer.paciellogroup.com/blog/2016/01/the-state-of-hidden-content-support-in-2016/
Can you confirm @00travelgirl00 ?
After some reading about accessibility I would like to recommend removing
aria-hidden=true, because it is meant for elements with multiple states (like collapsing items).antispam-bee/antispam_bee.php
Line 1210 in 9c97b84
If we want to hide forever it seems that
display: none;is sufficient. Maybe we could addhiddenas this has good support in modern browsers:https://caniuse.com/#search=hidden
But it seems that this is necessary to hide content from screen readers:
https://web.wsu.edu/accessibility-guides/hidden-content/
https://developer.paciellogroup.com/blog/2016/01/the-state-of-hidden-content-support-in-2016/
Can you confirm @00travelgirl00 ?