@dxos/react-ui-form 0.8.2-staging.7ac8446 → 0.8.2

package/src/components/Form/Form.stories.tsx

@@ -5,36 +5,40 @@
 import '@dxos-theme';
 
 import { type Meta, type StoryObj } from '@storybook/react';
+import { Schema } from 'effect';
 import React, { useCallback, useState } from 'react';
 
-import { AST, type BaseObject, Format, S } from '@dxos/echo-schema';
+import { ContactType } from '@dxos/client/testing';
+import { type BaseObject, Expando, Format, getDXN, Ref, type TypeAnnotation } from '@dxos/echo-schema';
+import { live } from '@dxos/live-object';
+import { withSurfaceVariantsLayout } from '@dxos/react-ui/testing';
 import { Testing } from '@dxos/schema/testing';
-import { withLayout, withTheme } from '@dxos/storybook-utils';
+import { withTheme } from '@dxos/storybook-utils';
 
 import { SelectInput } from './Defaults';
 import { Form, type FormProps } from './Form';
 import translations from '../../translations';
 import { TestLayout, TestPanel } from '../testing';
 
-const AddressSchema = S.Struct({
-  street: S.optional(S.String.annotations({ [AST.TitleAnnotationId]: 'Street' })),
-  city: S.optional(S.String.annotations({ [AST.TitleAnnotationId]: 'City' })),
-  zip: S.optional(S.String.pipe(S.pattern(/^\d{5}(-\d{4})?$/)).annotations({ [AST.TitleAnnotationId]: 'ZIP' })),
-  location: S.optional(Format.GeoPoint.annotations({ [AST.TitleAnnotationId]: 'Location' })),
-}).annotations({ [AST.TitleAnnotationId]: 'Address' });
+const AddressSchema = Schema.Struct({
+  street: Schema.optional(Schema.String.annotations({ title: 'Street' })),
+  city: Schema.optional(Schema.String.annotations({ title: 'City' })),
+  zip: Schema.optional(Schema.String.pipe(Schema.pattern(/^\d{5}(-\d{4})?$/)).annotations({ title: 'ZIP' })),
+  location: Schema.optional(Format.GeoPoint.annotations({ title: 'Location' })),
+}).annotations({ title: 'Address' });
 
 // TODO(burdon): Translations?
-const TestSchema = S.Struct({
-  name: S.optional(S.String.annotations({ [AST.TitleAnnotationId]: 'Name' })),
-  active: S.optional(S.Boolean.annotations({ [AST.TitleAnnotationId]: 'Active' })),
-  rank: S.optional(S.Number.annotations({ [AST.TitleAnnotationId]: 'Rank' })),
-  website: S.optional(Format.URL.annotations({ [AST.TitleAnnotationId]: 'Website' })),
-  address: S.optional(AddressSchema),
-}).pipe(S.mutable);
+const TestSchema = Schema.Struct({
+  name: Schema.optional(Schema.String.annotations({ title: 'Name' })),
+  active: Schema.optional(Schema.Boolean.annotations({ title: 'Active' })),
+  rank: Schema.optional(Schema.Number.annotations({ title: 'Rank' })),
+  website: Schema.optional(Format.URL.annotations({ title: 'Website' })),
+  address: Schema.optional(AddressSchema),
+}).pipe(Schema.mutable);
 
-type TestType = S.Schema.Type<typeof TestSchema>;
+type TestType = Schema.Schema.Type<typeof TestSchema>;
 
