Giphy / giphy-react-native-sdk

GIPHY React Native SDK
https://developers.giphy.com
Apache License 2.0
69 stars 25 forks source link

Improve accessibility labelling #113

Closed oherst closed 1 year ago

oherst commented 1 year ago

Duplicates

Latest version

Summary 💡

All buttons and images (or gifs) should have appropriate accessibility labelling.

Motivation 🔦

Hello! I believe this to be an enhancement rather than a bug so I figured a feature request would be more appropriate but I can make this request somewhere else if this is not the right place.

We are using the Giphy RN sdk in our app and have recently been audited by a third party for accessibility. They picked up some issues with our Giphy implementation and I wondered if you might be able to give your thoughts on them?

Additionally I note that the buttons at the bottom of the giphy main window are read as "sticker underscore type underscore button, button" which could be better (eg, sticker button). To be clear, this was not picked up by the auditors it is just something I noticed whilst testing.

Would this be something you would consider for a future enhancement?

Thanks very much.

Screenshot 2023-06-05 at 14 44 18 Screenshot 2023-06-05 at 14 44 13 Screenshot 2023-06-05 at 14 44 47 IMG_2C41A574B9CA-1

pshoniuk commented 1 year ago

Hi @oherst Thanks for the detailed report. I’ll pass this feedback on to my team & keep you in the loop.

pshoniuk commented 1 year ago

@oherst We released v2.4.1 with improved accessibility labels. If some of the accessibility issues mentioned here aren't resolved in v2.4.1, please reopen this issue or create a new one.

efstathiosntonas commented 1 year ago

thank you for this guys

oherst commented 1 year ago

thanks so much, can't wait to try it out!