On Mon, 17 Aug 2026 18:29:54 GMT, Kevin Rushforth <[email protected]> wrote:
>> I don't understand - it does not matter what kind of parameters they are,
>> they need to be documented. How am I supposed to provide VISIBLE_ITEM_RANGE
>> in RichTextArea?
>>
>> `ListViewSkin` L386 returns the `int[]` with `{ firstIndex, count }` which
>> needs to be documented.
>>
>> Specifically,
>> - `int[] { firstIndex, count }` must be documented
>> - `int[] { 0, 0 }` case must be documented
>> - the fact that it is a non-null int[2] should probably be mentioned
>
> I was referring to the first inline comment that you made above on this line,
> which is `* <li>Parameters: </li>`:
>
>> Parameters: should describe the structure (see BOUNDS_FOR_RANGE for example)
>
> There are no parameters.
>
>> ListViewSkin L386 returns the int[] with { firstIndex, count } which needs
>> to be documented.
>
> So you are saying that the _return value_ (not parameters) needs to better
> documented. Quite possibly.
I am using terminology used in the `AccessibleAttribute` class. For example,
L91 specifies the "Parameters:", whatever that is. Perhaps it should be
renamed "Structure:" or something like that.
edit: Also, we should probably drop the "Parameters:" `<li>` when it's empty.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2217#discussion_r3797882042
