Fix #9: improve accessibility labels for image upload previews - #12
Fix #9: improve accessibility labels for image upload previews#12magnet78 wants to merge 2 commits into
Conversation
immerSIR
left a comment
There was a problem hiding this comment.
Thanks for tackling #9. This still needs a small pass before merge.
CodeRabbit flagged, and I confirmed against the surrounding UI, that the new preview alt text is English and redundant (Image upload preview — selected file preview) while these flows are French. Please switch both preview labels to concise French copy, ideally including the selected file name when available, for example Aperçu de l'image sélectionnée : ${selectedImage.name} with a fallback.
Also, #9 specifically asks that screen readers can distinguish the AI detection upload control from the content-verification upload control. This PR updates only the preview images; please add clear accessible names to the hidden file inputs/buttons in both flows, such as distinct aria-label values that match each page context.
If the existing client test setup can cover it, please add/update a lightweight accessibility-label test; otherwise note why it is not practical in this PR.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Closes #9
Improved accessibility labels for image upload previews in both AI image detection and content verification flows:
Changes:
alt="Aperçu"toalt="Image upload preview — selected file preview"in bothAIImageDetection.jsandImageContentVerification.jsFiles modified:
client/src/components/AIImageDetection.jsclient/src/components/ImageContentVerification.js