RFR: 8336031: Create implementation of NSAccessibilityStaticText protocol [v2]

[Andy Goryachev]

On Wed, 17 Jul 2024 04:55:25 GMT, Alexander Zuev <kizune at openjdk.org> wrote:

>> modules/javafx.graphics/src/main/native-glass/mac/a11y/AccessibleBase.m line 53:
>> 
>>> 51:     [rolesMap setObject:@"JFXCheckboxAccessibility" forKey:@"CHECK_BOX"];
>>> 52:     [rolesMap setObject:@"JFXCheckboxAccessibility" forKey:@"TOGGLE_BUTTON"];
>>> 53:     [rolesMap setObject:@"JFXStaticTextAccessibility" forKey:@"TEXT"];
>> 
>> are these keys mapped to AccessibleRole.java enum?  if so,
>> 1. should we add a comment stating that (and possibly add a comment in AccessibleRole as well?
>> 2. are we going to keep adding roles to this block as we go along?
>
> 1. Yes, these are linked to the JFX accessible roles defined in that enum. I can add comment here - not sure about the AccessibleRole, it is a definition that might be used for more than one platform so linking platform specific implementation from there sound incorrect;
> 2. Yes, as we are converting more and more functionality to the new API we are going to add roles implementations to this block until we have no roles not covered with the new API.

thank you for clarifying.  fully agree with #1, referencing from platform specific to the common should be sufficient.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1507#discussion_r1681191348