On Wed, 19 Mar 2025 13:49:29 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> JTable.selectAll doesn't do anything if there are no rows or no columns. 
>> But it should still select all columns if there are no rows and the other 
>> way round. 
>> It is seen that isColumnSelected() will return false for all columns after 
>> calling selectAll() if there happened to be no rows.
>> 
>> Fix is made to select all columns even if there are no rows and similarly 
>> for rows if there are no columns.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Add check for rowselection and columnselection

src/java.desktop/share/classes/javax/swing/JTable.java line 2190:

> 2188:         if (rowCount > 0 && columnCount > 0) {
> 2189:             selectRows(rowCount);
> 2190:             selectColumns(columnCount);

Here the checks are missing. Both row and column selection allowed check.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24025#discussion_r2004823571

Reply via email to