RFR: 8054572: [macosx] JComboBox paints the border incorrectly [v7]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Sep 9 07:06:55 UTC 2022
On Wed, 31 Aug 2022 17:49:35 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:
>> src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxUI.java line 471:
>>
>>> 469:
>>> 470: if (comboBox.getComponentOrientation().isLeftToRight()) {
>>> 471: return new Rectangle(insets.left, insets.top + midHeight,
>>
>> OK. Since Apple has confirmed combobox height shouldn't change with font size, we can have fixed height.
>>
>> Only thing I guess, till we sort out JDK-8023912: [macosx] JComboBox RTL orientation problem
>> lets use LTR as it seems it might give a false impression that we support RTL when we dont even know if Apple supports it or not. Do you see any difference with combobox if you use LTR code vs RTL code with RTL testcase present in the above JBS? I dont in my BigSur.
>>
>> It will also be good to test it out in non-retina display to see if this "22" height is being shown ok there.
>
> I don't see a difference either but was following the same logic as used in BasicComboBoxUI. I can remove the LTR vs RTL logic until the other issue is resolved if needed.
You can keep this for now...You can take in JDK-8023912 incase which I guess needs a followup question to Apple.
-------------
PR: https://git.openjdk.org/jdk/pull/9473
More information about the client-libs-dev
mailing list