Bug 10917

Summary: Patch support does not use mnemonics
Product: [Eclipse Project] Platform Reporter: Nick Edgar <n.a.edgar>
Component: CompareAssignee: Adam Kiezun <akiezun>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 Keywords: accessibility
Version: 2.0   
Target Milestone: 2.0 F2   
Hardware: PC   
OS: Windows 2000   
Whiteboard:

Description Nick Edgar CLA 2002-03-06 21:39:45 EST
Build 20020305

The Compare with / Patch wizard is missing mnemonics for controls on both 
pages.

On the Verify Patch page, it would help to give the checkbox tree a label with 
a mnemonic since the user often has to go back to it.
It would also help to have Select All and Deselect All buttons.
Comment 1 Till Bay CLA 2002-04-10 09:45:32 EDT
Already in the context menu leading to that wizard a mnemonic is missing why 
not use the same one that's used for :T&eam:Create &Patch...

Comment 2 Andre Weinand CLA 2002-05-29 06:32:34 EDT
Fixed for 20020529.
I didn't add Check All and Uncheck All options on the verify page because 
changing the check boxes is rarely necessary. If all goes well all hunks in a 
patch can be applied and they are automatically checked on. I they cannot be 
applied (and the checkboxes are off) the user is not able to force it to on.
So the only case where you want to change the checkbox state is when the file 
or hunk matches but you deliberately don't want to take it.
Comment 3 Erich Gamma CLA 2002-06-02 17:48:47 EDT
pls verify
Comment 4 Adam Kiezun CLA 2002-06-03 08:01:51 EDT
verified