@dxos/react-ui-table 0.6.13-main.ed424a1 → 0.6.13

package/src/components/ColumnMenu/ColumnMenu.stories.tsx

@@ -7,12 +7,11 @@ import '@dxos-theme';
 import { type SortDirection } from '@tanstack/react-table';
 import React, { useState } from 'react';
 
-import { FieldValueType } from '@dxos/schema';
 import { withTheme } from '@dxos/storybook-utils';
 
 import { ColumnMenu } from './ColumnMenu';
-import { type ColumnSettingsProps } from './ColumnSettings';
-import { type ColumnDef } from '../../schema';
+import { type ColumnSettingsFormProps } from './ColumnSettingsForm';
+import { type ColumnProps } from '../../schema';
 
 export default {
   title: 'react-ui-table/ColumnMenu',
@@ -20,27 +19,27 @@ export default {
   decorators: [withTheme],
 };
 
-const defs: Pick<ColumnSettingsProps, 'tableDef' | 'tablesToReference'> = {
+const defs: Pick<ColumnSettingsFormProps, 'tableDef' | 'tablesToReference'> = {
   tableDef: {
     id: '1',
     name: 'table 1',
-    columns: [{ id: 'col-1', prop: 'one', type: FieldValueType.String }],
+    columns: [{ id: 'col-1', prop: 'one-foo', type: 'string' }],
   },
   tablesToReference: [
     {
       id: '1',
       name: 'table 1',
-      columns: [{ id: 'col-1', prop: 'one', type: FieldValueType.String }],
+      columns: [{ id: 'col-1', prop: 'one-foo', type: 'string' }],
     },
     {
       id: '2',
       name: 'table 2',
-      columns: [{ id: 'col-1', prop: 'two', type: FieldValueType.String }],
+      columns: [{ id: 'col-1', prop: 'two-foo', type: 'string' }],
     },
   ],
 };
 
-const functions: Pick<ColumnSettingsProps, 'onUpdate' | 'onDelete'> = {
+const funcs: Pick<ColumnSettingsFormProps, 'onUpdate' | 'onDelete'> = {
   onUpdate: (...args: any) => console.log('onUpdate args', args),
   onDelete: (...args: any) => console.log('onDelete args', args),
 };
@@ -50,10 +49,10 @@ export const Default = {
     column: {
       id: 'test',
       label: 'test',
-    } as ColumnDef,
+    } as ColumnProps,
 
     ...defs,
-    ...functions,
+    ...funcs,
 
     context: {
       header: {
@@ -90,13 +89,11 @@ export const ReactiveSort = {
       column: {
         id: 'test',
         label: 'test',
-      } as ColumnDef,
+      } as ColumnProps,
       context: {
         header: {
           column: {
             getCanSort: () => true,
-            getIsSorted: () => sort,
-            clearSorting: () => setSort(false),
             toggleSorting: (desc?: boolean) => {
               if (desc === undefined) {
                 setSort(false);
@@ -106,6 +103,8 @@ export const ReactiveSort = {
                 setSort('asc');
               }
             },
+            getIsSorted: () => sort,
+            clearSorting: () => setSort(false),
             getToggleSortingHandler: () => () => {
               if (sort === 'asc') {
                 setSort('desc');

package/src/components/ColumnMenu/ColumnMenu.tsx

@@ -9,25 +9,22 @@ import React, { useRef, useState, useCallback } from 'react';
 import { Button, Popover, DropdownMenu } from '@dxos/react-ui';
 import { getSize, mx } from '@dxos/react-ui-theme';
 
-import { ColumnSettings } from './ColumnSettings';
+import { ColumnSettingsForm } from './ColumnSettingsForm';
 import { useSortColumn } from '../../hooks';
-import { type TableDef, type ColumnDef } from '../../schema';
+import { type TableDef, type ColumnProps } from '../../schema';
 
 export type ColumnMenuProps<TData extends RowData, TValue> = {
   context: HeaderContext<TData, TValue>;
   tablesToReference: TableDef[];
   tableDef: TableDef;
-  column: ColumnDef;
+  column: ColumnProps;
   columnOrderable: boolean;
   columnPosition: 'start' | 'end' | undefined;
   onColumnReorder: (columnId: string, direction: 'left' | 'right') => void;
-  onUpdate?: (id: string, column: ColumnDef) => void;
+  onUpdate?: (id: string, column: ColumnProps) => void;
   onDelete?: (id: string) => void;
 };
 
-/**
- * @deprecated
- */
 export const ColumnMenu = <TData extends RowData, TValue>({
   column,
   columnOrderable,
@@ -141,7 +138,7 @@ export const ColumnMenu = <TData extends RowData, TValue>({
           <Popover.Portal>
             <Popover.Content>
               <Popover.Viewport>
-                <ColumnSettings {...props} column={column} onClose={() => setIsColumnSettingsOpen(false)} />
+                <ColumnSettingsForm {...props} column={column} onClose={() => setIsColumnSettingsOpen(false)} />
               </Popover.Viewport>
               <Popover.Arrow />
             </Popover.Content>

package/src/components/ColumnMenu/{ColumnSettings.stories.tsx → ColumnSettingsForm.stories.tsx}

@@ -3,38 +3,25 @@
 //
 
 import '@dxos-theme';
-
 import React from 'react';
 
-import { FieldValueType } from '@dxos/schema';
-import { withTheme, withLayout } from '@dxos/storybook-utils';
+import { withTheme } from '@dxos/storybook-utils';
 
-import { ColumnSettings, type ColumnSettingsProps } from './ColumnSettings';
+import { ColumnSettingsForm, type ColumnSettingsFormProps } from './ColumnSettingsForm';
 import { type TableDef } from '../../schema';
-import translations from '../../translations';
-
-const Story = (props: ColumnSettingsProps) => (
-  <div className='flex w-[240px] m-2 p-2 border border-separator rounded'>
-    <ColumnSettings {...props} />
-  </div>
-);
 
 export default {
-  title: 'react-ui-table/ColumnSettings',
-  component: ColumnSettings,
-  decorators: [withTheme, withLayout()],
-  parameters: {
-    translations,
-  },
-  render: Story,
+  title: 'react-ui-table/ColumnSettingsForm',
+  component: ColumnSettingsForm,
+  decorators: [withTheme],
 };
 
 const sampleTableDef: TableDef = {
   id: '1',
   name: 'Sample Table',
   columns: [
-    { id: 'col-1', prop: 'name', type: FieldValueType.String, label: 'Name' },
-    { id: 'col-2', prop: 'age', type: FieldValueType.Number, label: 'Age', digits: 0 },
+    { id: 'col-1', prop: 'name', type: 'string', label: 'Name' },
+    { id: 'col-2', prop: 'age', type: 'number', label: 'Age', digits: 0 },
   ],
 };
 
@@ -44,17 +31,17 @@ const sampleTablesToReference: TableDef[] = [
     id: '2',
     name: 'Reference Table',
     columns: [
-      { id: 'ref-col-1', prop: 'id', type: FieldValueType.String, label: 'ID' },
-      { id: 'ref-col-2', prop: 'description', type: FieldValueType.String, label: 'Description' },
+      { id: 'ref-col-1', prop: 'id', type: 'string', label: 'ID' },
+      { id: 'ref-col-2', prop: 'description', type: 'string', label: 'Description' },
     ],
   },
 ];
 
-const defaultArgs: ColumnSettingsProps = {
+const defaultArgs: ColumnSettingsFormProps = {
   column: {
     id: 'col-1',
     prop: 'name',
-    type: FieldValueType.String,
+    type: 'string',
     label: 'Name',
   },
   tableDef: sampleTableDef,
@@ -74,7 +61,7 @@ export const NumberColumn = {
     column: {
       id: 'col-2',
       prop: 'age',
-      type: FieldValueType.Number,
+      type: 'number',
       label: 'Age',
       digits: 0,
     },
@@ -87,7 +74,7 @@ export const ReferenceColumn = {
     column: {
       id: 'col-3',
       prop: 'reference',
-      type: FieldValueType.Ref,
+      type: 'ref',
       label: 'Reference',
       refTable: '2',
       refProp: 'ref-col-1',
@@ -101,8 +88,17 @@ export const NewColumn = {
     column: {
       id: 'new-col',
       prop: '',
-      type: FieldValueType.String,
+      type: 'string',
       label: '',
     },
   },
 };
+
+const Template = (args: ColumnSettingsFormProps) => (
+  <div className='p-4 max-is-64'>
+    <ColumnSettingsForm {...args} />
+  </div>
+);
+
+export const Interactive = Template.bind({});
+(Interactive as any).args = defaultArgs;

package/src/components/ColumnMenu/{ColumnSettings.tsx → ColumnSettingsForm.tsx}

@@ -1,39 +1,31 @@
 //
 // Copyright 2024 DXOS.org
 //
-
 import React, { useRef, useState } from 'react';
 
 import { Button, Input, Select, useTranslation } from '@dxos/react-ui';
-import { FieldValueTypes, type FieldValueType } from '@dxos/schema';
 import { safeParseInt } from '@dxos/util';
 
-import { type TableDef, type ColumnDef } from '../../schema';
+import { type TableDef, type ColumnProps, type ColumnType } from '../../schema';
 import { translationKey } from '../../translations';
 
-export type ColumnSettingsProps = {
-  column: ColumnDef;
+export type ColumnSettingsFormProps = {
+  column: ColumnProps;
   tableDef: TableDef;
-  // TODO(burdon): Rename.
   tablesToReference: TableDef[];
-  onUpdate?: (id: string, column: ColumnDef) => void;
-  onDelete?: (id: string) => void;
+  onUpdate?: ((id: string, column: ColumnProps) => void) | undefined;
+  onDelete?: ((id: string) => void) | undefined;
   onClose?: () => void;
 };
 
-/**
- * @deprecated
- */
-export const ColumnSettings = ({
+export const ColumnSettingsForm = ({
   column,
   tableDef,
   tablesToReference,
   onUpdate,
   onDelete,
   onClose,
-}: ColumnSettingsProps) => {
-  const { t } = useTranslation(translationKey);
-  const propRef = useRef<HTMLInputElement>(null);
+}: ColumnSettingsFormProps) => {
   const [formState, setFormState] = useState({
     prop: column.id,
     refTable: column.refTable,
@@ -43,6 +35,9 @@ export const ColumnSettings = ({
     digits: String(column.digits ?? '0'),
   });
 
+  const propRef = useRef<HTMLInputElement>(null);
+  const { t } = useTranslation(translationKey);
+
   const handleSave = () => {
     const { prop, refTable, refProp, type, label, digits } = formState;
 
@@ -67,7 +62,7 @@ export const ColumnSettings = ({
       id: prop, // TODO(burdon): Make unique.
       prop,
       label,
-      type: type as FieldValueType,
+      type: type as ColumnProps['type'],
       refTable,
       refProp,
       digits: safeParseInt(digits),
@@ -80,10 +75,9 @@ export const ColumnSettings = ({
     onClose?.();
   };
 
-  // TODO(burdon): Standardize dialog/popup.
   return (
-    <div className='flex flex-col w-full gap-4'>
-      <div className='flex flex-col gap-2'>
+    <div className='p-1 space-y-2'>
+      <div className='space-y-1'>
         <Input.Root>
           <Input.Label>{t('column label label')}</Input.Label>
           <Input.TextInput
@@ -94,7 +88,8 @@ export const ColumnSettings = ({
             data-testid='table.column-settings.label'
           />
         </Input.Root>
-
+      </div>
+      <div className='space-y-1'>
         <Input.Root>
           <Input.Label>{t('property key label')}</Input.Label>
           <Input.TextInput
@@ -106,7 +101,8 @@ export const ColumnSettings = ({
             }
           />
         </Input.Root>
-
+      </div>
+      <div className='space-y-1'>
         <Input.Root>
           <Input.Label>{t('column type label')}</Input.Label>
           <Select.Root
@@ -117,7 +113,7 @@ export const ColumnSettings = ({
             <Select.Portal>
               <Select.Content>
                 <Select.Viewport>
-                  {FieldValueTypes.map((type) => (
+                  {(['number', 'boolean', 'string', 'ref'] as ColumnType[]).map((type) => (
                     <Select.Option key={type} value={type}>
                       {t(`${type} column type label`)}
                     </Select.Option>
@@ -127,8 +123,9 @@ export const ColumnSettings = ({
             </Select.Portal>
           </Select.Root>
         </Input.Root>
-
-        {formState.type === 'number' && (
+      </div>
+      {formState.type === 'number' && (
+        <div className='space-y-1'>
           <Input.Root>
             <Input.Label>{t('digits label')}</Input.Label>
             <Input.TextInput
@@ -136,10 +133,12 @@ export const ColumnSettings = ({
               onChange={(event) => setFormState((prevState) => ({ ...prevState, digits: event.target.value }))}
             />
           </Input.Root>
-        )}
+        </div>
+      )}
 
-        {formState.type === 'ref' && (
-          <>
+      {formState.type === 'ref' && (
+        <>
+          <div className='space-y-1'>
             <Input.Root>
               <Input.Label>Table</Input.Label>
               <Select.Root
@@ -162,8 +161,9 @@ export const ColumnSettings = ({
                 </Select.Portal>
               </Select.Root>
             </Input.Root>
-
-            {formState.refTable && (
+          </div>
+          {formState.refTable && (
+            <div className='space-y-1'>
               <Input.Root>
                 <Input.Label>Table property</Input.Label>
                 <Select.Root
@@ -186,21 +186,20 @@ export const ColumnSettings = ({
                   </Select.Portal>
                 </Select.Root>
               </Input.Root>
-            )}
-          </>
-        )}
-      </div>
-
-      <div>
-        {/* TODO(burdon): Change delete to cancel. Delete should be menu item. */}
-        <div className='flex flex-row justify-end gap-1'>
-          <Button onClick={handleDelete} data-testid='table.column-settings.delete'>
-            {t('delete label')}
-          </Button>
-          <Button variant='primary' onClick={handleSave} data-testid='table.column-settings.save'>
-            {t('save label')}
-          </Button>
-        </div>
+            </div>
+          )}
+        </>
+      )}
+
+      <div role='none' className='h-1' />
+
+      <div className='flex flex-row justify-end gap-1'>
+        <Button onClick={handleDelete} data-testid='table.column-settings.delete'>
+          {t('delete label')}
+        </Button>
+        <Button variant='primary' onClick={handleSave} data-testid='table.column-settings.save'>
+          {t('save label')}
+        </Button>
       </div>
     </div>
   );

package/src/components/ColumnMenu/index.ts

@@ -3,4 +3,4 @@
 //
 
 export * from './ColumnMenu';
-export * from './ColumnSettings';
+export * from './ColumnSettingsForm';

package/src/components/Table/Table.stories.tsx

@@ -8,11 +8,11 @@ import { Plugs, PlugsConnected } from '@phosphor-icons/react';
 import React, { useCallback, useEffect, useMemo, useState } from 'react';
 
 import { create } from '@dxos/echo-schema';
+import { registerSignalRuntime } from '@dxos/echo-signals/react';
 import { PublicKey } from '@dxos/keys';
-import { log } from '@dxos/log';
 import { faker } from '@dxos/random';
 import { Button, DensityProvider } from '@dxos/react-ui';
-import { withSignals, withTheme } from '@dxos/storybook-utils';
+import { withTheme } from '@dxos/storybook-utils';
 import { range } from '@dxos/util';
 
 import { Table } from './Table';
@@ -406,117 +406,105 @@ export const TenThousandRows = {
   },
 };
 
-// TODO(burdon): Warning: Encountered two children with the same key.
+registerSignalRuntime();
+const state = create({ items: createItems(10) });
 
-type StoryProps = {
-  periodicMutations: boolean;
-  mutationInterval: number;
-  periodicDeletions: boolean;
-  deletionInterval: number;
-  periodicInsertions: boolean;
-  insertionInterval: number;
-};
-
-const Story = ({
-  periodicMutations,
-  mutationInterval,
-  periodicDeletions,
-  deletionInterval,
-  periodicInsertions,
-  insertionInterval,
-}: StoryProps) => {
-  const [state] = useState(() => create({ items: createItems(10) }));
-
-  useEffect(() => {
-    if (!periodicMutations) {
-      return;
-    }
-
-    const i = setInterval(() => {
-      if (state.items.length === 0) {
+export const RealTimeUpdates = {
+  args: {
+    periodicMutations: true,
+    mutationInterval: 1000,
+    periodicDeletions: false,
+    deletionInterval: 5000,
+    periodicInsertions: false,
+    insertionInterval: 3000,
+  },
+  argTypes: {
+    mutationInterval: { control: 'number' },
+    periodicDeletions: { control: 'boolean' },
+    deletionInterval: { control: 'number' },
+    periodicInsertions: { control: 'boolean' },
+    insertionInterval: { control: 'number' },
+  },
+  render: ({
+    periodicMutations,
+    mutationInterval,
+    periodicDeletions,
+    deletionInterval,
+    periodicInsertions,
+    insertionInterval,
+  }: any) => {
+    useEffect(() => {
+      if (!periodicMutations) {
         return;
       }
 
-      const idx = Math.floor(Math.random() * state.items.length);
-      log('mutating', { idx });
-      state.items[idx].name = faker.commerce.productName();
-      state.items[idx].count = Math.floor(Math.random() * 1000);
-      state.items[idx].started = new Date();
-    }, mutationInterval);
+      const interval = setInterval(() => {
+        if (state.items.length === 0) {
+          return;
+        }
 
-    return () => clearInterval(i);
-  }, [periodicMutations, mutationInterval, state.items]);
+        const randomIndex = Math.floor(Math.random() * state.items.length);
+        console.log('Mutating row', randomIndex);
+        state.items[randomIndex].name = faker.commerce.productName();
+        state.items[randomIndex].count = Math.floor(Math.random() * 1000);
+        state.items[randomIndex].started = new Date();
+      }, mutationInterval);
 
-  useEffect(() => {
-    if (!periodicInsertions) {
-      return;
-    }
+      return () => clearInterval(interval);
+    }, [periodicMutations, mutationInterval, state.items]);
 
-    const i = setInterval(() => {
-      log('inserting...');
-      state.items.push(createItems(1)[0]);
-    }, insertionInterval);
+    useEffect(() => {
+      if (!periodicInsertions) {
+        return;
+      }
 
-    return () => clearInterval(i);
-  }, [periodicInsertions, insertionInterval, state.items]);
+      const interval = setInterval(() => {
+        console.log('Inserting...');
+        state.items.push(createItems(1)[0]);
+      }, insertionInterval);
 
-  useEffect(() => {
-    if (!periodicDeletions) {
-      return;
-    }
+      return () => clearInterval(interval);
+    }, [periodicInsertions, insertionInterval, state.items]);
 
-    const i = setInterval(() => {
-      if (state.items.length === 0) {
+    useEffect(() => {
+      if (!periodicDeletions) {
         return;
       }
 
-      log('deleting...');
-      const randomIndex = Math.floor(Math.random() * state.items.length);
-      state.items.splice(randomIndex, 1);
-    }, deletionInterval);
+      const interval = setInterval(() => {
+        if (state.items.length === 0) {
+          return;
+        }
 
-    return () => clearInterval(i);
-  }, [periodicDeletions, deletionInterval, state.items]);
+        console.log('Deleting');
 
-  const onUpdate = useCallback((...args: any[]) => {}, []);
-  const columns = useMemo(() => makeColumns(onUpdate), []);
+        // Randomly delete a row from state.items
+        const randomIndex = Math.floor(Math.random() * state.items.length);
+        state.items.splice(randomIndex, 1);
+      }, deletionInterval);
 
-  return (
-    <Table.Root>
-      <Table.Viewport classNames='fixed inset-0'>
-        <Table.Main<Item>
-          role='grid'
-          rowsSelectable='multi'
-          keyAccessor={(row) => row.publicKey.toHex()}
-          columns={columns}
-          data={state.items}
-          fullWidth
-          stickyHeader
-          border
-          pinLastRow
-        />
-      </Table.Viewport>
-    </Table.Root>
-  );
-};
+      return () => clearInterval(interval);
+    }, [periodicDeletions, deletionInterval, state.items]);
 
-export const RealTimeUpdates = {
-  argTypes: {
-    periodicMutations: { control: 'boolean' },
-    mutationInterval: { control: 'number' },
-    periodicDeletions: { control: 'boolean' },
-    deletionInterval: { control: 'number' },
-    periodicInsertions: { control: 'boolean' },
-    insertionInterval: { control: 'number' },
+    const onUpdate = useCallback((...args: any[]) => {}, []);
+    const columns = useMemo(() => makeColumns(onUpdate), []);
+
+    return (
+      <Table.Root>
+        <Table.Viewport classNames='fixed inset-0'>
+          <Table.Main<Item>
+            role='grid'
+            rowsSelectable='multi'
+            keyAccessor={(row) => row.publicKey.toHex()}
+            columns={columns}
+            data={state.items}
+            fullWidth
+            stickyHeader
+            border
+            pinLastRow
+          />
+        </Table.Viewport>
+      </Table.Root>
+    );
   },
-  args: {
-    periodicMutations: true,
-    mutationInterval: 1000,
-    periodicDeletions: false,
-    deletionInterval: 5000,
-    periodicInsertions: false,
-    insertionInterval: 3000,
-  } satisfies StoryProps,
-  decorators: [withSignals],
-  render: Story,
 };