For next steps I'd like to see the "(deprecated)" text in the roles' characteristics tables say "Deprecated in ARIA 1.2" next to each of these attributes.

I'd also like to see text added to each of these attributes stating they're being deprecated from being global attributes, and in future versions of ARIA they will be allowed only on specific roles that do not indicate these attributes as being deprecated.

I can work on adding this text in if others agree.

There are some cases with multiple inherritance where the script isn't doing quite what we want.

The columnheader role is a subclass of gridcell. The gridcell role supports aria-disabled, aria-expanded, aria-haspopup, and aria-invalid. But, the script is incorrectly deprecating these on columnheader. Note that we have a note in the columnheader prose saying that in a future version we plan to limit use of aria-disabled to columnheaders in a grid context; that note does not mention aria-expanded, aria-haspopup, or aria-invalid. I am not sure why; I don't remember the history of that note. rowheader has similar problems.

In the last working draft, aria-haspopup was not deprecated on menuitemcheckbox and menuitemradio. It makes sense not to support it on these two roles. However, I don't understand why the script is deprecating on these given that one of their superclasses is menuitem, and menuitem supports aria-haspopup. Perhaps this is similar to the columnheader and rowheader issue. One of the superclasses of menuitemcheckbox is checkbox, and it doesn't support aria-haspopup.

I don't recall discussion of why aria-haspopup is not supported on the option role. We have it on treeitem and tab; seems like it should be on option.

The script is incorrectly deprecating aria-disabled on treeitem, perhaps because of its listitem superclass.

This is kind of a nit, but the wording for abstract roles, e.g., command, should be something like "use on concrete subclass roles of this role prohibited in ARIA 1.2"

hey. i was able to steal some time to review this today. sorry it's been so long...

The only thing that jumps out at me right now is that in the pr preview i'm seeing aria-expanded as now being listed as a global state and property (that is now also noted as being deprecated in aria 1.2).

The 4 deprecated attributes are:

• aria-disabled
• aria-errormessage
• aria-haspopup
• aria-invalid

Some of this code seems to be mixing up aria-errormessage with aria-expanded. (See inline comments).

+1

Looks good.

One question:

• treeitem inherits from option (which supports aria-disabled) and listitem (which does not).
• treegrid inherits from tree (which supports aria-errormessage and aria-invalid) and grid (which does not).

Did we discuss these conflicts? Just want to make sure we actually decided to support aria-disabled, errormessage, and invalid on these, and it's not just some artifact of the script. :)

Reminder to commit the common stuff to aria-common.