RFR: 8309733: [macOS, Accessibility] VoiceOver: Incorrect announcements of JRadioButton
Alexey Ivanov
aivanov at openjdk.org
Thu Jun 22 20:48:05 UTC 2023
On Thu, 22 Jun 2023 18:45:27 GMT, Alexander Zuev <kizune at openjdk.org> wrote:
>> src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java line 185:
>>
>>> 183: // Do send check box state changes to native side
>>> 184: if (thisRole == AccessibleRole.CHECK_BOX) {
>>> 185: if (newValue != null && !newValue.equals(oldValue)) {
>>
>> Is it possible to have "newValue == null" and the "oldValue != null"? Do we need to update the native in this case?
>
> No. Null value is not a correct value to be set for these types of control so if null is passed then it is not a real value update event.
[`Objects.equals`](https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/Objects.html#equals(java.lang.Object,java.lang.Object)) can be used to avoid null-check, it handles `null` values gracefully.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14605#discussion_r1239014266
More information about the client-libs-dev
mailing list