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

Bug 333983

Summary: [Accessibility] Legacy AC warning dialog fails accessibility tests
Product: z_Archived Reporter: Mike Reid <mikereid>
Component: TPTPAssignee: Mike Reid <mikereid>
Status: CLOSED FIXED QA Contact: Kathy Chan <kathy>
Severity: normal    
Priority: P3 CC: ewchan, jcayne
Version: unspecifiedFlags: jcayne: review+
Target Milestone: ---   
Hardware: PC   
OS: Windows All   
Whiteboard:
Attachments:
Description Flags
Patch
none
Updated patch
none
Patch to re-use existing 'Warning' message none

Description Mike Reid CLA 2011-01-11 09:52:00 EST 
Bug 323137 introduced detection and warning of legacy Agent Controllers (i.e. ACs where the new exec framework is unsupported). However the warning dialog used is not accessible on several counts: mouse keys don't work properly, expanding details crashes JAWS, high contrast mode results in unreadable text, etc.

This bug is opened to correct these issues.
Comment 1 Mike Reid CLA 2011-01-11 09:53:21 EST 
Created attachment 186505 [details]
Patch

Patch to fix accessibility problems.
Comment 2 Mike Reid CLA 2011-01-11 10:00:49 EST 
Joel can you please review this patch?

The patch simply replaces the use of the "StatusManager" mechanism with org.eclipse.jface.dialogs.ErrorDialog, which is more accessibility friendly.
Comment 3 Mike Reid CLA 2011-01-11 12:58:19 EST 
Created attachment 186534 [details]
Updated patch

Updated patch with comments from Joel.
Comment 4 Joel Cayne CLA 2011-01-11 13:51:42 EST 
Patch looks good.
Comment 5 Kathy Chan CLA 2011-01-12 11:19:03 EST 
Project approved for 4.7.2.
Comment 6 Mike Reid CLA 2011-01-12 11:31:33 EST 
Patch checked into HEAD.
Comment 7 Mike Reid CLA 2011-01-21 10:52:37 EST 
Re-opening. The accessibility fix introduced a new "Warning" string whereas one already exists in the same .properties file. To save on translation resources, pulling out the new string and just referencing the existing string.
Comment 8 Mike Reid CLA 2011-01-21 10:54:13 EST 
Created attachment 187294 [details]
Patch to re-use existing 'Warning' message

Re-use warning patch attached.
Comment 9 Mike Reid CLA 2011-01-21 10:55:56 EST 
'Warning' re-use patch checked into HEAD.
Comment 10 Mike Reid CLA 2011-04-04 11:07:56 EDT 
Closing.