On the <input> element - changed display: none; to class="visually-hidden" and added focus to <label> element.
(Utilizing opacity: 0 does not take the element out of the flow and thus caused spacing issues)
This issue is not related to a particular fix#
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
[x] PR is descriptively titled 📑 and links the original issue above 🔗
[x] tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
[x] code is in uniquely-named feature branch and has no merge conflicts 📁
[x] screenshots/GIFs are attached 📎 in case of UI updation
[x] ask @publiclab/reviewers for help, in a comment below
@jywarren, @TildaDares, @cesswairimu -
Correcting this accessibility issue has helped me to identify other areas that need A11y attention. I will add these to my planning issues.
On the
<input>
element - changeddisplay: none;
toclass="visually-hidden"
and added focus to<label>
element. (Utilizingopacity: 0
does not take the element out of the flow and thus caused spacing issues)This issue is not related to a particular fix# Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
@publiclab/reviewers
for help, in a comment below