@dxos/react-ui-thread 0.8.3 → 0.8.4-main.f9ba587

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxos/react-ui-thread",
-  "version": "0.8.3",
+  "version": "0.8.4-main.f9ba587",
   "description": "A set of components to use with threads, including comments and messages.",
   "homepage": "https://dxos.org",
   "bugs": "https://github.com/dxos/dxos/issues",
@@ -26,8 +26,8 @@
     "@preact-signals/safe-react": "^0.9.0",
     "@radix-ui/react-primitive": "2.0.2",
     "date-fns": "^3.3.1",
-    "@dxos/react-ui-editor": "0.8.3",
-    "@dxos/util": "0.8.3"
+    "@dxos/react-ui-editor": "0.8.4-main.f9ba587",
+    "@dxos/util": "0.8.4-main.f9ba587"
   },
   "devDependencies": {
     "@phosphor-icons/react": "^2.1.5",
@@ -36,22 +36,22 @@
     "react": "~18.2.0",
     "react-dom": "~18.2.0",
     "vite": "5.4.7",
-    "@dxos/echo-schema": "0.8.3",
-    "@dxos/live-object": "0.8.3",
-    "@dxos/echo-db": "0.8.3",
-    "@dxos/log": "0.8.3",
-    "@dxos/random": "0.8.3",
-    "@dxos/keys": "0.8.3",
-    "@dxos/react-ui": "0.8.3",
-    "@dxos/react-ui-theme": "0.8.3",
-    "@dxos/storybook-utils": "0.8.3"
+    "@dxos/echo-db": "0.8.4-main.f9ba587",
+    "@dxos/echo-schema": "0.8.4-main.f9ba587",
+    "@dxos/keys": "0.8.4-main.f9ba587",
+    "@dxos/live-object": "0.8.4-main.f9ba587",
+    "@dxos/log": "0.8.4-main.f9ba587",
+    "@dxos/random": "0.8.4-main.f9ba587",
+    "@dxos/react-ui": "0.8.4-main.f9ba587",
+    "@dxos/react-ui-theme": "0.8.4-main.f9ba587",
+    "@dxos/storybook-utils": "0.8.4-main.f9ba587"
   },
   "peerDependencies": {
     "@phosphor-icons/react": "^2.1.5",
     "react": "~18.2.0",
     "react-dom": "~18.2.0",
-    "@dxos/react-ui": "0.8.3",
-    "@dxos/react-ui-theme": "0.8.3"
+    "@dxos/react-ui": "0.8.4-main.f9ba587",
+    "@dxos/react-ui-theme": "0.8.4-main.f9ba587"
   },
   "publishConfig": {
     "access": "public"

package/src/Message/Message.stories.tsx

@@ -11,9 +11,9 @@ import { hoverableControls, hoverableFocusedWithinControls } from '@dxos/react-u
 import { withLayout, withTheme } from '@dxos/storybook-utils';
 
 import { MessageRoot } from './Message';
-import { threadLayout } from '../Thread';
-import { ThreadStoryContainer, MessageStoryText, type MessageEntity } from '../testing';
-import translations from '../translations';
+import { Thread } from '../Thread';
+import { MessageStoryText, type MessageEntity } from '../testing';
+import { translations } from '../translations';
 
 const DefaultStory = () => {
   const [identityKey] = useState(PublicKey.random());
@@ -25,13 +25,13 @@ const DefaultStory = () => {
   });
 
   return (
-    <ThreadStoryContainer>
-      <div className={threadLayout}>
+    <div className='mli-auto is-96 overflow-y-auto'>
+      <Thread.Root id='t1'>
         <MessageRoot {...message} classNames={[hoverableControls, hoverableFocusedWithinControls]}>
           <MessageStoryText {...message} onDelete={() => console.log('delete')} />
         </MessageRoot>
-      </div>
-    </ThreadStoryContainer>
+      </Thread.Root>
+    </div>
   );
 };
 

package/src/Thread/Comments.stories.tsx