-type StoryProps<T extends BaseObject> = { schema: S.Schema<T> } & FormProps<T>;
+type StoryProps<T extends BaseObject> = { schema: Schema.Schema<T> } & FormProps<T>;
 
 const DefaultStory = <T extends BaseObject>({ schema, values: initialValues, ...props }: StoryProps<T>) => {
   const [values, setValues] = useState(initialValues);
@@ -42,6 +46,15 @@ const DefaultStory = <T extends BaseObject>({ schema, values: initialValues, ...
     setValues(values);
   }, []);
 
+  return <Form<T> schema={schema} values={values} onSave={handleSave} {...props} />;
+};
+
+const DebugStory = <T extends BaseObject>({ schema, values: initialValues, ...props }: StoryProps<T>) => {
+  const [values, setValues] = useState(initialValues);
+  const handleSave = useCallback<NonNullable<FormProps<T>['onSave']>>((values) => {
+    setValues(values);
+  }, []);
+
   return (
     <TestLayout json={{ values, schema: schema.ast.toJSON() }}>
       <TestPanel>
@@ -54,11 +67,17 @@ const DefaultStory = <T extends BaseObject>({ schema, values: initialValues, ...
 const meta: Meta<StoryProps<any>> = {
   title: 'ui/react-ui-form/Form',
   component: Form,
-  render: DefaultStory,
-  decorators: [withLayout({ fullscreen: true, tooltips: true }), withTheme],
+  render: DebugStory,
+  decorators: [withTheme],
   parameters: {
     translations,
   },
+  argTypes: {
+    readonly: {
+      control: 'boolean',
+      description: 'Readonly',
+    },
+  },
 };
 
 export default meta;
@@ -66,6 +85,8 @@ export default meta;
 type Story<T extends BaseObject> = StoryObj<StoryProps<T>>;
 
 export const Default: Story<TestType> = {
+  render: DefaultStory,
+  decorators: [withSurfaceVariantsLayout(), withTheme],
   args: {
     schema: TestSchema,
     values: {
@@ -75,27 +96,30 @@ export const Default: Story<TestType> = {
         zip: '11205',
       },
     },
+    readonly: false,
   },
 };
 
-export const Org: Story<Testing.Org> = {
+export const Organization: Story<Testing.Organization> = {
   args: {
-    schema: Testing.OrgSchema,
+    schema: Testing.OrganizationSchema,
     values: {
       name: 'DXOS',
-      // website: 'https://dxos.org',
+      website: 'https://dxos.org',
     },
+    readonly: false,
   },
 };
 
-export const OrgAutoSave: Story<Testing.Org> = {
+export const OrganizationAutoSave: Story<Testing.Organization> = {
   args: {
-    schema: Testing.OrgSchema,
+    schema: Testing.OrganizationSchema,
     values: {
       name: 'DXOS',
-      // website: 'https://dxos.org',
+      website: 'https://dxos.org',
     },
     autoSave: true,
+    readonly: false,
   },
 };
 
@@ -111,96 +135,58 @@ export const OrgAutoSave: Story<Testing.Org> = {
 //   },
 // };
 
-//
-// TODO(burdon): Move into separate storybook and use test types.
-//
-
-const ShapeSchema = S.Struct({
-  shape: S.optional(
-    S.Union(
-      S.Struct({
-        type: S.Literal('circle').annotations({ [AST.TitleAnnotationId]: 'Type' }),
-        radius: S.optional(S.Number.annotations({ [AST.TitleAnnotationId]: 'Radius' })),
+const ShapeSchema = Schema.Struct({
+  shape: Schema.optional(
+    Schema.Union(
+      Schema.Struct({
+        type: Schema.Literal('circle').annotations({ title: 'Type' }),
+        radius: Schema.optional(Schema.Number.annotations({ title: 'Radius' })),
       }),
-      S.Struct({
-        type: S.Literal('square').annotations({ [AST.TitleAnnotationId]: 'Type' }),
-        size: S.optional(S.Number.pipe(S.nonNegative()).annotations({ [AST.TitleAnnotationId]: 'Size' })),
+      Schema.Struct({
+        type: Schema.Literal('square').annotations({ title: 'Type' }),
+        size: Schema.optional(Schema.Number.pipe(Schema.nonNegative()).annotations({ title: 'Size' })),
       }),
-    ).annotations({ [AST.TitleAnnotationId]: 'Shape' }),
+    ).annotations({ title: 'Shape' }),
   ),
-}).pipe(S.mutable);
-
-type ShapeType = S.Schema.Type<typeof ShapeSchema>;
+}).pipe(Schema.mutable);
 
-type DiscriminatedUnionStoryProps = FormProps<ShapeType>;
+type ShapeType = Schema.Schema.Type<typeof ShapeSchema>;
 
-const DiscriminatedUnionStory = ({ values: initialValues }: DiscriminatedUnionStoryProps) => {
-  const [values, setValues] = useState(initialValues);
-  const handleSave = useCallback<NonNullable<FormProps<ShapeType>['onSave']>>((values) => {
-    setValues(values);
-  }, []);
-
-  return (
-    <TestLayout json={{ values, schema: ShapeSchema.ast.toJSON() }}>
-      <TestPanel>
-        <Form<ShapeType>
-          schema={ShapeSchema}
-          values={values}
-          onSave={handleSave}
-          Custom={{
-            ['shape.type' as const]: (props) => (
-              <SelectInput
-                {...props}
-                options={['circle', 'square'].map((value) => ({
-                  value,
-                  label: value,
-                }))}
-              />
-            ),
-          }}
-        />
-      </TestPanel>
-    </TestLayout>
-  );
-};
-
-export const DiscriminatedShape: StoryObj<DiscriminatedUnionStoryProps> = {
-  render: DiscriminatedUnionStory,
+export const DiscriminatedShape: Story<ShapeType> = {
   args: {
+    schema: ShapeSchema,
+    readonly: false,
     values: {
       shape: {
         type: 'circle',
         radius: 5,
       },
     },
+    Custom: {
+      ['shape.type' as const]: (props) => (
+        <SelectInput
+          {...props}
+          options={['circle', 'square'].map((value) => ({
+            value,
+            label: value,
+          }))}
+        />
+      ),
+    },
   },
 };
 
-const ArraysSchema = S.Struct({
-  names: S.Array(S.String.pipe(S.nonEmptyString())),
-  addresses: S.Array(AddressSchema),
-}).pipe(S.mutable);
-
-type ArraysType = S.Schema.Type<typeof ArraysSchema>;
+const ArraysSchema = Schema.Struct({
+  names: Schema.Array(Schema.String.pipe(Schema.nonEmptyString())),
+  addresses: Schema.Array(AddressSchema),
+}).pipe(Schema.mutable);
 
-const ArraysStory = ({ values: initialValues }: FormProps<ArraysType>) => {
-  const [values, setValues] = useState(initialValues);
-  const handleSave = useCallback<NonNullable<FormProps<ArraysType>['onSave']>>((values) => {
-    setValues(values);
-  }, []);
-
-  return (
-    <TestLayout json={{ values, schema: ArraysSchema.ast.toJSON() }}>
-      <TestPanel>
-        <Form<ArraysType> schema={ArraysSchema} values={values} onSave={handleSave} />
-      </TestPanel>
-    </TestLayout>
-  );
-};
+type ArraysType = Schema.Schema.Type<typeof ArraysSchema>;
 
 export const Arrays: StoryObj<FormProps<ArraysType>> = {
-  render: ArraysStory,
   args: {
+    schema: ArraysSchema,
+    readonly: false,
     values: {
       names: ['Alice', 'Bob'],
       addresses: [],
@@ -208,34 +194,73 @@ export const Arrays: StoryObj<FormProps<ArraysType>> = {
   },
 };
 
-const ColorSchema = S.Struct({
-  color: S.Union(S.Literal('red'), S.Literal('green'), S.Literal('blue')).annotations({
-    [AST.TitleAnnotationId]: 'Color',
+const ColorSchema = Schema.Struct({
+  color: Schema.Union(Schema.Literal('red'), Schema.Literal('green'), Schema.Literal('blue')).annotations({
+    title: 'Color',
   }),
-}).pipe(S.mutable);
+}).pipe(Schema.mutable);
+
+type ColorType = Schema.Schema.Type<typeof ColorSchema>;
+
+export const Enum: StoryObj<FormProps<ColorType>> = {
+  args: {
+    schema: ColorSchema,
+    readonly: false,
+    values: {
+      color: 'red',
+    },
+  },
+};
 
-type ColorType = S.Schema.Type<typeof ColorSchema>;
+const RefSchema = Schema.Struct({
+  contact: Ref(ContactType).annotations({ title: 'Contact Reference' }),
+  optionalContact: Schema.optional(Ref(ContactType).annotations({ title: 'Optional Contact Reference' })),
+  refArray: Schema.optional(Schema.Array(Ref(ContactType))),
+  unknownExpando: Schema.optional(Ref(Expando).annotations({ title: 'Optional Ref to an Expando (DXN Input)' })),
+});
 
-const EnumStory = ({ values: initialValues }: FormProps<ColorType>) => {
+const contact1 = live(ContactType, { identifiers: [] });
+const contact2 = live(ContactType, { identifiers: [] });
+
+const RefStory = ({ values: initialValues, readonly }: FormProps<any>) => {
   const [values, setValues] = useState(initialValues);
-  const handleSave = useCallback<NonNullable<FormProps<ColorType>['onSave']>>((values) => {
+  const handleSave = useCallback<NonNullable<FormProps<any>['onSave']>>((values) => {
     setValues(values);
   }, []);
 
+  const onQueryRefOptions = useCallback((typeInfo: TypeAnnotation) => {
+    switch (typeInfo.typename) {
+      case ContactType.typename:
+        return [
+          { dxn: getDXN(contact1)!, label: 'John Coltraine' },
+          { dxn: getDXN(contact2)!, label: 'Erykah Badu' },
+        ];
+      default:
+        return [];
+    }
+  }, []);
+
   return (
-    <TestLayout json={{ values, schema: ColorSchema.ast.toJSON() }}>
+    <TestLayout json={{ values, schema: RefSchema.ast.toJSON() }}>
       <TestPanel>
-        <Form<ColorType> schema={ColorSchema} values={values} onSave={handleSave} />
+        <Form
+          schema={RefSchema}
+          values={values}
+          readonly={readonly}
+          onSave={handleSave}
+          onQueryRefOptions={onQueryRefOptions}
+        />
       </TestPanel>
     </TestLayout>
   );
 };
 
-export const Enum: StoryObj<FormProps<ColorType>> = {
-  render: EnumStory,
+export const Refs: StoryObj<FormProps<ContactType>> = {
+  render: RefStory,
   args: {
+    readonly: false,
     values: {
-      color: 'red',
-    },
+      refArray: [Ref.make(contact1), Ref.make(contact2)],
+    } as any,
   },
 };

package/src/components/Form/Form.tsx

@@ -2,9 +2,10 @@
 // Copyright 2024 DXOS.org
 //
 
+import { type Schema } from 'effect';
 import React, { type ReactElement, useEffect, useMemo, useRef } from 'react';
 
-import { type BaseObject, type S, type PropertyKey } from '@dxos/echo-schema';
+import { type BaseObject, type PropertyKey } from '@dxos/echo-schema';
 import { type ThemedClassName } from '@dxos/react-ui';
 import { mx } from '@dxos/react-ui-theme';
 import { type SchemaProperty } from '@dxos/schema';
@@ -13,13 +14,13 @@ import { FormActions } from './FormActions';
 import { FormFields } from './FormContent';
 import { FormProvider } from './FormContext';
 import { type InputProps, type InputComponent } from './Input';
-import { type FormOptions } from '../../hooks';
+import { type FormOptions, type QueryRefOptions } from '../../hooks';
 
 export type PropsFilter<T extends BaseObject> = (props: SchemaProperty<T>[]) => SchemaProperty<T>[];
 
 export type ComponentLookup = (args: {
   prop: string;
-  schema: S.Schema<any>;
+  schema: Schema.Schema<any>;
   inputProps: InputProps;
 }) => ReactElement | undefined;
 
@@ -41,6 +42,7 @@ export type FormProps<T extends BaseObject> = ThemedClassName<
     autoSave?: boolean;
     testId?: string;
     onCancel?: () => void;
+    onQueryRefOptions?: QueryRefOptions;
     lookupComponent?: ComponentLookup;
     /**
      * Map of custom renderers for specific properties.
@@ -65,6 +67,7 @@ export const Form = <T extends BaseObject>({
   onValidate,
   onSave,
   onCancel,
+  onQueryRefOptions,
   lookupComponent,
   Custom,
 }: FormProps<T>) => {
@@ -92,14 +95,14 @@ export const Form = <T extends BaseObject>({
       onValid={onValid}
       onSave={onSave}
     >
-      {/* TODO(burdon): Remove padding. */}
-      <div ref={formRef} role='none' className={mx('p-2', classNames)} data-testid={testId}>
+      <div ref={formRef} role='none' className={mx('flex flex-col gap-4', classNames)} data-testid={testId}>
         <FormFields
           schema={schema}
           path={path}
           readonly={readonly}
           filter={filter}
           sort={sort}
+          onQueryRefOptions={onQueryRefOptions}
           lookupComponent={lookupComponent}
           Custom={Custom}
         />

package/src/components/Form/FormActions.tsx

@@ -5,7 +5,6 @@
 import React from 'react';
 
 import { IconButton, useTranslation } from '@dxos/react-ui';
-import { mx } from '@dxos/react-ui-theme';
 
 import { useFormContext } from './FormContext';
 import { translationKey } from '../../translations';
@@ -15,27 +14,25 @@ export type FormActionsProps = {
   onCancel?: () => void;
 };
 
-export const FormActions = ({ onCancel, readonly }: FormActionsProps) => {
+export const FormActions = ({ readonly, onCancel }: FormActionsProps) => {
   const { t } = useTranslation(translationKey);
   const { canSave, handleSave } = useFormContext();
 
   return (
-    <div role='none' className='mt-2 flex justify-center'>
-      <div role='none' className={mx(onCancel && !readonly && 'grid grid-cols-2 gap-2')}>
-        {onCancel && !readonly && (
-          <IconButton data-testid='cancel-button' icon='ph--x--regular' label={t('button cancel')} onClick={onCancel} />
-        )}
-        {handleSave && (
-          <IconButton
-            type='submit'
-            data-testid='save-button'
-            disabled={!canSave}
-            icon='ph--check--regular'
-            label={t('button save')}
-            onClick={handleSave}
-          />
-        )}
-      </div>
+    <div role='none' className='flex [&_button]:grow gap-1'>
+      {onCancel && !readonly && (
+        <IconButton data-testid='cancel-button' icon='ph--x--regular' label={t('button cancel')} onClick={onCancel} />
+      )}
+      {handleSave && (
+        <IconButton
+          type='submit'
+          data-testid='save-button'
+          disabled={!canSave}
+          icon='ph--check--regular'
+          label={t('button save')}
+          onClick={handleSave}
+        />
+      )}
     </div>
   );
 };