RFR: 8340344: The first item in TreeView is not aligned in the beginning [v2]
Andy Goryachev
angorya at openjdk.org
Wed May 7 23:12:55 UTC 2025
On Wed, 7 May 2025 15:56:57 GMT, Ziad El Midaoui <zelmidaoui at openjdk.org> wrote:
>> The issue occurred because items preceding an item with children (items with a disclosure node) had different widths, which led to misalignment. This can be fixed by requesting a cell relayout whenever the disclosure node's width changes.
>
> Ziad El Midaoui has updated the pull request incrementally with one additional commit since the last revision:
>
> Update TreeCellSkin.java
modules/javafx.controls/src/main/java/javafx/scene/control/skin/TreeCellSkin.java line 226:
> 224:
> 225: final VirtualFlow<?> flow = getVirtualFlow();
> 226: for (IndexedCell cell : flow.cells) {
`getVirtualFlow()` might return null, right?
modules/javafx.controls/src/main/java/javafx/scene/control/skin/TreeCellSkin.java line 261:
> 259: Parent p = getSkinnable();
> 260: while (p != null) {
> 261: if (p instanceof VirtualFlow) {
can be simplified:
if (p instanceof VirtualFlow vf) {
return vf;
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2078623155
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2078622098
More information about the openjfx-dev
mailing list