RFR: 8340344: The first item in TreeView is not aligned in the beginning [v6]
Ziad El Midaoui
zelmidaoui at openjdk.org
Wed May 21 10:22:10 UTC 2025
On Wed, 21 May 2025 05:09:28 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>> Ziad El Midaoui has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Avoid re-layout items twice after disclosure node width value updated
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/TreeCellSkin.java line 232:
>
>> 230: cell.requestLayout();
>> 231: cell.layout();
>> 232: }
>
> Just a minor change please.
> The for loop could break when `cell.getIndex() >= indexWithDisclosureNode`, so that we avoid looping through the cells that are after disclosureNode.
>
>
> for (IndexedCell cell : flow.cells) {
> if (cell != null) {
> if (cell.getIndex() >= indexWithDisclosureNode) {
> break;
> } else if (!cell.isEmpty()) {
> cell.requestLayout();
> cell.layout();
> }
> }
> }
Thanks for the suggestion,
I have pushed new changes.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2099917922
More information about the openjfx-dev
mailing list