RFR: 8283214: [macos] Screen magnifier does not show the magnified text for JcomboBox [v11]

Abhishek Kumar abhiscxk at openjdk.org
Fri Nov 10 18:20:28 UTC 2023


On Fri, 10 Nov 2023 11:58:19 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   jtreg update
>
> src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxButton.java line 276:
> 
>> 274:                     name = c.getAccessibleContext().getAccessibleName();
>> 275:                 }
>> 276:             }
> 
> For the button, super usually returns `""` (empty string) which is the default caption of the button. This code preserves the common behaviour of `getAccessibleName`, in particular it returns `accessibleName` field if it's set.
> 
> Since the arrow button is an internal component, we may drop calling `super`. However, I'm unsure about it.
> 
> The two `if` statements could be combined into one:
> 
> 
>             if ((name == null || name.isEmpty())
>                 && (!comboBox.isEditable() && comboBox.getSelectedItem() != null)) {

Conditions combined.

> Since the arrow button is an internal component, we may drop calling super. However, I'm unsure about it.

I think we can keep this as it is.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14497#discussion_r1389729338


More information about the client-libs-dev mailing list