make HTML5 <mark> accessible

ASSIGNED
Assigned to

Status

()

Core
Disability Access APIs
ASSIGNED
4 years ago
4 months ago

People

(Reporter: steve faulkner, Assigned: MarcoZ)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
unsure exactly how but be good to expose something via acc API to indicate marked text
http://www.w3.org/html/wg/drafts/html/master/text-level-semantics.html#the-mark-element

Comment 1

4 years ago
I'd say it makes sense to expose elements like HTML mark, strong, em and others as text attributes to expose their semantics to AT (in contrast to as we do that now: expose them as bunch of style text attributes).

Jamie, do you ever need to be aware of these elements presence?

Updated

4 years ago
Blocks: 389237

Comment 2

3 years ago
Related NVDA issue ticket: http://community.nvda-project.org/ticket/4247

We've had a request to support this in NVDA. It's a bit controversial because even though strong has semantic meaning where bold doesn't, most people just think of it as bold anyway and bold kind of has semantic meaning to most people anyway. However, I do think that mark is more semantically important than strong, and I don't think we should expect users to understand colour changes. Therefore, I'm leaning towards semantically indicating mark but not strong or em.

Comment 3

3 years ago
I'm fine to go with mark only for now, it's fresh and relatively new and hopefully the community is able to prevent it when it's used for styling :)
(Reporter)

Comment 4

2 years ago
(In reply to alexander :surkov from comment #3)
> I'm fine to go with mark only for now, it's fresh and relatively new and
> hopefully the community is able to prevent it when it's used for styling :)

Follwoing up on this as nothing appears to have happened :-)
(Reporter)

Comment 5

2 years ago
Have updated implementation info for <mark> for review
(Reporter)

Comment 6

2 years ago
(In reply to steve faulkner from comment #5)
> Have updated implementation info for <mark> for review

and the link is...
http://w3c.github.io/aria/html-aam/html-aam.html#el-mark

Comment 7

2 years ago
(In reply to steve faulkner from comment #6)
> (In reply to steve faulkner from comment #5)
> > Have updated implementation info for <mark> for review
> 
> and the link is...
> http://w3c.github.io/aria/html-aam/html-aam.html#el-mark

I have to give some thought to what the support in Orca will look like, but the mapping you've provided makes sense for ATK/AT-SPI2. Thanks!
(Assignee)

Comment 8

2 years ago
Created attachment 8614689 [details] [diff] [review]
Patch

For the text attribute test change, I get the following entry with a test failure:

2536 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/tests/mochitest/elm/test_HTMLSpec.html | Wrong start offset for [xpconnect wrapped (nsISupports, nsIAccessible, nsIAccessibleText)] at offset 0 
2537 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/tests/mochitest/elm/test_HTMLSpec.html | Wrong end offset for [xpconnect wrapped (nsISupports, nsIAccessible, nsIAccessibleText)] at offset 0 
2538 INFO TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/tests/mochitest/elm/test_HTMLSpec.html | There is no expected attribute 'background-color'  for [xpconnect wrapped (nsISupports, nsIAccessible, nsIAccessibleText)] at offset 0 - expected PASS
2539 INFO TEST-PASS | chrome://mochitests/content/a11y/accessible/tests/mochitest/elm/test_HTMLSpec.html | Attribute 'background-color' has wrong value for [xpconnect wrapped (nsISupports, nsIAccessible, nsIAccessibleText)] at offset 0 

Alex, do you have any idea what's wrong, why this fails on the attribute's presence, but then checks the background color correctly?
Assignee: nobody → mzehe
Status: NEW → ASSIGNED
Attachment #8614689 - Flags: review?(surkov.alexander)

Comment 9

2 years ago
(In reply to Joanmarie Diggs from comment #7)
> (In reply to steve faulkner from comment #6)
> > (In reply to steve faulkner from comment #5)
> > > Have updated implementation info for <mark> for review
> > 
> > and the link is...
> > http://w3c.github.io/aria/html-aam/html-aam.html#el-mark
> 
> I have to give some thought to what the support in Orca will look like, but
> the mapping you've provided makes sense for ATK/AT-SPI2. Thanks!

Do we have to have an accessible object for it. Wouldn't it be enough to expose 'mark:true' text attribute like we do for HTML sub and sup elements?

Comment 10

2 years ago
(In reply to Marco Zehe (:MarcoZ) from comment #8)

> 2539 INFO TEST-PASS |
> chrome://mochitests/content/a11y/accessible/tests/mochitest/elm/
> test_HTMLSpec.html | Attribute 'background-color' has wrong value for
> [xpconnect wrapped (nsISupports, nsIAccessible, nsIAccessibleText)] at
> offset 0 
> 
> Alex, do you have any idea what's wrong, why this fails on the attribute's
> presence, but then checks the background color correctly?

I think this color is default for the mark element and thus it's not exposed at 0 offset.

Comment 11

2 years ago
Comment on attachment 8614689 [details] [diff] [review]
Patch

I'm not sure if we have to have accessible object for the mark element, so cancelling review for now.
Attachment #8614689 - Flags: review?(surkov.alexander)
(Reporter)

Comment 12

a year ago
any update on this?

Comment 13

a year ago
(In reply to steve faulkner from comment #12)
> any update on this?

Steve, do you have ideas on comment #9 (#9, #9, #9 ;) )?
Spec says to map to xml-roles:mark
You need to log in before you can comment on or make changes to this bug.