<Swing Dev> [9] Review request for 8160054: The FileChooser didn't displayed large font with GTK LAF option.

Semyon Sadetsky semyon.sadetsky at oracle.com
Mon Sep 5 16:11:04 UTC 2016


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.
>
>
>> 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
>>>>>
>>>>>
>>>>
>>>
>>
>
>




More information about the swing-dev mailing list