<Swing Dev> [9] Review request for 8008657 JSpinner setComponentOrientation doesn't affect on text orientation

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Thu Jan 23 15:06:59 UTC 2014



Could you review the updated fix:
   http://cr.openjdk.java.net/~alexsch/8008657/webrev.01

   - revalidate()/repaint() is added after component orientation 
applying in the same way as it is done for the JTextFiled
   - there is one more issue that a text in the Number editor should 
stick to the arrows. It is fixed in the NumberEditor class.

  Thanks,
  Alexandr.

On 1/22/2014 6:00 PM, Sergey Bylokhov wrote:
> Hello, Alexander.
> Is updateUI needed to fix the problem? Since applyComponentOrientation 
> calls invalidateIfValid  anyway.
>
> On 20.01.2014 17:50, Alexander Scherbatiy wrote:
>>
>> Hello,
>>
>> Could you review the fix:
>>   bug: https://bugs.openjdk.java.net/browse/JDK-8008657
>>   webrev: http://cr.openjdk.java.net/~alexsch/8008657/webrev.00
>>
>>  The "componentOrientation" property change event should be 
>> propagated to the JSpinner editor component.
>>
>> Thanks,
>> Alexandr.
>>
>
>




More information about the swing-dev mailing list