<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>22090</bug_id>
          
          <creation_ts>2013-05-18 16:07:15 +0000</creation_ts>
          <short_desc>feedbck from James Craig part 2</short_desc>
          <delta_ts>2014-01-29 22:44:30 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>HTML WG</product>
          <component>Using ARIA in HTML</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows NT</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="steve faulkner">faulkner.steve</reporter>
          <assigned_to name="dmacdona">david100</assigned_to>
          <cc>david100</cc>
    
    <cc>faulkner.steve</cc>
    
    <cc>jcraig</cc>
    
    <cc>mike</cc>
    
    <cc>public-html-bugzilla</cc>
          
          <qa_contact name="This bug has no owner yet - up for the taking">dave.null</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>87878</commentid>
    <comment_count>0</comment_count>
    <who name="steve faulkner">faulkner.steve</who>
    <bug_when>2013-05-18 16:07:15 +0000</bug_when>
    <thetext>Starting with the &quot;Recommended ARIA usage&quot; table:

&gt; dl

Please note that there is no direct ARIA role match for description lists, so it&apos;s in appropriate to override the native role in this case unless the author is retrofitting an improper use of the DL element.

&gt; hgroup

Probably need to remove this.

&gt; iframe

Probably need to add another row for iframe[seamless], where it is okay to override with any general grouping role.

&gt; input type=checkbox

This note is no longer relevant for ARIA 1.0: &quot;it is required that an explicitrole=checkbox is declared &quot;
The host language implicit role is enough without the redundant role attribute.

&gt; input type=number
&gt; input type=range

Should add that it&apos;s okay to use aria-valuetext on these, with or without an explicit role.

&gt; math

Why is it recommended to use the ARIA role here? The ARIA role is mainly useful for math that is not marked up in such a way as to be programmatically determinable. MathML on the other hand, provides enough structure and metadata to determine much more information about the math content. I would say that it is not necessary or recommended to explicitly use a math role on MathML.

&gt; meter

This one is listed as N/A, but I think progressbar is the equivalent role here. I would still list this as NO, though.

&gt; ol

Add &quot;group&quot; to the role list.

&gt; output

Not YES. I&apos;d say it DEPENDS. If you add role=&quot;status&quot;, it will force this to be a broadcasting live region. In some cases, this is fine, but if this is an element whose value changes constantly due to scripted updates rather than explicit user action (like a progressbar), it&apos;d be more appropriate to set aria-live=&quot;off&quot;. Otherwise some AT users are going to find this very distracting, even impossible to use. Even &quot;polite&quot; live regions can be annoying if they are overused.

&gt; progress

Should add that it&apos;s okay to use aria-valuetext on this (e.g. &quot;Step 2 of 10&quot;), with or without an explicit role.

&gt; tbody, thead, tfoot

Change &quot;NONE&quot; to &quot;role=rowgroup&quot;
Change &quot;N/A&quot; to &quot;NO&quot;
Change &quot;these elements&quot; to &quot;these unrendered elements.&quot;

&gt; Text level elements not listed elsewhere

Out of curiousity, why weren&apos;t INS and DEL listed here instead of on their own row?

&gt; Element with an inert attribute

Change &quot;aria-disabled&quot; to &quot;aria-hidden&quot;
Change &quot;NO&quot; to either &quot;YES&quot; or &quot;DEPENDS.&quot;

Remove the note about disabled content and add a new note indicating that inert content is used for things like the background contents while showing a modal dialog. In this case, it would be appropriate and recommended to use aria-hidden=&quot;true&quot;… If there are cases where inert is being used to deactivate foreground content that is still intended for consumption, @inert should be combined with an explicit aria-hidden=&quot;false&quot; to override the default rendering engine behavior.

&gt; Element is focusable

I think you should split this into a couple rows.

1. Focusable elements that are natively focusable (links, buttons, etc.) NO, don&apos;t override.
2. Focusable elements that are not natively focusable (li, span, div, etc.) YES, override with any widget role.

For example, &lt;span tabindex=&quot;0&quot;&gt; is focusable but has no role in most rendering engines. The document has &quot;NONE&quot; in the role column, but I think it needs a role here in a lot of cases.

&gt; Element that is a candidate for constraint validation but…

The note column should be updated to link to what HTML defines what constitutes the user has interacted with the control &quot;significantly.&quot; Also ARIA defines aria-invalid=&quot;grammar&quot; and aria-invalid=&quot;spelling&quot; which are not mentioned here.

---

That&apos;s all I noticed. Great document! Thanks for publishing it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88340</commentid>
    <comment_count>1</comment_count>
    <who name="dmacdona">david100</who>
    <bug_when>2013-05-29 00:24:38 +0000</bug_when>
    <thetext>Removed hgroup

Will continue to go through this making appropriate changes in spare time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88343</commentid>
    <comment_count>2</comment_count>
    <who name="dmacdona">david100</who>
    <bug_when>2013-05-29 01:43:28 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; Removed hgroup
&gt; 
&gt; Will continue to go through this making appropriate changes in spare time.

Removed
&quot;, as per current browser implementations it is required that an explicit &lt;code&gt;role=checkbox&lt;/code&gt; is declared and if so you may use any &lt;code&gt;aria-*&lt;/code&gt; attributes applicable to the &lt;code&gt;checkbox&lt;/code&gt; role&quot;

Added &lt;p&gt;It is okay to use aria-valuetext on these, with or without an explicit role.&lt;/p&gt;

Removed &lt;code&gt;role=math&lt;/code&gt;

Did the rest of the recommendations, except the last two &gt; Element is focusable, &gt; Element that is a candidate for constraint validation but… and &gt; iframe [seemless]
I will look at those next time.


Added role=progressbar to the meter element

I&apos;m not sure if I got the right place for the disabled content note that Craig is talking about. I&apos;ll confirm later... in the meantime, I added it with a big ADD: and REMOVE: not on each change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>88372</commentid>
    <comment_count>3</comment_count>
    <who name="dmacdona">david100</who>
    <bug_when>2013-05-29 10:41:03 +0000</bug_when>
    <thetext>I&apos;ve addressed everything in this bug. Outstanding issues (questions) are:

1) iframe (seemless) row that I added needs tweaks, and live link to element in HTML5
2)Element is not natively focusable (li, span, div, etc.) row I added needs some example
3) Element that is a candidate for constraint validation but that does not satisfy its constraints... needs link to html5 doc
4) Element with an inert attribute... needs to be checked. YES (DEPENDS)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98701</commentid>
    <comment_count>4</comment_count>
    <who name="dmacdona">david100</who>
    <bug_when>2014-01-17 13:17:37 +0000</bug_when>
    <thetext>Moved outstanding issues to Bug 24320</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>98703</commentid>
    <comment_count>5</comment_count>
    <who name="dmacdona">david100</who>
    <bug_when>2014-01-17 13:19:04 +0000</bug_when>
    <thetext>This can now be closed</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>