Create namerules based tests for accessible name of images.

RESOLVED FIXED in mozilla12

Status

()

Core
Disability Access APIs
RESOLVED FIXED
9 years ago
6 years ago

People

(Reporter: MarcoZ, Assigned: MarcoZ)

Tracking

({access})

Trunk
mozilla12
access
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

9 years ago
Spun off bug 476796. The namerules architecture currently does not allow alt=  to be defined, which is used to specifically denote a decorative image, and whose accessible name must also be testable.
(Assignee)

Comment 1

9 years ago
Created attachment 360687 [details] [diff] [review]
Patch

Turns out that I could test this with two sets of rules and markup.
Assignee: nobody → marco.zehe
Status: NEW → ASSIGNED
Attachment #360687 - Flags: review?(surkov.alexander)
(Assignee)

Comment 2

9 years ago
Created attachment 363103 [details] [diff] [review]
Patch updated to trunk

The patch no longer applied cleanly to trunk. Updated.
Attachment #360687 - Attachment is obsolete: true
Attachment #363103 - Flags: review?(surkov.alexander)
Attachment #360687 - Flags: review?(surkov.alexander)

Comment 3

9 years ago
Marco, thank you for keeping it up to dated.

I would prefer actually to introduce markup support for this special cases. Your way works but it might be not easy understandable. I'll think how to extend our tests to test null names.
ping? (going through surkov's reviews and found this)

Comment 5

6 years ago
Comment on attachment 363103 [details] [diff] [review]
Patch updated to trunk

I've never got around to think how to improve this. I think we can take the patch as it is.
Attachment #363103 - Flags: review?(surkov.alexander) → review+
(Assignee)

Comment 6

6 years ago
Inbound land: http://hg.mozilla.org/integration/mozilla-inbound/rev/e170001f3e9b
Flags: in-testsuite+
Target Milestone: --- → mozilla12

Comment 7

6 years ago
https://hg.mozilla.org/mozilla-central/rev/e170001f3e9b
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.