Skip to content

Commit b8ed8e4

Browse files
Copilotdsmmcken
andcommitted
Rewrite best practice tip as shorter body text without callout formatting
Co-authored-by: dsmmcken <1576283+dsmmcken@users.noreply.github.com>
1 parent a951480 commit b8ed8e4

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

plugins/ui/docs/components/table.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -433,8 +433,7 @@ Deephaven only fetches data for visible rows and columns within a window around
433433

434434
The `always_fetch_columns` prop takes a single column name, a list of column names, or a boolean to always fetch all columns. The data for these columns is included in row event data (e.g. `on_row_press`) and context menu callbacks.
435435

436-
> [!TIP]
437-
> **Best Practice for Event Callbacks**: When using event callbacks (such as `on_row_press`, `on_cell_press`, or `on_selection_change`), it's recommended to include any column you will be using in the callback in `always_fetch_columns`. This prevents potentially undefined columns if users hide those columns or scroll far right where the original columns are no longer in the viewport.
436+
When using event callbacks, include any columns referenced in the callback in `always_fetch_columns` to prevent undefined columns when users hide columns or scroll beyond the viewport.
438437

439438
> [!WARNING]
440439
> Setting `always_fetch_columns` to `True` will fetch all columns and can be slow for tables with many columns.

0 commit comments

Comments
 (0)