RFR: 8301312: Create implementation of NSAccessibilityButton protocol [v4]
Alexander Zuev
kizune at openjdk.org
Wed May 17 20:31:27 UTC 2023
On Sat, 8 Apr 2023 14:36:30 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Properly use NSObject instead of the GlassAccessible type.
>
> modules/javafx.graphics/src/main/native-glass/mac/a11y/AccessibleBase.m line 156:
>
>> 154: NSString *roleName = jStringToNSString(env, forRole);
>> 155: Class classType = [AccessibleBase getComponentAccessibilityClass:roleName];
>> 156: GlassAccessible* accessible = NULL;
>
> It might be clearer to use `NSObject` here (which is the common supertype of `GlassAccessible` and `AccessibileBase`).
Fixed.
> modules/javafx.graphics/src/main/native-glass/mac/a11y/ButtonAccessibility.h line 37:
>
>> 35: - (NSRect)accessibilityFrame;
>> 36: @end
>> 37:
>
> Very minor: extra newline at the end of file.
Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1084#discussion_r1197014453
PR Review Comment: https://git.openjdk.org/jfx/pull/1084#discussion_r1197017319
More information about the openjfx-dev
mailing list