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

Bug 323096

Summary: [Accessibility] VoiceOver says SortButton in Tree & Table when column name is empty on Leopard
Product: [Eclipse Project] Platform Reporter: Scott Kovatch <skovatch>
Component: SWTAssignee: Scott Kovatch <skovatch>
Status: RESOLVED WONTFIX QA Contact: Carolyn MacLeod <carolynmacleod4>
Severity: normal    
Priority: P3 CC: duongn, skovatch
Version: 3.4Keywords: accessibility
Target Milestone: 3.7Flags: carolynmacleod4: review-
Hardware: Macintosh   
OS: Mac OS X   
Whiteboard:
Bug Depends on: 233579    
Bug Blocks:    
Attachments:
Description Flags
Fix for Tree & Table for 3.6 branch none

Description Scott Kovatch CLA 2010-08-18 19:08:55 EDT
This bug is for 3.6.1.

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

Run the ControlExample and select the Tree or Table.
Select the SW.CHECK style, turn on Multiple Columns and Header Visible
The Checkbox column has no name and is read as "SortButton" by VoiceOver.
Comment 1 Scott Kovatch CLA 2010-08-18 19:09:58 EDT
Created attachment 176950 [details]
Fix for Tree & Table for 3.6 branch

Attaching patch here for review.
Comment 2 Carolyn MacLeod CLA 2010-08-24 15:25:57 EDT
I don't see any difference in 3.6.1 when I apply this patch.   :)
(I do see the difference when I run from HEAD for 3.7).
So there may be something missing.
However, don't worry about it, because I don't think this is important enough to get into 3.6.1 accessibility because the only way to get VoiceOver to speak the header for column 0 is to hover with the mouse.

FYI, SSQ and I had a look at the 3.7 change as well (to try to understand what the 3.6.1 change wanted to do) and we found a 1-pixel bug (I created Bug 323535 for it).
Comment 3 Scott Kovatch CLA 2010-08-24 16:59:35 EDT
(In reply to comment #2)
> I don't see any difference in 3.6.1 when I apply this patch.   :)
> (I do see the difference when I run from HEAD for 3.7).

That was expected. I didn't put the visual change for column 0 into 3.6.1.

> So there may be something missing.
> However, don't worry about it, because I don't think this is important enough
> to get into 3.6.1 accessibility because the only way to get VoiceOver to speak
> the header for column 0 is to hover with the mouse.

Okay. Yeah, I did notice that you can't navigate into the header view.