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 2198:
> 2196: if (getColumnSelectionAllowed()) {
> 2197: selectColumns(columnCount);
> 2198: }
Can we simplify the code from line 2188 - 2198, since the intension is to
selectRows/Columns if its available.
`
if (rowCount > 0 ) {
if (getRowSelectionAllowed()) {
selectRows(rowCount);
}
} else if (columnCount > 0 ) {
if (getColumnSelectionAllowed()) {
selectColumns(columnCount);
}`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24025#discussion_r2004835032
