Bug 240958

Hey,

last week @work I had to edit some data in our databases and was annoyed, by the fact, that I could not just use my keyboard efficiently:

1. I selected the relevant data
2. I opened the first column to edit by clicking on it
3. I wanted to change the following values in the same column by pressing "enter" to go to the next row, same column.

This would be the normal behaviour of a JTable, but the focus return to cell at [0,0] after editing a cell. This can be tracked back to:

JXTableRowHeader.CountingTableModel#setCount

That method is invoked via JXTableRowHeader#tableModelListener on every change to the underlying table.

setCount now propagates that event as a TableModelEvent to all listeners.

This causes JTable#sortedTableChanged to be called. That determines, that the whole table changed (no row indices are provided in the call described above) and this causes JTable to clear the selection. The selectionModel of the headerTable and the mainTable are connected (JXTableRowHeader -> construtor -> headerTable.setSelectionModel).

The fix just makes sure, that in setCount the update and event generation are only done, if the count really differs.