On Tue, 20 May 2025 17:53:54 GMT, Andy Goryachev <[email protected]> wrote:
>> Yes, the cells after disclosure node use the new `defaultDisclosureWidth`.
>> But their layout happens twice using the new `defaultDisclosureWidth`.
>>
>> **First time**: The loop `for (IndexedCell cell : flow.cells) {` initiates
>> the layout of all the cells, including the cells that are after the
>> disclosure node.
>>
>> **Second time**: and layout of those after cells happens again as part of
>> ongoing layout of `TreeView` from this method `Parent.layout()`
>
> as long as both layouts happens within the same pulse, we should be ok.
> I did not see any flicker with the last change.
Yes,
I did limit the re-layout in my fix to happen only for the cells before the
disclosure node cell.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2098719206
