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

Artem Semenov asemenov at openjdk.org
Thu Jun 22 19:09:07 UTC 2023


On Mon, 19 Jun 2023 08:41:48 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

>> src/java.desktop/macosx/classes/com/apple/laf/AquaComboBoxButton.java line 162:
>> 
>>> 160:             AccessibleContext ac = this.getAccessibleContext();
>>> 161:             if (ac != null && (comboBox.getSelectedItem() != null)) {
>>> 162:                 ac.setAccessibleName(comboBox.getSelectedItem().toString());
>> 
>> Why are you using the currently selected element for the button name and not a specific name? Whether it will sound right when using a screen reader. Wouldn't this be misleading to a blind user? Especially since it will only change when this method is called.
>
>> Why are you using the currently selected element for the button name and not a specific name?
> 
> As it is required to show the magnified text for the selected item in JComboBox. Setting it to some specific name always results into that name irrespective of selected item in JCombobox.
> 
>> Whether it will sound right when using a screen reader. Wouldn't this be misleading to a blind user?
> 
> I checked with VO announcement with or without this chunk of code, it is same in both case. 
> 
>> Especially since it will only change when this method is called.
> 
> This method is called whenever the current selection is changed so accessible name is set to correct item.

JSON with enumeration of fields is also a text description of the object.
But if you checked that there is no garbage, then good.

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

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



More information about the client-libs-dev mailing list