RFR: 8340344: The first item in TreeView is not aligned in the beginning [v2]
Ziad El Midaoui
zelmidaoui at openjdk.org
Thu May 8 14:59:40 UTC 2025
On Wed, 7 May 2025 23:09:51 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> 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?
Yes,
I added a check for null value
> 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;
I made the change , thanks
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2079888335
PR Review Comment: https://git.openjdk.org/jfx/pull/1715#discussion_r2079888974
More information about the openjfx-dev
mailing list