Bug 207616 - Axis2: Table in Axis2 Web Service Skeleton Java Bean Configuration Page not Accessible
Summary: Axis2: Table in Axis2 Web Service Skeleton Java Bean Configuration Page not ...
Status: CLOSED FIXED
Alias: None
Product: WTP Webservices
Classification: WebTools
Component: jst.ws (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 M6   Edit
Assignee: Andrew Mak CLA
QA Contact: Kathy Chan CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-10-26 15:24 EDT by Zina Mostafia CLA
Modified: 2008-06-06 11:41 EDT (History)
2 users (show)

See Also:


Attachments
patch (15.15 KB, patch)
2008-03-19 17:36 EDT, Andrew Mak CLA
bjorn.freeman-benson: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zina Mostafia CLA 2007-10-26 15:24:08 EDT
Build ID: 3.3

Steps To Reproduce:
The Name Space-Package table in the Axix2 Web Service Skeleton Java Bean Configuration Page is not accessible.

1- You cannot step on any cell in the second colomn using keyboard only.
2-You cannot edit any cell ( first or second column) untill you click on it by the mouse. Stepping on the cell by means of a keyboard does not enable editing.

More information:
The same problem in the Client Web Service Configuration page.


In The Axis 1 Client Wizard , in the Web Service Client Namespace to Package Mapping page, we have a similar table. The problem is resolved by adding buttons under the table for using in Adding / Editing data inside the table. This solution is acceptable in terms of Accessibility.
Comment 1 Lahiru Sandakith CLA 2007-10-31 04:50:26 EDT
I tested this and what I saw was if you are doing it from keyboard only with the tab index then we can not actually get the focus on the table entries to edit it becaue the action event involved in getting the focus inside the raw of a table is actually mouse down (click on any raw). And then as I see it from the tab index its actually set focus to the namespace to package table there after and it can be  traverse through the keyboard. So I am actually bit gray on what is the actual requirement on this, is this needed in kind of a UI automation testing. I ll try to fix it as its already fixed in the Axis1 case.
Comment 2 Lahiru Sandakith CLA 2007-10-31 05:10:37 EDT
Okey I looked at the how the Axis1 have solved the problem, but I am not impressed with it, mainly because of the fact that is not pre-populated. What I am thinking of is to have some kind of tab index working and get the focus on to the namespace package table and then with again after that with the tab (esc to get out focus), then again go through the main controls in the page with tab navigation. Is this can not be achieved with the SWT controls? I will try look in to this more. 
thanks 
Lahiru
Comment 3 Kathy Chan CLA 2008-03-16 12:10:03 EDT
Since this involves UI changes, it needs to be done for WTP 3.0 M6.
Comment 4 Andrew Mak CLA 2008-03-19 11:50:32 EDT
Hi,

I agree that buttons are probably not necessary in axis2 case since the table is already populated with all the namespaces in the WSDL file.  There is one thing I want to clarify on however, which is, what's the purpose of allowing the user to edit the namespace column?   If the namespaces shown are already those found inside the WSDL, would users ever need to modify them?  If the namespace does not need to be editable, it would simplify the fix for this bug since we don't have to handle navigating between columns.
Comment 5 Andrew Mak CLA 2008-03-19 17:36:24 EDT
Created attachment 92965 [details]
patch

Here is a proposed fix for this bug, which assumes that we restrict the namespace column to be read-only.
Comment 6 Kathy Chan CLA 2008-03-26 14:56:01 EDT
Patch committed and released to HEAD as v200803261855.
Comment 7 David Williams CLA 2008-04-24 00:45:31 EDT
mass change to add 'contributed' keyword based on bugzilla query, please correct if that's not accurate (by marking patches as obsolete and removing the 'contributed' keyword. 
Comment 8 Zina Mostafia CLA 2008-06-06 11:41:13 EDT
Fix verified. Thanks.