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

Bug 314951

Summary: [Accessibility] Spinners report back as 'unknown'
Product: [Eclipse Project] Platform Reporter: Scott Kovatch <skovatch>
Component: SWTAssignee: Scott Kovatch <skovatch>
Status: RESOLVED FIXED QA Contact: Carolyn MacLeod <carolynmacleod4>
Severity: normal    
Priority: P3 Flags: carolynmacleod4: review+
Version: 3.5   
Target Milestone: 3.6.1   
Hardware: Macintosh   
OS: Mac OS X   
Whiteboard:
Attachments:
Description Flags
Fix none

Description Scott Kovatch CLA 2010-05-28 16:16:11 EDT
I20100527-1700

Spinners don't report back what kind of widget they are. The sub-controls inside them correctly identify themselves ('edit text' and 'stepper'), but moving up to the containing view reports back 'unknown'.

I'm fairly certain this is not a regression, but hadn't noticed until now.

1. Launch ControlExample or ControlsWithAccessibleNamesExample.
2. Turn on VoiceOver.
3. Click on the spinner's text field.
--> VoiceOver reports back the contents of the text field and says 'edit text'.
4. Press ctrl-option-shift-up arrow to go up one container level. The view containing the controls is now under the VoiceOver cursor.
--> VoiceOver reports back 'unknown'.

Expected:

Not sure what it should say, but 'unknown' is definitely not right.
Comment 1 Scott Kovatch CLA 2010-05-28 17:15:16 EDT
The Cocoa docs for roles and attributes (http://developer.apple.com/Mac/library/documentation/Accessibility/Conceptual/AccessibilityMacOSX/OSXAXRoleReference/OSXAXRoleReference.html) imply that the composite itself should not have a role distinct from the stepper arrows.
Comment 2 Scott Kovatch CLA 2010-06-15 17:25:43 EDT
Created attachment 171997 [details]
Fix

Refactored Control so multiple views inside a Composite can have their accessibility properties overridden.
Comment 3 Scott Kovatch CLA 2010-06-15 17:26:36 EDT
One more to review, Car. I'm trying to whittle the accessibility list down to zero.
Comment 4 Scott Kovatch CLA 2010-06-15 19:31:21 EDT
Targeting 3.6.1.
Comment 5 Scott Kovatch CLA 2010-07-14 17:29:11 EDT
Fixed in HEAD > 20100714, since I added a fix for 319677 to Control. 319677 should go into 3.6.1 as well, so I'll update that bug separately.
Comment 6 Scott Kovatch CLA 2010-08-18 14:01:05 EDT
Fixed in R3_6_maintenance > 20100818.
Comment 7 Carolyn MacLeod CLA 2010-08-18 16:05:18 EDT
Looks ok. Thanks, Scott.