On Tue, 15 Sep 2026 20:04:17 GMT, Marius Hanl <[email protected]> wrote:

>> modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/behavior/CellBehaviorBase.java
>>  line 219:
>> 
>>> 217:         // If the mouse event is not contained within this TreeCell, 
>>> then
>>> 218:         // we don't want to react to it.
>>> 219:         if (!cell.contains(x, y)) {
>> 
>> we need to keep `cell.isEmpty()` check here because the model size can 
>> change before an existing empty cell is updated (similarly to the cell 
>> filter reproducer I've provided earlier)
>
> I'm confused. Clicking an empty cell is allowed, I can't see why it should 
> not.

Instead, `getItemCount()` should not throw.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2129#discussion_r4019725537

Reply via email to