Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314951 - [Accessibility] Spinners report back as 'unknown'
Summary: [Accessibility] Spinners report back as 'unknown'
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.5   Edit
Hardware: Macintosh Mac OS X
: P3 normal (vote)
Target Milestone: 3.6.1   Edit
Assignee: Scott Kovatch CLA
QA Contact: Carolyn MacLeod CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-28 16:16 EDT by Scott Kovatch CLA
Modified: 2010-08-18 16:05 EDT (History)
0 users

See Also:
carolynmacleod4: review+

Attachments
Fix (5.93 KB, patch)
2010-06-15 17:25 EDT, Scott Kovatch CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.