Closed Bug 670870 Opened 13 years ago Closed 13 years ago

HTML table and caption should be linked by label relations

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla8
Tracking Flags:
Tracking Status
firefox7 --- fixed

People

(Reporter: surkov, Assigned: surkov)

References

Details

(Keywords: access, Whiteboard: [qa-])

Attachments

(1 file)

patch
(deleted), patch
MarcoZ
: review+
asa
: approval-mozilla-aurora+
Details | Diff | Splinter Review
Attached patch patch (deleted) — — Splinter Review
spun off bug 666212 where we started to expose caption as name (against description). That means caption and table should be linked by labelled_by/label_for relations (not description_for/described_by).
Attachment #545330 - Flags: review?(marco.zehe)
Comment on attachment 545330 [details] [diff] [review] patch Ah, thanks for fixing this Alex! :)
Attachment #545330 - Flags: review?(marco.zehe) → review+
landed http://hg.mozilla.org/mozilla-central/rev/4274ccb37cf6 Marco, could you drive to port the patch into mozilla 7 since this bug and bug 666212 should be distributed together?
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
Target Milestone: --- → mozilla8
Comment on attachment 545330 [details] [diff] [review] patch Requesting approval for Mozilla-aurora since this bug completes a feature implemented for Mozilla7. Zero risk, has tests.
Attachment #545330 - Flags: approval-mozilla-aurora?
Attachment #545330 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Landed on Aurora for Firefox 7: http://hg.mozilla.org/releases/mozilla-aurora/rev/4d2a4e9e9730
status-firefox7: --- → fixed
Version: unspecified → Trunk
(In reply to comment #4) > Landed on Aurora for Firefox 7: > http://hg.mozilla.org/releases/mozilla-aurora/rev/4d2a4e9e9730 Thanks, Marco!
Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0 Can anyone please help me with a test case or with STR/guidelines in order to get this issue checked on QA side? Thanks!
(In reply to Simona B from comment #6) > Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0 > > Can anyone please help me with a test case or with STR/guidelines in order > to get this issue checked on QA side? > Thanks! The patch has mochitests so maybe it's not necessary to check it manually. But if you think it's necessary then it makes sense to ask Marco how can it be checked with screen readers or ask him to do that since he's accessibility QA.
qa- as no QA fix verification needed
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: