<AWT Dev> Review request for 8019280: CFileDialog selection button on choose directory mode should read " Choose "

[Avik Niyogi]

Hi Semyon,

I checked the rest of the JDK and found out that there is no localisation bundle so there is no need for NSLocalizedString. I will change that and create new webrev.
Trailing white space is used because the button size is not sufficient.

With Regards,
Avik Niyogi
> On 24-Nov-2015, at 4:03 pm, Semyon Sadetsky <semyon.sadetsky at oracle.com> wrote:
> 
> Hi Avik,
> 
> Did you try to change the locale? Did button get the localized text then?
> Why do you use trailing white-spaces for the key?
> The second argument of NSLocalizedString should be a comment. Perhaps, it'd be better to write a key description there instead of repeating the key.
> 
> --Semyon
> 
> On 11/23/2015 9:34 AM, Avik Niyogi wrote:
>> Hi All,
>> 
>> Kindly review the fix for JDK9.
>> Bug: 
>> https://bugs.openjdk.java.net/browse/JDK-8019280 <https://bugs.openjdk.java.net/browse/JDK-8019280>
>> 
>> Webrev: 
>> http://cr.openjdk.java.net/~aniyogi/8019280/webrev.00/ <http://cr.openjdk.java.net/%7Eaniyogi/8019280/webrev.00/>
>> 
>> Issue: The OpenJDK implementation of CFileDialog for mac is inconsistent with how Apple's implementation would show the selection button when the dialog was set to choose directories.
>> Expected correct caption on the selection button should be " Choose " , not " Open " according to Mac OSX UI guidelines.
>> 
>> Cause: Incorrect implementation within CFileDialog.m
>> 
>> Fix:Added logic to setPrompt when opening a directory selection File Dialog
>> 
>> With Regards,
>> Avik Niyogi
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20151124/d2a747ca/attachment.html>