On Mon, 13 Oct 2025 12:24:06 GMT, Marius Hanl <[email protected]> wrote:
>> At a minimum, replace the first occurrence of "repopulate" with "rebuild".
>>
>>
>> * Calling {@code refresh()} forces the TableView control to rebuild the
>> * cells necessary to populate the visual bounds of the control.
>>
>>
>> I wouldn't over-specify this by saying what `VirtualFlow` will do, but if
>> you want to add a sentence saying that this will request a layout that would
>> be fine:
>>
>>
>> * Calling {@code refresh()} forces the TableView control to rebuild the
>> * cells necessary to populate the visual bounds of the control.
>> * This will request a layout of the TableView cells.
>
> Changed to rebuild. I did not add the request layout line, in case we may
> want to change this later. Since as @hjohn and @johanvos mentioned, it is
> rather weird right now.
Thank you, though I would insist on actually explaining what "rebuild" means,
as it is not clear from the context.
`VirtualFlow` offers more detailed explanation, so perhaps we should borrow
that.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1830#discussion_r2426720297
