@dxos/react-ui-table 0.7.5-labs.e27f9b9 → 0.7.5-labs.f400bbc

package/src/hooks/useTableModel.ts

@@ -2,10 +2,13 @@
 // Copyright 2024 DXOS.org
 //
 
+import { effect } from '@preact/signals-core';
 import { useEffect, useState } from 'react';
 
 import { type ReactiveObject } from '@dxos/react-client/echo';
+import { useSelectionActions } from '@dxos/react-ui-attention';
 import { type ViewProjection } from '@dxos/schema';
+import { isNonNullable } from '@dxos/util';
 
 import { type BaseTableRow, TableModel, type TableModelProps } from '../model';
 import { type TableType } from '../types';
@@ -48,5 +51,23 @@ export const useTableModel = <T extends BaseTableRow = { id: string }>({
     }
   }, [model, objects]);
 
+  const { select, clear } = useSelectionActions([table?.id, table?.view?.target?.query.type].filter(isNonNullable));
+
+  useEffect(() => {
+    if (!model) {
+      return;
+    }
+
+    const unsubscribe = effect(() => {
+      select([...model.selection.selection.value]);
+    });
+
+    // Maybe clear the selection here?
+    return () => {
+      clear();
+      unsubscribe();
+    };
+  }, [model]);
+
   return model;
 };

package/src/model/table-model.ts

@@ -1,5 +1,5 @@
 //
-// Copyright 2024 DXOS.org
+// Copyright 2025 DXOS.org
 //
 
 import { computed, effect, signal, type ReadonlySignal } from '@preact/signals-core';

package/src/playwright/smoke.spec.ts

@@ -145,7 +145,7 @@ test.describe('Table', () => {
     await page.getByRole('option', { name: 'Anita Mayer' }).click();
 
     // Assert that the value is shown in the cell.
-    await expect(page.getByRole('gridcell', { name: 'Anita Mayer' })).toBeVisible();
+    await expect(page.getByRole('gridcell', { name: 'Anita Mayer' }).first()).toBeVisible();
 
     // Create new object.
     await table.grid.cell(4, 1, 'grid').click();
@@ -157,7 +157,7 @@ test.describe('Table', () => {
     await page.getByTestId('save-button').click();
 
     // Scroll to the left.
-    await page.getByRole('gridcell', { name: 'test' }).click();
+    await page.getByRole('gridcell', { name: 'test' }).first().click();
 
     await page.close();
   });
@@ -173,7 +173,7 @@ test.describe('Table', () => {
     await page.getByTestId('table-switch').nth(7).click();
 
     // Test that checks are durable in the data model by sorting.
-    await table.sortColumn(1, 'descending');
+    await table.sortColumn(3, 'descending');
 
     // Assert the first two switch checkboxes are checked.
     await expect(page.getByTestId('table-switch').first()).toBeChecked();

package/src/util/table.ts

@@ -46,7 +46,7 @@ export const initializeTable = async ({
     // We need to get the schema properties here. For now, only simple types and refs, not compound types
     // are going to be supported.
     const fields = getSchemaProperties(schema.ast)
-      .filter((prop) => prop.type !== 'object' || prop.format === FormatEnum.Ref)
+      .filter((prop) => prop.format !== FormatEnum.Ref)
       .map((prop) => prop.name);
 
     table.view = makeRef(

package/dist/types/src/playwright/playwright.config.d.ts

@@ -1,3 +0,0 @@
-declare const _default: import("@playwright/test").PlaywrightTestConfig<{}, {}>;
-export default _default;
-//# sourceMappingURL=playwright.config.d.ts.map

package/dist/types/src/playwright/playwright.config.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"playwright.config.d.ts","sourceRoot":"","sources":["../../../../src/playwright/playwright.config.ts"],"names":[],"mappings":";AASA,wBAQG"}