RFR: 8284542: [Accessibility] [Win] Missing attribute for toggle state of CheckBox in CheckBoxTreeItem [v3]

[Kevin Rushforth]

On Fri, 12 May 2023 22:13:00 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> It does work in the actual javadocs (so almost certainly an Eclipse issue).
>> 
>> Minor suggestion: use `{@link ...}` in the Description (and then you can remove the `@see`) like this:
>> 
>> 
>> This enum describes the values for the {@link TOGGLE_STATE} attribute.
>
> In Eclipse, only this works:
> 
> {@link AccessibleAttribute#TOGGLE_STATE}

Then I think Eclipse has a bug. `TOGGLE_STATE` is a field (enum value) in the outer enum class, `AccessibleAttribute`, so can be referenced from the nested enum class without qualifying it with the outer class name. If we wanted to be more clear (and work around the Eclipse limitation), it could be written as:


{@link AccessibleAttribute#TOGGLE_STATE TOGGLE_STATE}

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1088#discussion_r1192983753