RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]
Andy Goryachev
angorya at openjdk.org
Tue May 20 17:56:56 UTC 2025
On Tue, 20 May 2025 09:35:45 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>>> Would it be possible to avoid re-layout of the cells that are after the Cell with a disclosureNode ?
>>
>> This is actually what is happening, when we scroll down and find a TreeCell that has a disclosure node this is when the condition `disclosureWidth > defaultDisclosureWidth` is true and we do a re-layout of the visible previous cells before the disclosure node and the new value is saved in the `maxDisclosureWidthMap` so the cells that are layouted after the disclosure node when scrolling are using the new value of `defaultDisclosureWidth`.
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2098561451
More information about the openjfx-dev
mailing list