Bug 155403 – AX: "AXLandmarkApplication" is an inappropriate subrole for ARIA "application" since it's no longer a landmark

• Component: Accessibility
• Status: RESOLVED FIXED

[James Craig]

AX: "AXLandmarkApplication" is an inappropriate subrole for ARIA "application" since it's no longer a landmark

[Radar WebKit Bug Importer]

<rdar://problem/25130771>

[James Craig]

Don't use AXApplication since that's reserved for native apps.

[Joanmarie Diggs]

Chris: Any thoughts on this one?

We need to wrap up the mappings for the Core AAM spec.

[chris fleizach]

i think we gotta change it everywhere

[Joanmarie Diggs]

(In reply to comment #4)
> i think we gotta change it everywhere

Meaning....?

[chris fleizach]

(In reply to comment #5)
> (In reply to comment #4)
> > i think we gotta change it everywhere
> 
> Meaning....?

we should change WebCore code so that Application is not a landmark type. i don't know what the new names will be though since we haven't done it

[Joanmarie Diggs]

(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #4)
> > > i think we gotta change it everywhere
> > 
> > Meaning....?
> 
> we should change WebCore code so that Application is not a landmark type. i
> don't know what the new names will be though since we haven't done it

Aha. Gotcha. So both for the purpose of making the change everywhere, and also filling out the cell in the table in the spec, if you could sort out what the role, subrole, and role description should be that would be great.

Then, if you and Nan are busy, I'm happy to make the change for our platforms once I know what they should be.

Thanks again!

[chris fleizach]

(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > (In reply to comment #4)
> > > > i think we gotta change it everywhere
> > > 
> > > Meaning....?
> > 
> > we should change WebCore code so that Application is not a landmark type. i
> > don't know what the new names will be though since we haven't done it
> 
> Aha. Gotcha. So both for the purpose of making the change everywhere, and
> also filling out the cell in the table in the spec, if you could sort out
> what the role, subrole, and role description should be that would be great.
> 
> Then, if you and Nan are busy, I'm happy to make the change for our
> platforms once I know what they should be.

Yes please go ahead. I think you'll be able to make reasonable decisions for all platforms

> 
> Thanks again!

[Joanmarie Diggs]

Created attachment 276038 [details]
Patch

[Joanmarie Diggs]

@Chris: Due to some local build issues I'm having with the iOS Simulator, along with having no Windows environment, I'm tossing this patch at EWS to see what test failures I get before asking for formal review.

In the meantime, how do you feel about a subrole of AXWebApplication and a role description of "web application"? I wasn't sure if the role description should be left alone, or if the distinction between a web app and a native/top-level app mattered.

[chris fleizach]

(In reply to comment #10)
> @Chris: Due to some local build issues I'm having with the iOS Simulator,
> along with having no Windows environment, I'm tossing this patch at EWS to
> see what test failures I get before asking for formal review.
> 
> In the meantime, how do you feel about a subrole of AXWebApplication and a
> role description of "web application"? I wasn't sure if the role description
> should be left alone, or if the distinction between a web app and a
> native/top-level app mattered.

sounds good to me

looks like EWS is green

[Joanmarie Diggs]

(In reply to comment #11)
> (In reply to comment #10)
> > @Chris: Due to some local build issues I'm having with the iOS Simulator,
> > along with having no Windows environment, I'm tossing this patch at EWS to
> > see what test failures I get before asking for formal review.
> > 
> > In the meantime, how do you feel about a subrole of AXWebApplication and a
> > role description of "web application"? I wasn't sure if the role description
> > should be left alone, or if the distinction between a web app and a
> > native/top-level app mattered.
> 
> sounds good to me
> 
> looks like EWS is green

Yup. Please review then. Thanks!

[WebKit Commit Bot]

Comment on attachment 276038 [details]
Patch

Clearing flags on attachment: 276038

Committed r199260: <http://trac.webkit.org/changeset/199260>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.