CORE-AAM 1.1: Implement feed role on macOS

[richschwer]

Moved from this action in tracker: https://www.w3.org/WAI/ARIA/track/actions/2119

[cookiecrook]

https://bugs.webkit.org/show_bug.cgi?id=151691

[joanmarie]

@cookiecrook I still have some doubts about the value of this role, but I hope we can agree that role="feed" is no less valuable than role="group". And failing that, I hope we can at least agree that if an author uses role="feed", we shouldn't prune if from the accessibility tree. (It turns out WebKit was pruning it in some instances.)

It seemed to me that the mapping should be the same as role="group", at least for now. I just landed a patch that does that for both of our platforms. See https://trac.webkit.org/changeset/215886. I hope that's alright with you. If not, please open a new bug, assign it to me, and I'll fix it tomorrow or this weekend.

On the other hand, if it's ok with you, then we need to make a couple changes to the Core AAM:

1. Remove the "Need to verify." text that is currently in the mapping for feed.

2. Update the AXSubrole value from <nil> to AXApplicationGroup.

I'm happy to take care of that as well.

Regardless of your decision, please chime in here because we need to wrap up the Core AAM soon. Thanks in advance!

[mcking65]

@joanmarie commented:

On the other hand, if it's ok with you, then we need to make a couple changes to the Core AAM:

1. Remove the "Need to verify." text that is currently in the mapping for  feed .
2. Update the  AXSubrole  value from    to  AXApplicationGroup .

Core AAM currently has:

AXRole: AXGroup
AXSubrole:
AXRoleDescription: 'group' 

Shouldn't AXRoleDescription: be 'feed'?

[joanmarie]

Shouldn't AXRoleDescription: be 'feed'?

That is up to the owners of the accessibility API.

[cookiecrook]

@mcking65 wrote:

Shouldn't AXRoleDescription: be 'feed'?

That seems fine. I updated WebKit 151691. The article navigation was already implemented in WebKit 170330.

[joanmarie]

WebKit patch landed: https://trac.webkit.org/changeset/216007
Pull request 578 opened to make the changes to the spec

[joanmarie]

Spec changes pull request merged. Closing this issue.