Last Comment Bug 982125 - make HTML5 <mark> accessible
: make HTML5 <mark> accessible
Status: ASSIGNED
:
Product: Core
Classification: Components
Component: Disability Access APIs (show other bugs)
: unspecified
: x86_64 Windows 8.1
-- normal (vote)
: ---
Assigned To: Marco Zehe (:MarcoZ)
:
: alexander :surkov
Mentors:
Depends on:
Blocks: html5a11y
  Show dependency treegraph
 
Reported: 2014-03-11 08:37 PDT by steve faulkner
Modified: 2017-04-28 07:55 PDT (History)
9 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch (3.46 KB, patch)
2015-06-03 07:47 PDT, Marco Zehe (:MarcoZ)
no flags Details | Diff | Splinter Review

Description User image steve faulkner 2014-03-11 08:37:40 PDT
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 User image alexander :surkov 2014-03-12 07:57:12 PDT
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?
Comment 2 User image James Teh [:Jamie] 2014-08-07 22:03:06 PDT
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 User image alexander :surkov 2014-08-11 13:21:47 PDT
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 :)
Comment 4 User image steve faulkner 2015-05-31 08:15:07 PDT
(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 :-)
Comment 5 User image steve faulkner 2015-05-31 09:00:17 PDT
Have updated implementation info for <mark> for review
Comment 6 User image steve faulkner 2015-05-31 09:02:00 PDT
(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 User image Joanmarie Diggs 2015-05-31 10:14:30 PDT
(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!
Comment 8 User image Marco Zehe (:MarcoZ) 2015-06-03 07:47:51 PDT
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?
Comment 9 User image alexander :surkov 2015-06-03 09:23:21 PDT
(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 User image alexander :surkov 2015-06-03 09:27:27 PDT
(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 User image alexander :surkov 2015-06-03 09:28:19 PDT
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.
Comment 12 User image steve faulkner 2016-07-15 03:43:28 PDT
any update on this?
Comment 13 User image alexander :surkov 2016-07-15 05:15:58 PDT
(In reply to steve faulkner from comment #12)
> any update on this?

Steve, do you have ideas on comment #9 (#9, #9, #9 ;) )?
Comment 14 User image David Bolter [:davidb] 2017-04-28 07:55:34 PDT
Spec says to map to xml-roles:mark

Note You need to log in before you can comment on or make changes to this bug.