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

Andrew Brygin andrew.brygin at oracle.com
Tue Apr 28 14:56:28 UTC 2015


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