deephaven / deephaven-plugins

Deephaven Plugins
11 stars 14 forks source link

refactor: Remove row and column indexes from table press handlers #592

Closed mattrunyon closed 2 months ago

mattrunyon commented 3 months ago

Fixes #528

BREAKING CHANGE: ui.table row and cell press handlers had their first parameters of row/column index removed. These were an unsafe way to index the table as the index matches the user's view of the table, not the server's view.

For row events, you can add your own key column and use always_fetch_columns=["my_key_column"]. This column's data will be included with every row press.

For cell events, switch to a row event with the change above if you need to identify the row of the cell.