Closed Bug 1030377 Opened 8 years ago Closed 8 years ago

Search clear button is not labeled for accessibility

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: eeejay, Assigned: mancas, Mentored)

References

Details

(Keywords: access, Whiteboard: [b2ga11y p=1][good first bug])

Attachments

(1 file)

When typing in something in the search field, a button with an x shows up. It is not labeled.
Re-aligning priorities with 2.1 accessibility goals.
Whiteboard: [b2ga11y p=1]
Whiteboard: [b2ga11y p=1] → --do_not_change-- [good first bug]
Whiteboard: --do_not_change-- [good first bug] → [b2ga11y p=1][good first bug]
Mentor: yzenevich, eitan
Assignee: nobody → b.mcb
Hello Eitan i've been trying to contact you by IRC but it seems you are afk. I'm new member of Telefonica I+D and I've picked this bug to learn the internal workflow.
Attachment #8452994 - Flags: review?(dale)
Comment on attachment 8452994 [details] [review]
Localizing the aria-label attribute

This looks good to me, going to flag accessibility folks to make sure its following the correct procedures as I dont do much accessibility work before landing
Attachment #8452994 - Flags: review?(dale) → review+
Comment on attachment 8452994 [details] [review]
Localizing the aria-label attribute

Just wanted to make sure this was good by you
Attachment #8452994 - Flags: feedback?(eitan)
Comment on attachment 8452994 [details] [review]
Localizing the aria-label attribute

Eitan is on PTO till Monday. Looks good to me, tested and works nicely with the SR.
Attachment #8452994 - Flags: feedback?(eitan) → feedback+
master: bb08923a6e728b66b45a9039b5a4dcc0a400274e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.