RFR: 5108458: JTable does not properly layout its content [v5]

[Prasanta Sadhukhan]

On Mon, 20 Nov 2023 10:10:12 GMT, Tejesh R <tr at openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTableUI.java line 2087:
>> 
>>> 2085:             for (int row = rMin; row <= rMax; row++) {
>>> 2086:                 cellRect = table.getCellRect(row, cMin, false);
>>> 2087:                 cellRect.x = getXPosition(cMin);
>> 
>> Why do we need to calculate cellRect.x separately? What is wrong in getting the value already there in `cellRect` ?
>
> The value we get from `table.getCellRect(row, cMin, false);` doesn't consider x position offset, so we need to update with the offset value.

getCellRect spec says
`Returns a rectangle for the cell that lies at the intersection of row and column`

What's the x position offset we are talking here?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16374#discussion_r1398979460