<Swing Dev> [9] request for review: 8073001: Java's system LnF on OS X: editable JComboBoxes are being rendered incorrectly

Alexander Potochkin alexander.potochkin at oracle.com
Wed Apr 29 13:44:52 UTC 2015


Hello Andrew

Looks good!

Thanks
alexp

On 4/28/2015 5:56 PM, Andrew Brygin wrote:
> Hello Alexander,
>
>  suggested way also works fine. Please take a look to updated fix:
>
> http://cr.openjdk.java.net/~bae/8073001/9/webrev.01/
>
> Thanks,
> Andrew
>
> On 4/27/2015 5:13 PM, Alexander Potochkin wrote:
>> Hello Andrew
>>
>> Can we update the existing focusListener class
>> created in AquaComboBoxUI.createFocusListener() method
>>
>> and add there
>>
>> if (arrowButton != null) {
>>                  arrowButton.repaint();
>> }
>>
>> to re use the methods configure/unconfigureEditor methods from the 
>> base class?
>>
>> Thanks
>> alexp
>>
>> On 4/23/2015 6:23 PM, Andrew Brygin wrote:
>>> Hello,
>>>
>>>  could you please review a fix for CR 8073001?
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8073001
>>> Webrev: http://cr.openjdk.java.net/~bae/8073001/9/webrev.00/
>>>
>>>  To resolve the problem with combobox rendering, suggested fix
>>>  adjusts text border insets in order to match dimension of the
>>>  arrow button.
>>>
>>>  Also, it unifies focus handling in order to get the arrow button
>>>  updated both for cases of standard and custom editors.
>>>  Supplied regression test verifies that the arrow button is
>>>  updated correctly.
>>>
>>>  Please take a look.
>>>
>>> Thanks,
>>> Andrew.
>>
>




More information about the swing-dev mailing list