Hi All, Please find below the web rev with inputs from comments. http://cr.openjdk.java.net/~aniyogi/8019280/webrev.02/ <http://cr.openjdk.java.net/~aniyogi/8019280/webrev.02/> With Regards, Avik Niyogi
> On 24-Nov-2015, at 5:16 pm, Avik Niyogi <avik.niy...@oracle.com> wrote: > > 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.sadet...@oracle.com >> <mailto:semyon.sadet...@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 >> >