@@ -30,10 +30,10 @@ import {
 import { hoverableControls, hoverableFocusedWithinControls } from '@dxos/react-ui-theme';
 import { withTheme } from '@dxos/storybook-utils';
 
-import { Thread, ThreadFooter, ThreadHeading } from './Thread';
+import { Thread } from './Thread';
 import { MessageBody, MessageHeading, MessageRoot, MessageTextbox } from '../Message';
 import { type MessageEntity } from '../testing';
-import translations from '../translations';
+import { translations } from '../translations';
 
 faker.seed(101);
 
@@ -157,7 +157,7 @@ const StoryThread: FC<{
   };
 
   return (
-    <Thread current={selected} onClickCapture={onSelect}>
+    <Thread.Root current={selected} onClickCapture={onSelect}>
       <div className='col-start-2 flex gap-1 text-xs text-description'>
         <span>id:{thread.id.slice(0, 4)}</span>
         <span>y:{thread.yPos}</span>
@@ -165,9 +165,9 @@ const StoryThread: FC<{
         {!thread.cursor && <Trash />}
       </div>
 
-      <ThreadHeading>
+      <Thread.Header>
         {thread.range?.from} &ndash; {thread.range?.to}
-      </ThreadHeading>
+      </Thread.Header>
 
       {thread.messages.map((message) => (
         <MessageRoot key={message.id} classNames={[hoverableControls, hoverableFocusedWithinControls]} {...message}>
@@ -185,12 +185,12 @@ const StoryThread: FC<{
           onEditorFocus={onSelect}
           onSend={handleCreateMessage}
         />
-        <ThreadFooter />
+        <Thread.Status />
         <Button variant='ghost' classNames='px-1' title='Resolve' onClick={onResolve}>
           <Check />
         </Button>
       </div>
-    </Thread>
+    </Thread.Root>
   );
 };
 
@@ -359,15 +359,13 @@ export default {
 
 const str = (...lines: string[]) => lines.join('\n');
 
-const document = str(
-  '# Comments',
-  '',
-  str(...faker.helpers.multiple(() => [faker.lorem.paragraph({ min: 5, max: 10 }), ''], { count: 20 }).flat()),
-  '',
-);
-
 export const Default = {
   args: {
-    text: document,
+    text: str(
+      '# Comments',
+      '',
+      str(...faker.helpers.multiple(() => [faker.lorem.paragraph({ min: 5, max: 10 }), ''], { count: 20 }).flat()),
+      '',
+    ),
   },
 };

package/src/Thread/Thread.stories.tsx

@@ -10,12 +10,12 @@ import { PublicKey } from '@dxos/keys';
 import { faker } from '@dxos/random';
 import { createBasicExtensions, createThemeExtensions } from '@dxos/react-ui-editor';
 import { hoverableControls, hoverableFocusedWithinControls } from '@dxos/react-ui-theme';
-import { withLayout, withTheme } from '@dxos/storybook-utils';
+import { type Meta, withLayout, withTheme } from '@dxos/storybook-utils';
 
-import { Thread, ThreadFooter } from './Thread';
+import { Thread } from './Thread';
 import { MessageRoot, MessageTextbox } from '../Message';
-import { ThreadStoryContainer, MessageStoryText, type MessageEntity } from '../testing';
-import translations from '../translations';
+import { MessageStoryText, type MessageEntity } from '../testing';
+import { translations } from '../translations';
 
 faker.seed(1);
 
@@ -59,13 +59,14 @@ const DefaultStory = () => {
   };
 
   return (
-    <ThreadStoryContainer>
-      <Thread id='t1'>
+    <div className='mli-auto is-96 overflow-y-auto'>
+      <Thread.Root id='t1'>
         {messages.map((message) => (
           <MessageRoot key={message.id} classNames={[hoverableControls, hoverableFocusedWithinControls]} {...message}>
             <MessageStoryText {...message} text={message.text} onDelete={() => console.log('delete')} />
           </MessageRoot>
         ))}
+
         <MessageTextbox
           id={String(_count)}
           authorId={identityKey1.toHex()}
@@ -73,18 +74,21 @@ const DefaultStory = () => {
           extensions={extensions}
           onSend={handleSend}
         />
-        <ThreadFooter activity>Processing...</ThreadFooter>
-      </Thread>
-    </ThreadStoryContainer>
+
+        <Thread.Status activity>Processing...</Thread.Status>
+      </Thread.Root>
+    </div>
   );
 };
 
-export default {
+const meta: Meta<typeof Thread.Root> = {
   title: 'ui/react-ui-thread/Thread',
-  component: Thread,
+  component: Thread.Root,
   render: DefaultStory,
   decorators: [withTheme, withLayout({ fullscreen: true })],
   parameters: { translations },
 };
 
+export default meta;
+
 export const Default = {};

package/src/Thread/Thread.tsx

@@ -11,11 +11,16 @@ import { hoverableControlItem, hoverableFocusedWithinControls, mx } from '@dxos/
 import { translationKey } from '../translations';
 import type { ThreadEntity } from '../types';
 
-export type ThreadProps = ThemedClassName<ComponentPropsWithRef<'div'>> & ThreadEntity & { current?: boolean };
-
+// TODO(burdon): Why is this exported?
 export const threadLayout = 'is-full place-self-start grid grid-cols-[var(--rail-size)_1fr]';
 
-export const Thread = forwardRef<HTMLDivElement, ThreadProps>(
+//
+// Root
+//
+
+type ThreadRootProps = ThemedClassName<ComponentPropsWithRef<'div'>> & ThreadEntity & { current?: boolean };
+
+const ThreadRoot = forwardRef<HTMLDivElement, ThreadRootProps>(
   ({ current, children, classNames, ...props }, forwardedRef) => {
     return (
       <div
@@ -26,8 +31,7 @@ export const Thread = forwardRef<HTMLDivElement, ThreadProps>(
         className={mx(
           threadLayout,
           hoverableFocusedWithinControls,
-          'pbs-2 pbe-4 bg-[var(--surface-bg)] border-separator first:border-bs-0 border-be',
-          'current-related attention-surface [--controls-opacity:0]',
+          'bg-[var(--surface-bg)] current-related attention-surface [--controls-opacity:0]',
           classNames,
         )}
         ref={forwardedRef}
@@ -38,9 +42,13 @@ export const Thread = forwardRef<HTMLDivElement, ThreadProps>(
   },
 );
 
-export type ThreadHeadingProps = ThemedClassName<ComponentPropsWithRef<'div'>> & { detached?: boolean };
+//
+// Heading
+//
+
+type ThreadHeaderProps = ThemedClassName<ComponentPropsWithRef<'div'>> & { detached?: boolean };
 
-export const ThreadHeading = forwardRef<HTMLParagraphElement, ThreadHeadingProps>(
+const ThreadHeader = forwardRef<HTMLParagraphElement, ThreadHeaderProps>(
   ({ classNames, children, detached, ...props }, forwardedRef) => {
     return (
       <>
@@ -66,11 +74,15 @@ export const ThreadHeading = forwardRef<HTMLParagraphElement, ThreadHeadingProps
   },
 );
 
-export type ThreadFooterProps = ThemedClassName<ComponentPropsWithRef<'div'>> & {
+//
+// Status
+//
+
+type ThreadStatusProps = ThemedClassName<ComponentPropsWithRef<'div'>> & {
   activity?: boolean;
 };
 
-export const ThreadFooter = forwardRef<HTMLDivElement, ThreadFooterProps>(
+const ThreadStatus = forwardRef<HTMLDivElement, ThreadStatusProps>(
   ({ activity, classNames, children, ...props }, forwardedRef) => {
     const { t } = useTranslation(translationKey);
     return (
@@ -95,3 +107,11 @@ export const ThreadFooter = forwardRef<HTMLDivElement, ThreadFooterProps>(
     );
   },
 );
+
+export const Thread = {
+  Root: ThreadRoot,
+  Header: ThreadHeader,
+  Status: ThreadStatus,
+};
+
+export type { ThreadRootProps, ThreadHeaderProps as ThreadHeadingProps, ThreadStatusProps };

package/src/index.ts

@@ -2,7 +2,7 @@
 // Copyright 2023 DXOS.org
 //
 
-import translations from './translations';
+import { translations } from './translations';
 
 export * from './Thread';
 export * from './Message';

package/src/testing.tsx

@@ -3,7 +3,7 @@
 //
 
 import { X } from '@phosphor-icons/react';
-import React, { type PropsWithChildren } from 'react';
+import React from 'react';
 
 import { Button } from '@dxos/react-ui';
 import { hoverableControlItem, hoverableControls, hoverableFocusedWithinControls, mx } from '@dxos/react-ui-theme';
@@ -56,7 +56,3 @@ export const MessageStoryBlock = <PartValue,>({ part }: MessagePartProps<PartVal
     </div>
   );
 };
-
-export const ThreadStoryContainer = ({ children }: PropsWithChildren) => {
-  return <div className={mx('bg-white dark:bg-black mli-auto is-96 overflow-y-auto')}>{children}</div>;
-};

package/src/translations.ts

@@ -2,9 +2,11 @@
 // Copyright 2023 DXOS.org
 //
 
-export const translationKey = 'thread';
+import { type Resource } from '@dxos/react-ui';
 
-export default [
+export const translationKey = 'react-ui-thread';
+
+export const translations = [
   {
     'en-US': {
       [translationKey]: {
@@ -21,4 +23,4 @@ export default [
       },
     },
   },
-];
+] as const satisfies Resource[];