Bug 559486

Summary: [Accessibility] Orphan content found that does not reside within a WAI-ARIA landmark or labelled region role
Product: [Eclipse Project] Platform Reporter: Masayuki Fuse <fuse>
Component: DocAssignee: Lakshmi P Shanmugam <lshanmug>
Status: NEW --- QA Contact:
Severity: normal    
Priority: P3 CC: camle, daniel_megert, kitlo, lshanmug, niraj.modi
Version: 4.15   
Target Milestone: ---   
Hardware: PC   
OS: Windows 10   
See Also: https://git.eclipse.org/r/157348
https://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=80fbb52cd58719c116cb221eb7869dd979b189be
https://git.eclipse.org/r/157625
https://git.eclipse.org/c/platform/eclipse.platform.ua.git/commit/?id=83631414d312d7ad36d6696048528570eef6c7ca
Whiteboard:
Attachments:
Description Flags
DAP
none
DAP none

Description Masayuki Fuse CLA 2020-01-24 01:30:45 EST
Build: 4.15 M1
Page : http://127.0.0.1:51721/help/advanced/tabs.jsp

Issue: Orphan content found that does not reside within a WAI-ARIA landmark or labelled region role.
XPath: /html[1]/body[1]/table[1]
Code: <table valign="middle" height="100%" width="100%" border="0" cellpadding="0" cellspacing="0">
Comment 1 Masayuki Fuse CLA 2020-01-24 01:49:11 EST
Page : http://127.0.0.1:51721/help/advanced/contentToolbar.jsp
Comment 3 Masayuki Fuse CLA 2020-01-24 02:00:33 EST
Page : http://127.0.0.1:51721/help/advanced/bookmarksToolbar.jsp
Comment 5 Masayuki Fuse CLA 2020-01-24 02:25:51 EST
Page : http://127.0.0.1:51721/help/advanced/searchToolbar.jsp
Comment 7 Lakshmi P Shanmugam CLA 2020-02-05 07:04:01 EST
Most of the pages are different types of toolbars. Do we need to specify a landmark role for toolbars? If yes, what will be correct landmark role? role="navigation" will not be suitable for all of them.
Or can we just specify the widget role, role="toolbar"?

Similarly, http://127.0.0.1:51721/help/advanced/tabs.jsp provides tabs, so can we just specify role="tablist" or do we need a landmark role here?
Comment 8 Masayuki Fuse CLA 2020-02-06 02:06:58 EST
Hmm, please add landmarks respectively. Thanks
Comment 9 Eclipse Genie CLA 2020-02-07 05:31:12 EST
New Gerrit change created: https://git.eclipse.org/r/157348
Comment 11 Lakshmi P Shanmugam CLA 2020-02-07 06:07:21 EST
(In reply to Lakshmi Shanmugam from comment #7)
> Most of the pages are different types of toolbars. Do we need to specify a
> landmark role for toolbars? If yes, what will be correct landmark role?
> role="navigation" will not be suitable for all of them.
> Or can we just specify the widget role, role="toolbar"?
> 
> Similarly, http://127.0.0.1:51721/help/advanced/tabs.jsp provides tabs, so
> can we just specify role="tablist" or do we need a landmark role here?

>> Hmm, please add landmarks respectively. Thanks

I've added the tablist and toolbar roles. The jsp pages are not used as standalone pages so I think it doesn't need to have the landmark roles. If the tool still complains will, will add the landmark roles to the html page. (In reply to Masayuki Fuse from comment #8)
Comment 12 Masayuki Fuse CLA 2020-02-11 20:43:40 EST
Created attachment 281791 [details]
DAP

The error still exists and shows new violation "An interactive element/widget must have accessible name"
Comment 13 Masayuki Fuse CLA 2020-02-13 03:35:15 EST
Created attachment 281807 [details]
DAP
Comment 14 Masayuki Fuse CLA 2020-02-13 03:37:02 EST
Page : http://127.0.0.1:51721/help/topic/org.eclipse.help.base/doc/help_home.html, adding main looked fine.
Comment 15 Eclipse Genie CLA 2020-02-13 05:01:07 EST
New Gerrit change created: https://git.eclipse.org/r/157625
Comment 17 Lakshmi P Shanmugam CLA 2020-02-13 05:35:15 EST
(In reply to Masayuki Fuse from comment #14)
> Page :
> http://127.0.0.1:51721/help/topic/org.eclipse.help.base/doc/help_home.html,
> adding main looked fine.

Thanks, I've made the change.
Comment 18 Masayuki Fuse CLA 2020-02-14 03:51:03 EST
(In reply to Lakshmi Shanmugam from comment #17)
> (In reply to Masayuki Fuse from comment #14)
> > Page :
> > http://127.0.0.1:51721/help/topic/org.eclipse.help.base/doc/help_home.html,
> > adding main looked fine.
> 
> Thanks, I've made the change.

Verified the fixed. Thanks.

There are 9 violations on toolbar and index view.
navigation role may solve some of them, however tab key and JAWS looks working fine. I'm considering to defer the fixing.
Comment 19 Lakshmi P Shanmugam CLA 2020-02-14 05:19:08 EST
(In reply to Masayuki Fuse from comment #18)
> (In reply to Lakshmi Shanmugam from comment #17)
> > (In reply to Masayuki Fuse from comment #14)
> > > Page :
> > > http://127.0.0.1:51721/help/topic/org.eclipse.help.base/doc/help_home.html,
> > > adding main looked fine.
> > 
> > Thanks, I've made the change.
> 
> Verified the fixed. Thanks.

Thanks for your help in resolving this issue.

> 
> There are 9 violations on toolbar and index view.
> navigation role may solve some of them, however tab key and JAWS looks
> working fine. I'm considering to defer the fixing.

I agree we should defer it. I'm not sure navigation will be the correct role for all the toolbar views in the page, such as search toolbar or bookmarks toolbar.
Comment 20 Masayuki Fuse CLA 2020-02-16 19:13:37 EST
(In reply to Lakshmi Shanmugam from comment #19)
> (In reply to Masayuki Fuse from comment #18)
> > (In reply to Lakshmi Shanmugam from comment #17)
> > > (In reply to Masayuki Fuse from comment #14)
> > > > Page :
> > > > http://127.0.0.1:51721/help/topic/org.eclipse.help.base/doc/help_home.html,
> > > > adding main looked fine.
> > > 
> > > Thanks, I've made the change.
> > 
> > Verified the fixed. Thanks.
> 
> Thanks for your help in resolving this issue.
> 
> > 
> > There are 9 violations on toolbar and index view.
> > navigation role may solve some of them, however tab key and JAWS looks
> > working fine. I'm considering to defer the fixing.
> 
> I agree we should defer it. I'm not sure navigation will be the correct role
> for all the toolbar views in the page, such as search toolbar or bookmarks
> toolbar.

okay. deferring. Thanks