On Fri, 5 Mar 2021 15:20:36 GMT, mstr2 <github.com+43553916+ms...@openjdk.org> 
wrote:

>> It does seems like there is a possible inconsistency for `TreeCell` and 
>> `TreeTableRow`. I'm not sure it matters in practice, since an application 
>> doesn't directly manage the creation of the cells (other than by providing a 
>> cell factory) or setting the item. The pseudo-class state of the cell is a 
>> proxy for the item it points to. Ideally, it would only contain the "empty" 
>> state if the item is null, but given that such a cell would never be 
>> rendered it would be hard to point to something that doesn't work as a 
>> result.
>> 
>> In any case, since there are larger issues than just setting the default 
>> state for `TreeCell` and `TreeTableRow` I'd prefer to decouple them from 
>> this issue and file a follow-up issue (which doesn't need to be looked at 
>> any time soon).
>
> I've reverted the changes to `TreeCell` and `TreeTableRow`.

Thanks. I filed [JDK-8263096](https://bugs.openjdk.java.net/browse/JDK-8263096) 
to track the follow-issue for  `TreeCell` and `TreeTableRow`.

-------------

PR: https://git.openjdk.java.net/jfx/pull/413

Reply via email to