On Wed, 31 Aug 2022 17:49:35 GMT, Damon Nguyen <dngu...@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
