<Swing Dev> [9] Review request for 8160054: The FileChooser didn't displayed large font with GTK LAF option.
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Mon Sep 5 16:51:53 UTC 2016
On 05.09.16 19:11, Semyon Sadetsky wrote:
>
> On 9/5/2016 6:55 PM, Sergey Bylokhov wrote:
>> On 02.09.16 12:50, Alexandr Scherbatiy wrote:
>>> The fix looks good to me.
>>
>> Probably the comment between .00 and .01 should be updated also?
>> "// if font property got different value then return it"
> I will change it to "got a value" upon the push if you don't mind.
ok
>>
>>
>>> On 9/2/2016 11:41 AM, Semyon Sadetsky wrote:
>>>> On 7/14/2016 6:31 PM, Alexandr Scherbatiy wrote:
>>>>
>>>>> On 6/27/2016 9:23 PM, Semyon Sadetsky wrote:
>>>>>> Hello,
>>>>>>
>>>>>> Please review fix for JDK9:
>>>>>>
>>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8160054
>>>>>>
>>>>>> webrev: http://cr.openjdk.java.net/~ssadetsky/8160054/webrev.00/
>>>>>>
>>>>>> The root case is GTK LnF does not allow to change its default fonts.
>>>>>>
>>>>>> The proposed solution let's to use LnF font properties to change the
>>>>>> defaults.
>>>>> 292 if (propFont != null && !font.equals(propFont)) {
>>>>> 293 // if font property got different value then
>>>>> return it
>>>>> 294 return propFont;
>>>>> 295 }
>>>>>
>>>>> It looks like the !font.equals(propFont) condition is not necessary
>>>>> because when two fonts are equal it is right to return the propFont
>>>>> as well.
>>>> Please review the updated fix
>>>> http://cr.openjdk.java.net/~ssadetsky/8160054/webrev.01/
>>>>
>>>> --Semyon
>>>>>
>>>>> Thanks,
>>>>> Alexandr.
>>>>>>
>>>>>> --Semyon
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>>
>
--
Best regards, Sergey.
More information about the swing-dev
mailing list