<Swing Dev> Review request for JDK-8078489 JSpinner using SpinnerNumberModel doesn't honor orientation flag
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Wed Dec 23 15:37:41 UTC 2015
Hi, Rajeev.
A few notes:
- the cast in the "if (o != (ComponentOrientation) e.getOldValue())"
is not necessary.
- Is it possible to cover all look and feels in the test?
- Why the test is for osx only?
On 22/12/15 12:54, Rajeev Chamyal wrote:
> Hello All,
>
> Please review the following fix for Jdk9:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8078489
>
> Webrev:http://cr.openjdk.java.net/~rchamyal/8078489/webrev.00/
>
> Issue: Jspinner text in Aqua LAF is not following Right to left
> component orientation.
>
> Cause: Code for JSpinner Component orientation is missing.
>
> Fix: Updated the property change listener of Aqua LAF to update editor
> text orientation.
>
> Regards,
>
> Rajeev Chamyal
>
--
Best regards, Sergey.
More information about the swing-dev
mailing list