RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]
Ziad El Midaoui
zelmidaoui at openjdk.org
Tue May 20 19:34:57 UTC 2025
On Tue, 20 May 2025 17:53:54 GMT, Andy Goryachev <angorya at openjdk.org> 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
More information about the openjfx-dev
mailing list