On Thu, 22 Jun 2023 13:47:56 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org>
wrote:
> If a JTable is wrapped in JLayer and first row is selected and VK_PAGE_DOWN
> is pressed, the last row of the table is selected instead of the next row
> after current viewport.
> This is because table.getParent.getSize() returns the whole table size
> whereas it should be the JViewport which size should be used to calculate the
> "delta" to be used to scroll the table and select the row and
> "SwingUtilities.getUnwrappedParent(table)" gets the viewport.
>
> Tested against SwingSet2 too where it worked too.
test/jdk/javax/swing/JTable/JLayerTableTest.java line 57:
> 55: @Override public int getRowCount() {return 100;}
> 56: @Override public int getColumnCount() {return 3;}
> 57: @Override public Object getValueAt(int row, int column)
> {return "(" + row + "," + column + ")";}
Please split the long lines to have 80 chars.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14614#discussion_r1239039647
