@dxos/react-ui-form 0.8.3-main.672df60 → 0.8.3-staging.0fa589b

package/src/components/ViewEditor/ViewEditor.tsx

@@ -8,16 +8,17 @@ import React, { useCallback, useMemo, useState } from 'react';
 import { type SchemaRegistry } from '@dxos/echo-db';
 import { EchoSchema, Format, type JsonProp, isMutable, toJsonSchema } from '@dxos/echo-schema';
 import { invariant } from '@dxos/invariant';
-import { Icon, IconButton, Message, type ThemedClassName, useTranslation } from '@dxos/react-ui';
+import { Callout, IconButton, type ThemedClassName, useTranslation } from '@dxos/react-ui';
 import { List } from '@dxos/react-ui-list';
+import { cardSpacing } from '@dxos/react-ui-stack';
 import { ghostHover, inputTextLabel, mx } from '@dxos/react-ui-theme';
 import { FieldSchema, type FieldType, type ViewType, ViewProjection, VIEW_FIELD_LIMIT } from '@dxos/schema';
 
 import { translationKey } from '../../translations';
 import { FieldEditor } from '../FieldEditor';
-import { Form } from '../Form';
+import { Form, type FormProps } from '../Form';
 
-const grid = 'grid grid-cols-[32px_1fr_32px] min-bs-[2.5rem]';
+const grid = 'grid grid-cols-[32px_1fr_32px_32px] min-bs-[2.5rem]';
 
 const ViewMetaSchema = Schema.Struct({
   name: Schema.String.annotations({
@@ -30,15 +31,17 @@ const ViewMetaSchema = Schema.Struct({
 
 type ViewMetaType = Schema.Schema.Type<typeof ViewMetaSchema>;
 
-export type ViewEditorProps = ThemedClassName<{
-  schema: Schema.Schema.AnyNoContext;
-  view: ViewType;
-  registry?: SchemaRegistry;
-  readonly?: boolean;
-  showHeading?: boolean;
-  onTypenameChanged?: (typename: string) => void;
-  onDelete?: (fieldId: string) => void;
-}>;
+export type ViewEditorProps = ThemedClassName<
+  {
+    schema: Schema.Schema.AnyNoContext;
+    view: ViewType;
+    registry?: SchemaRegistry;
+    readonly?: boolean;
+    showHeading?: boolean;
+    onTypenameChanged?: (typename: string) => void;
+    onDelete?: (fieldId: string) => void;
+  } & Pick<FormProps<any>, 'outerSpacing'>
+>;
 
 /**
  * Schema-based object form.
@@ -48,10 +51,11 @@ export const ViewEditor = ({
   schema,
   view,
   registry,
-  readonly,
+  readonly: _readonly,
   showHeading = false,
   onTypenameChanged,
   onDelete,
+  outerSpacing = true,
 }: ViewEditorProps) => {
   const { t } = useTranslation(translationKey);
   const projection = useMemo(() => {
@@ -60,7 +64,7 @@ export const ViewEditor = ({
     return new ViewProjection(jsonSchema, view);
   }, [schema, view]);
   const [field, setField] = useState<FieldType>();
-  const immutable = readonly || !isMutable(schema);
+  const readonly = _readonly || !isMutable(schema);
 
   // TODO(burdon): Should be reactive.
   const viewValues = useMemo(() => {
@@ -74,60 +78,60 @@ export const ViewEditor = ({
 
   const handleSelect = useCallback(
     (field: FieldType) => {
-      if (immutable) {
+      if (readonly) {
         return;
       }
       setField((f) => (f === field ? undefined : field));
     },
-    [immutable],
+    [readonly],
   );
 
   // TODO(burdon): Check if mutable; variant of useCallback that return undefined if readonly?
 
   const handleAdd = useCallback(() => {
-    invariant(!immutable);
+    invariant(!readonly);
     const field = projection.createFieldProjection();
     setField(field);
-  }, [schema, view]);
+  }, [schema, view, readonly]);
 
   const handleUpdate = useCallback(
     ({ name, typename }: ViewMetaType) => {
-      invariant(!immutable);
+      invariant(!readonly);
       requestAnimationFrame(() => {
         if (view.name !== name) {
           view.name = name;
         }
 
-        if (view.query.typename !== typename && !immutable) {
+        if (view.query.typename !== typename && !readonly) {
           onTypenameChanged?.(typename);
         }
       });
     },
-    [schema, view, onTypenameChanged, immutable],
+    [schema, view, onTypenameChanged, readonly],
   );
 
   const handleDelete = useCallback(
     (fieldId: string) => {
-      invariant(!immutable);
+      invariant(!readonly);
       if (fieldId === field?.id) {
         setField(undefined);
       }
 
       onDelete?.(fieldId);
     },
-    [schema, field, onDelete],
+    [schema, field, onDelete, readonly],
   );
 
   const handleMove = useCallback(
     (fromIndex: number, toIndex: number) => {
-      invariant(!immutable);
+      invariant(!readonly);
       // NOTE(ZaymonFC): Using arrayMove here causes a race condition with the kanban model.
       const fields = [...view.fields];
       const [moved] = fields.splice(fromIndex, 1);
       fields.splice(toIndex, 0, moved);
       view.fields = fields;
     },
-    [schema, view.fields],
+    [schema, view.fields, readonly],
   );
 
   const handleClose = useCallback(() => setField(undefined), []);
@@ -151,36 +155,35 @@ export const ViewEditor = ({
   );
 
   return (
-    <div role='none' className={mx('overflow-y-auto', classNames)}>
-      {immutable && (
-        <div role='none' className='mbe-2'>
-          <Message.Root valence='neutral' className='rounded'>
-            <Message.Title>
-              <Icon icon='ph--info--regular' size={5} classNames='inline' /> {t('system schema title')}
-            </Message.Title>
-            <Message.Body>{t('system schema description')}</Message.Body>
-          </Message.Root>
+    <div role='none' className={mx(classNames)}>
+      {readonly && (
+        <div role='none' className={mx('is-full', outerSpacing && 'plb-cardSpacingBlock pli-cardSpacingInline')}>
+          <Callout.Root valence='info'>
+            <Callout.Title>{t('system schema description')}</Callout.Title>
+          </Callout.Root>
         </div>
       )}
+
+      {/* TODO(burdon): Is the form read-only or just the schema? */}
+      {/* TODO(burdon): Readonly fields should take up the same space as editable fields (just be ghosted). */}
       <Form<ViewMetaType>
         autoSave
         schema={ViewMetaSchema}
         values={viewValues}
+        readonly={readonly}
         onSave={handleUpdate}
-        classNames='min-bs-0 overflow-y-auto'
+        outerSpacing={outerSpacing}
       />
 
-      <div role='none' className='min-bs-0 overflow-y-auto'>
-        {/* TODO(burdon): Clean up common form ux. */}
-        <div role='none' className='p-2'>
-          <label className={mx(inputTextLabel)}>{t('fields label')}</label>
-        </div>
+      <div role='none' className={outerSpacing ? cardSpacing : 'mlb-cardSpacingBlock'}>
+        <label className={mx(inputTextLabel)}>{t('fields label')}</label>
 
         <List.Root<FieldType>
           items={view.fields}
           isItem={Schema.is(FieldSchema)}
           getId={(field) => field.id}
-          onMove={immutable ? undefined : handleMove}
+          readonly={readonly}
+          onMove={readonly ? undefined : handleMove}
         >
           {({ items: fields }) => (
             <>
@@ -191,30 +194,29 @@ export const ViewEditor = ({
                 </div>
               )}
 
-              <div role='list' className='flex flex-col w-full'>
+              <div role='list' className='flex flex-col is-full'>
                 {fields?.map((field) => (
                   <List.Item<FieldType>
                     key={field.id}
                     item={field}
-                    classNames={mx(grid, ghostHover, !immutable && 'cursor-pointer')}
+                    classNames={mx(grid, ghostHover, 'overflow-hidden', !readonly && 'cursor-pointer')}
                   >
                     <List.ItemDragHandle />
                     <List.ItemTitle onClick={() => handleSelect(field)}>{field.path}</List.ItemTitle>
-                    <div className='flex items-center gap-2 -ml-4'>
-                      <List.ItemButton
-                        icon='ph--eye-slash--regular'
+                    <List.ItemButton
+                      data-testid='hide-field-button'
+                      icon='ph--eye-slash--regular'
+                      // TDOO(burdon): Is this the correct test?
+                      disabled={view.fields.length <= 1}
+                      onClick={() => handleHide(field.id)}
+                    />
+                    {/* TODO(burdon): Remove unless implement undo. */}
+                    {!readonly && (
+                      <List.ItemDeleteButton
                         disabled={view.fields.length <= 1}
-                        onClick={() => handleHide(field.id)}
-                        data-testid='hide-field-button'
+                        onClick={() => handleDelete(field.id)}
                       />
-                      {!immutable && (
-                        <List.ItemDeleteButton
-                          icon='ph--trash--regular'
-                          disabled={view.fields.length <= 1}
-                          onClick={() => handleDelete(field.id)}
-                        />
-                      )}
-                    </div>
+                    )}
                   </List.Item>
                 ))}
               </div>
@@ -223,10 +225,8 @@ export const ViewEditor = ({
         </List.Root>
 
         {hiddenProperties.length > 0 && (
-          <div>
-            <div role='none' className='p-2'>
-              <label className={mx(inputTextLabel)}>{t('hidden fields label')}</label>
-            </div>
+          <>
+            <label className={mx(inputTextLabel)}>{t('hidden fields label')}</label>
 
             <List.Root<string>
               items={hiddenProperties}
@@ -234,14 +234,14 @@ export const ViewEditor = ({
               getId={(property) => property}
             >
               {({ items: properties }) => (
-                <div role='list' className='flex flex-col w-full'>
+                <div role='list' className='flex flex-col is-full'>
                   {properties?.map((property) => (
                     <List.Item<string>
                       key={property}
                       item={property}
-                      classNames={mx(grid, ghostHover, !immutable && 'cursor-pointer')}
+                      classNames={mx(grid, ghostHover, !readonly && 'cursor-pointer')}
                     >
-                      <div />
+                      <List.ItemDragHandle disabled />
                       <List.ItemTitle>{property}</List.ItemTitle>
                       <List.ItemButton
                         icon='ph--eye--regular'
@@ -253,7 +253,7 @@ export const ViewEditor = ({
                 </div>
               )}
             </List.Root>
-          </div>
+          </>
         )}
       </div>
 
@@ -269,11 +269,11 @@ export const ViewEditor = ({
       )}
 
       {!readonly && !field && (
-        <div role='none' className='p-2'>
+        <div role='none' className={outerSpacing ? cardSpacing : 'mlb-cardSpacingBlock'}>
           <IconButton
             icon='ph--plus--regular'
             label={t('button add property')}
-            onClick={immutable ? undefined : handleAdd}
+            onClick={readonly ? undefined : handleAdd}
             // TODO(burdon): Show field limit in ux (not tooltip).
             disabled={view.fields.length >= VIEW_FIELD_LIMIT}
             classNames='flex is-full'

package/src/components/testing.tsx

@@ -5,16 +5,21 @@
 import React, { type PropsWithChildren } from 'react';
 
 import { type ThemedClassName } from '@dxos/react-ui';
+import { Card } from '@dxos/react-ui-stack';
 import { JsonFilter } from '@dxos/react-ui-syntax-highlighter';
-import { mx } from '@dxos/react-ui-theme';
-
-export const TestPanel = ({ classNames, children }: ThemedClassName<PropsWithChildren>) => (
-  <div className={mx('w-full max-w-[30rem] border border-separator p-2 rounded', classNames)}>{children}</div>
-);
+import { mx, textBlockWidth } from '@dxos/react-ui-theme';
 
 export const TestLayout = ({ classNames, json, children }: ThemedClassName<PropsWithChildren<{ json?: any }>>) => (
-  <div className='w-full h-full grid grid-cols-[1fr_1fr]'>
-    <div className={mx('m-2 justify-center overflow-y-auto', classNames)}>{children}</div>
+  <div className='w-full h-full grid grid-cols-[1fr_1fr] gap-2 overflow-hidden'>
+    <div className={mx('p-4 justify-center overflow-y-auto', classNames)}>{children}</div>
     <JsonFilter data={json} classNames='w-full text-xs' testId='debug' />
   </div>
 );
+
+export const TestPanel = ({ classNames, children }: ThemedClassName<PropsWithChildren>) => (
+  <Card.Content classNames={[textBlockWidth, classNames]}>{children}</Card.Content>
+);
+
+// Symbol for accessing debug objects in tests.
+export const VIEW_EDITOR_DEBUG_SYMBOL = Symbol('viewEditorDebug');
+export const FIELD_EDITOR_DEBUG_SYMBOL = Symbol('fieldEditorDebug');

package/src/translations.ts

@@ -35,6 +35,7 @@ export default [
         'format date-time': 'DateTime',
         'format did': 'DID',
         'format duration': 'Duration',
+        'format dxn': 'DXN',
         'format email': 'Email',
         'format formula': 'Formula',
         'format hostname': 'Hostname',
@@ -47,12 +48,14 @@ export default [
         'format regex': 'RegExp',
         'format string': 'String',
         'format single-select': 'Select',
+        'format multi-select': 'Multi-select',
         'format text': 'Long text',
         'format time': 'Time',
         'format timestamp': 'Timestamp',
         'format user': 'User',
         'format number': 'Number',
         'format uri': 'URI',
+        'format url': 'URL',
         'format uuid': 'UUID',
 
         // Placeholders.
@@ -68,7 +71,7 @@ export default [
 
         // System schema message.
         'system schema title': 'System Schema',
-        'system schema description': 'This schema is managed by the system and cannot be modified.',
+        'system schema description': 'This is a system schema and cannot be modified.',
       },
     },
   },