Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 317505

Summary:

[NLS] Accessibility: Labels of lists are not read by JAWS in several wizards

Product:

[WebTools] WTP Common Tools

Reporter:

Carl Anderson <ccc>

Component:

Faceted Project Framework

Assignee:

Konstantin Komissarchik <konstantin>

Status:

RESOLVED FIXED

QA Contact:

Konstantin Komissarchik <konstantin>

Severity:

normal

Priority:

CC:

cbridgha, ccc, david_williams, hshanka, konstantin

Version:

3.2

Flags:

ccc: review+

Target Milestone:

3.2.1

Hardware:

OS:

Windows XP

Whiteboard:

Bug Depends on:

315679

Bug Blocks:

Attachments:

Description	Flags
The java facet part of 315679	none
Patch v2	none

Description Carl Anderson

2010-06-21 15:13:13 EDT

Konstantin would like a separate bug to track the inclusion of this change into the Faceted Project Framework.

Please note that PMC approval was granted to bug 315679.  I would assume that that means that this bug is implicitly approved.

+++ This bug was initially created as a clone of Bug #315679 +++

Build Identifier: 321

1.- Open the wizards for EJB, APP client, Connector or dynamic web projects and go to the java facets page. Using the keyboard, and with JAWS started, tab to the list of source folders. The label is not read by JAWS
2.- Open the new EAR project wizard, and go to the Enterprise application page. Using the keyboard, and with JAWS started, tab to the list of modules. The label is not read by JAWS

Reproducible: Always

Comment 1 Carl Anderson

2010-06-21 15:15:40 EDT

Created attachment 172359 [details]
The java facet part of 315679

Comment 2 Carl Anderson

2010-06-23 13:07:40 EDT

I just committed the non-FacetedProjectFramework part of 315679.
The patch on this bug was part of the PMC approved patch, so no further approval should be necessary.

I am marking this as reviewed, simply because I am the Common Project lead, and I have reviewed this, and I believe that it is ready to go into WTP 3.2.1.

Comment 3 Konstantin Komissarchik

2010-06-23 16:48:35 EDT

There is a problem with this patch. I use Microsoft Inspect32 tool.

Before the patch, if I put focus on the table and the table has at least one item, the name is the label of that item. As I move focus between multiple table entries, the name changes to represent the selected item. The name of the parent (the table) is not set. This could cause confusion when table has items, but is a bigger issue when table has no items. In that case name of the selected widget will be "" on focus.

After the patch, the parent widget name is set correctly and the no item case works, however we've lost the ability to read the text of the selected items. No matter what I do, I cannot get inspect32 to show the item label.

I will look for a solution.

Comment 4 Konstantin Komissarchik

2010-06-23 17:02:31 EDT

Created attachment 172552 [details]
Patch v2

The trick is to check that childID is equals to -1 before setting the name. Other values of childID represent table items and we don't want to mess with the default behavior on those. 

Please re-review as appropriate. Also, all of my evaluation and changes have been based on using Inspect32 tool. If someone has access to JAWS, it would be good to confirm this separately as well.

Comment 5 Konstantin Komissarchik

2010-06-28 18:33:37 EDT

Released changes to WTP 3.2.1 and 3.3 streams along with fproj branch.