Firefox - Does not report map tag info correctly

RESOLVED FIXED in mozilla10

Status

()

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

People

(Reporter: Frank DiPalermo, Assigned: surkov)

Tracking

(Blocks: 1 bug)

Trunk
mozilla10
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
User-Agent:       Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; Tablet PC 2.0; MALC)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.13) Gecko/20101203 Firefox/3.6.13

To Reproduce:
1. Open the URL.

Note that the map tag that encloses the two page tabs (Biblio and Help) is not exposed to assistive technology.

Reproducible: Always
Isn't this incorrect use of html? Not sure whether we should do anything here -- at least not for FF4. We assume maps are for image maps.

Updated

6 years ago
Component: Disability Access → Disability Access APIs
Product: Firefox → Core
QA Contact: disability.access → accessibility-apis
Version: unspecified → Trunk

Comment 2

6 years ago
This seems to have regressed. I remember bug 431615 having fixed this once. We also have test coverage, or at least I thought we had, but for some reason, this got lost at some point. We do get an accessible for the links map, but we don't test the links underneath anymore in our mochitests.
(Reporter)

Comment 3

6 years ago
Yes, this is probably a poor use of the map tag, but doesn't it still point out a FF problem?
(Assignee)

Updated

6 years ago
Blocks: 540399
(Assignee)

Updated

6 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 4

6 years ago
Created attachment 563974 [details] [diff] [review]
patch
Assignee: nobody → surkov.alexander
Status: NEW → ASSIGNED
Attachment #563974 - Flags: review?(marco.zehe)
(Assignee)

Updated

6 years ago
OS: Windows 7 → All
Hardware: x86 → All

Comment 5

6 years ago
Comment on attachment 563974 [details] [diff] [review]
patch

r=me with one nit:

>+    // be used in conjuction with HTML image element and in this case we don't

typo: Conjunction (a missing n).

Also I wonder whether we should combine the test from tests/hyperlink/test_general.html into this one, too. There, we also only test if the links get created at all inside the map. What do you think?
Attachment #563974 - Flags: review?(marco.zehe) → review+
(Assignee)

Comment 6

6 years ago
(In reply to Marco Zehe (:MarcoZ) from comment #5)

> Also I wonder whether we should combine the test from
> tests/hyperlink/test_general.html into this one, too. There, we also only
> test if the links get created at all inside the map. What do you think?

map used for grouping is just generic hypertext accessible. All we need is to make sure accessibles for map and its children are created in this case. Hyperlink tests aren't necessary I think because don't test new paths of code logic.
(Assignee)

Comment 7

6 years ago
inbound land http://hg.mozilla.org/integration/mozilla-inbound/rev/628c7f04dbbc
https://hg.mozilla.org/mozilla-central/rev/628c7f04dbbc
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
(Assignee)

Updated

6 years ago
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.