chore: add a comment

fix(accounts): hide no rows label when there is only pinned row
2023-09-20 12:34:31 -07:00 · 2023-09-20 15:31:04 +02:00
1 changed files with 7 additions and 1 deletions
@@ -312,7 +312,6 @@ export const AccountTable = ({
  ]);

  const data = rowData?.filter((data) => data.asset.id !== pinnedAsset?.id);
-
  return (
    <AgGrid
      {...props}
@@ -321,6 +320,13 @@ export const AccountTable = ({
      rowData={data}
      defaultColDef={defaultColDef}
      columnDefs={colDefs}
+      overlayNoRowsTemplate={
+        // account for the pinned asset is filtered out to prevent duplicate
+        // data in the pinned row and the main table rows. AgGrid will not
+        // consider the pinned row when determining whether to show the no
+        // rows template or not so we need to override it
+        rowData?.length ? '<span />' : props.overlayNoRowsTemplate
+      }
      getRowHeight={getPinnedAssetRowHeight}
      pinnedTopRowData={pinnedRow ? [pinnedRow] : undefined}
    />
Author	SHA1	Message	Date
Matthew Russell	d7b60840da	chore: add a comment	2023-09-20 12:34:31 -07:00
Bartłomiej Głownia	e98725227d	fix(accounts): hide no rows label when there is only pinned row	2023-09-20 15:31:04 +02:00