On Fri, 27 Mar 2026 21:55:57 GMT, Philemon Hilscher <[email protected]> wrote:

> This fixes the issue adressed here https://bugs.openjdk.org/browse/JDK-8222454
> 
> The added TreeTableViewTests were failing without the main code changes. I 
> added the other tests to ensure the consistent functionality of related cell 
> based components.
> The method getItemCount() needed to be moved to CellBehaviorBase to do the 
> necessary index check in doSelect() below. All subclasses now implement this 
> method based on the cell container.
> 
> There is still some refactoring potential here, to reduce code duplications 
> and ensure consistent behavior of all cell based components. This could be 
> done in a follow-up PR.

modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/behavior/TableCellBehaviorBase.java
 line 103:

> 101:         final C tableCell = getNode();
> 102: 
> 103:         int count = getItemCount();

Moved up to check it earlier on

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

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

Reply via email to