<Swing Dev> [9] Review request for 8080628: No mnemonics on Open and Save buttons in JFileChooser

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu May 21 13:25:16 UTC 2015


Hi, Alexey.
If is obvious that all others l&f should work, then I do not see the 
reason why we cannot test them.

On 21.05.15 15:37, Alexey Ivanov wrote:
> Hi Sergey,
>
> Because other look-and-feels do not define their own resources.
> Actually, one LaF would be enough, if Metal did not define its own 
> Open and Save without mnemonics.
>
> I can modify the test to iterate over all the installed LaFs. I just 
> thought it was enough to use these two because either is available on 
> all the platforms, and eventually all LaFs read the values from 
> basic.properties after the fix.
>
> Regards,
> Alexey
>
> On 21.05.2015 15:28, Sergey Bylokhov wrote:
>> Hi, Alexey.
>> Why only two look and fells are covered(metal/nimbus)?
>>
>> On 21.05.15 15:10, Alexey Ivanov wrote:
>>> Hello Swing team,
>>>
>>> Could you please review the fix:
>>>     bug: https://bugs.openjdk.java.net/browse/JDK-8080628
>>>     webrev: http://cr.openjdk.java.net/~aivanov/8080628/jdk9/webrev.00/
>>>
>>> Description:
>>> Since Open and Save buttons in JFileChooser are default ones, 
>>> mnemonics are redundant on these buttons.
>>>
>>> There's no mnemonic on Open and Save buttons in Metal Look-and-Feel 
>>> but it's displayed on Open button when a directory is selected in 
>>> the file list. This is inconsistent.
>>>
>>> The fix:
>>> Remove mnemonics in JFileChooser from basic.properties which is used 
>>> as the base for translation updates.
>>> Remove separate JFileChooser keys from metal.properties since 
>>> they're the same as in basic.properties now.
>>>
>>> The test:
>>> Regression test checks Open, Save, and Cancel buttons in 
>>> JFileChooser have no mnemonics in all the supported localizations in 
>>> Metal and Nimbus look-and-feels.
>>>
>>> Thanks,
>>> Alexey
>>
>>
>


-- 
Best regards, Sergey.




More information about the swing-dev mailing list