@object-ui/plugin-list 0.5.1 → 3.0.0

package/src/ObjectGallery.tsx

@@ -1,27 +1,69 @@
+/**
+ * ObjectUI
+ * Copyright (c) 2024-present ObjectStack Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
 
 import React, { useState, useEffect } from 'react';
 import { useDataScope, useSchemaContext } from '@object-ui/react';
 import { ComponentRegistry } from '@object-ui/core';
+import { cn, Card, CardContent } from '@object-ui/components';
+import type { GalleryConfig } from '@object-ui/types';
 
-// Utility for class merging (assuming it's available in plugin context, 
-// usually provided by @object-ui/components or utils, but here I'll just use string concat if not imported)
-// Actually @object-ui/components exports cn
-import { cn } from '@object-ui/components';
+export interface ObjectGalleryProps {
+    schema: {
+        objectName?: string;
+        bind?: string;
+        filter?: unknown;
+        data?: Record<string, unknown>[];
+        className?: string;
+        gallery?: GalleryConfig;
+        /** @deprecated Use gallery.coverField instead */
+        imageField?: string;
+        /** @deprecated Use gallery.titleField instead */
+        titleField?: string;
+        subtitleField?: string;
+    };
+    data?: Record<string, unknown>[];
+    dataSource?: { find: (name: string, query: unknown) => Promise<unknown> };
+    onCardClick?: (record: Record<string, unknown>) => void;
+}
 
-export const ObjectGallery = (props: any) => {
+const GRID_CLASSES: Record<NonNullable<GalleryConfig['cardSize']>, string> = {
+    small: 'grid-cols-2 sm:grid-cols-3 md:grid-cols-4 lg:grid-cols-5 xl:grid-cols-6',
+    medium: 'grid-cols-1 sm:grid-cols-2 md:grid-cols-3 lg:grid-cols-4',
+    large: 'grid-cols-1 sm:grid-cols-1 md:grid-cols-2 lg:grid-cols-3',
+};
+
+const ASPECT_CLASSES: Record<NonNullable<GalleryConfig['cardSize']>, string> = {
+    small: 'aspect-square',
+    medium: 'aspect-[4/3]',
+    large: 'aspect-[16/10]',
+};
+
+export const ObjectGallery: React.FC<ObjectGalleryProps> = (props) => {
     const { schema } = props;
     const context = useSchemaContext();
     const dataSource = props.dataSource || context.dataSource;
     const boundData = useDataScope(schema.bind);
 
-    const [fetchedData, setFetchedData] = useState<any[]>([]);
+    const [fetchedData, setFetchedData] = useState<Record<string, unknown>[]>([]);
     const [loading, setLoading] = useState(false);
 
+    // Resolve GalleryConfig with backwards-compatible fallbacks
+    const gallery = schema.gallery;
+    const coverField = gallery?.coverField ?? schema.imageField ?? 'image';
+    const coverFit = gallery?.coverFit ?? 'cover';
+    const cardSize = gallery?.cardSize ?? 'medium';
+    const titleField = gallery?.titleField ?? schema.titleField ?? 'name';
+    const visibleFields = gallery?.visibleFields;
+
     useEffect(() => {
         let isMounted = true;
-        
-        // Use data prop if available (from ListView)
-        if ((props.data && Array.isArray(props.data))) {
+
+        if (props.data && Array.isArray(props.data)) {
             setFetchedData(props.data);
             return;
         }
@@ -30,22 +72,22 @@ export const ObjectGallery = (props: any) => {
             if (!dataSource || !schema.objectName) return;
             if (isMounted) setLoading(true);
             try {
-                // Apply filtering?
                 const results = await dataSource.find(schema.objectName, {
-                   $filter: schema.filter
+                    $filter: schema.filter,
                 });
-                
-                let data: any[] = [];
+
+                let data: Record<string, unknown>[] = [];
                 if (Array.isArray(results)) {
                     data = results;
                 } else if (results && typeof results === 'object') {
-                    if (Array.isArray((results as any).value)) {
-                        data = (results as any).value;
-                    } else if (Array.isArray((results as any).data)) {
-                        data = (results as any).data;
+                    const r = results as Record<string, unknown>;
+                    if (Array.isArray(r.records)) {
+                        data = r.records as Record<string, unknown>[];
+                    } else if (Array.isArray(r.data)) {
+                        data = r.data as Record<string, unknown>[];
                     }
                 }
-    
+
                 if (isMounted) {
                     setFetchedData(data);
                 }
@@ -55,51 +97,78 @@ export const ObjectGallery = (props: any) => {
                 if (isMounted) setLoading(false);
             }
         };
-    
+
         if (schema.objectName && !boundData && !schema.data && !props.data) {
             fetchData();
         }
         return () => { isMounted = false; };
-      }, [schema.objectName, dataSource, boundData, schema.data, schema.filter, props.data]);
+    }, [schema.objectName, dataSource, boundData, schema.data, schema.filter, props.data]);
 
-    const items = props.data || boundData || schema.data || fetchedData || [];
-    
-    // Config
-    const imageField = schema.imageField || 'image';
-    const titleField = schema.titleField || 'name';
-    const subtitleField = schema.subtitleField;
+    const items: Record<string, unknown>[] = props.data || boundData || schema.data || fetchedData || [];
 
     if (loading && !items.length) return <div className="p-4 text-sm text-muted-foreground">Loading Gallery...</div>;
     if (!items.length) return <div className="p-4 text-sm text-muted-foreground">No items to display</div>;
 
     return (
-        <div className={cn("grid grid-cols-2 md:grid-cols-3 lg:grid-cols-4 xl:grid-cols-5 gap-4 p-4", schema.className)}>
-            {items.map((item: any, i: number) => (
-                <div key={item._id || i} className="group relative border rounded-lg overflow-hidden bg-card text-card-foreground shadow-sm hover:shadow-md transition-all">
-                    <div className="aspect-square w-full overflow-hidden bg-muted relative">
-                        {item[imageField] ? (
-                            <img 
-                                src={item[imageField]} 
-                                alt={item[titleField]} 
-                                className="h-full w-full object-cover transition-transform group-hover:scale-105"
-                                onError={(e) => {
-                                    (e.target as HTMLImageElement).src = `https://placehold.co/400x400?text=${encodeURIComponent(item[titleField]?.[0] || '?')}`;
-                                }}
-                            />
-                        ) : (
-                            <div className="flex h-full w-full items-center justify-center bg-secondary/50 text-muted-foreground">
-                                <span className="text-4xl font-light opacity-20">{item[titleField]?.[0]?.toUpperCase()}</span>
-                            </div>
-                        )}
-                    </div>
-                    <div className="p-3 border-t">
-                        <h3 className="font-medium truncate text-sm" title={item[titleField]}>{item[titleField] || 'Untitled'}</h3>
-                        {subtitleField && (
-                             <p className="text-xs text-muted-foreground truncate mt-1">{item[subtitleField]}</p>
+        <div
+            className={cn('grid gap-4 p-4', GRID_CLASSES[cardSize], schema.className)}
+            role="list"
+        >
+            {items.map((item, i) => {
+                const id = (item._id ?? item.id ?? i) as string | number;
+                const title = String(item[titleField] ?? 'Untitled');
+                const imageUrl = item[coverField] as string | undefined;
+
+                return (
+                    <Card
+                        key={id}
+                        role="listitem"
+                        className={cn(
+                            'group overflow-hidden transition-all hover:shadow-md',
+                            props.onCardClick && 'cursor-pointer',
                         )}
-                    </div>
-                </div>
-            ))}
+                        onClick={props.onCardClick ? () => props.onCardClick!(item) : undefined}
+                    >
+                        <div className={cn('w-full overflow-hidden bg-muted relative', ASPECT_CLASSES[cardSize])}>
+                            {imageUrl ? (
+                                <img
+                                    src={imageUrl}
+                                    alt={title}
+                                    className={cn(
+                                        'h-full w-full transition-transform group-hover:scale-105',
+                                        coverFit === 'cover' && 'object-cover',
+                                        coverFit === 'contain' && 'object-contain',
+                                    )}
+                                />
+                            ) : (
+                                <div className="flex h-full w-full items-center justify-center bg-secondary/50 text-muted-foreground">
+                                    <span className="text-4xl font-light opacity-20">
+                                        {title[0]?.toUpperCase()}
+                                    </span>
+                                </div>
+                            )}
+                        </div>
+                        <CardContent className="p-3 border-t">
+                            <h3 className="font-medium truncate text-sm" title={title}>
+                                {title}
+                            </h3>
+                            {visibleFields && visibleFields.length > 0 && (
+                                <div className="mt-1 space-y-0.5">
+                                    {visibleFields.map((field) => {
+                                        const value = item[field];
+                                        if (value == null) return null;
+                                        return (
+                                            <p key={field} className="text-xs text-muted-foreground truncate">
+                                                {String(value)}
+                                            </p>
+                                        );
+                                    })}
+                                </div>
+                            )}
+                        </CardContent>
+                    </Card>
+                );
+            })}
         </div>
     );
 };
@@ -107,5 +176,5 @@ export const ObjectGallery = (props: any) => {
 ComponentRegistry.register('object-gallery', ObjectGallery, {
     namespace: 'plugin-list',
     label: 'Gallery View',
-    category: 'view'
+    category: 'view',
 });

package/src/ViewSwitcher.tsx

@@ -32,6 +32,8 @@ export interface ViewSwitcherProps {
   availableViews?: ViewType[];
   onViewChange: (view: ViewType) => void;
   className?: string;
+  /** Enable animated transitions between views (default: true) */
+  animated?: boolean;
 }
 
 const VIEW_ICONS: Record<ViewType, React.ReactNode> = {
@@ -59,16 +61,35 @@ export const ViewSwitcher: React.FC<ViewSwitcherProps> = ({
   availableViews = ['grid', 'kanban'],
   onViewChange,
   className,
+  animated = true,
 }) => {
+  const handleViewChange = React.useCallback(
+    (view: ViewType) => {
+      if (!animated || view === currentView) {
+        onViewChange(view);
+        return;
+      }
+
+      if (typeof document !== 'undefined' && 'startViewTransition' in document) {
+        (document as Document & {
+          startViewTransition: (cb: () => void) => { finished: Promise<void> };
+        }).startViewTransition(() => onViewChange(view));
+      } else {
+        onViewChange(view);
+      }
+    },
+    [animated, currentView, onViewChange],
+  );
+
   return (
-    <div className={cn("flex items-center gap-1 bg-transparent", className)}>
+    <div className={cn("flex items-center gap-1 bg-transparent oui-view-switcher", className)}>
       {availableViews.map((view) => {
         const isActive = currentView === view;
         return (
           <button
             key={view}
             type="button"
-            onClick={() => onViewChange(view)}
+            onClick={() => handleViewChange(view)}
             aria-label={VIEW_LABELS[view]}
             title={VIEW_LABELS[view]}
             aria-pressed={isActive}

package/src/__tests__/ListView.test.tsx

@@ -66,7 +66,7 @@ describe('ListView', () => {
     expect(container).toBeTruthy();
   });
 
-  it('should render search input', () => {
+  it('should render search button', () => {
     const schema: ListViewSchema = {
       type: 'list-view',
       objectName: 'contacts',
@@ -75,11 +75,11 @@ describe('ListView', () => {
     };
 
     renderWithProvider(<ListView schema={schema} />);
-    const searchInput = screen.getByPlaceholderText(/find/i);
-    expect(searchInput).toBeInTheDocument();
+    const searchButton = screen.getByRole('button', { name: /search/i });
+    expect(searchButton).toBeInTheDocument();
   });
 
-  it('should call onSearchChange when search input changes', () => {
+  it('should expand search and call onSearchChange when search input changes', () => {
     const onSearchChange = vi.fn();
     const schema: ListViewSchema = {
       type: 'list-view',
@@ -89,8 +89,12 @@ describe('ListView', () => {
     };
 
     renderWithProvider(<ListView schema={schema} onSearchChange={onSearchChange} />);
-    const searchInput = screen.getByPlaceholderText(/find/i);
     
+    // Click search button to expand
+    const searchButton = screen.getByRole('button', { name: /search/i });
+    fireEvent.click(searchButton);
+    
+    const searchInput = screen.getByPlaceholderText(/find/i);
     fireEvent.change(searchInput, { target: { value: 'test' } });
     expect(onSearchChange).toHaveBeenCalledWith('test');
   });
@@ -108,7 +112,7 @@ describe('ListView', () => {
       },
     };
 
-    renderWithProvider(<ListView schema={schema} />);
+    renderWithProvider(<ListView schema={schema} showViewSwitcher={true} />);
     
     // Find kanban view button and click it
     // ViewSwitcher uses buttons with aria-label
@@ -196,13 +200,18 @@ describe('ListView', () => {
     };
 
     renderWithProvider(<ListView schema={schema} />);
+    
+    // Click search button to expand search input
+    const searchButton = screen.getByRole('button', { name: /search/i });
+    fireEvent.click(searchButton);
+    
     const searchInput = screen.getByPlaceholderText(/find/i) as HTMLInputElement;
     
     // Type in search
     fireEvent.change(searchInput, { target: { value: 'test' } });
     expect(searchInput.value).toBe('test');
     
-    // Find and click clear button
+    // Find and click clear button (the X button inside the expanded search)
     const buttons = screen.getAllByRole('button');
     const clearButton = buttons.find(btn => 
       btn.querySelector('svg') !== null && searchInput.value !== ''

package/src/__tests__/ListViewPersistence.test.tsx

@@ -56,7 +56,7 @@ describe('ListView Persistence', () => {
       },
     };
 
-    renderWithProvider(<ListView schema={schema} />);
+    renderWithProvider(<ListView schema={schema} showViewSwitcher={true} />);
     
     // Simulate changing to kanban view
     const kanbanButton = screen.getByLabelText('Kanban'); 
@@ -89,7 +89,7 @@ describe('ListView Persistence', () => {
         },
      };
 
-     renderWithProvider(<ListView schema={viewB_Schema} />);
+     renderWithProvider(<ListView schema={viewB_Schema} showViewSwitcher={true} />);
      
      // Should use the schema default 'kanban' (since no storage exists for THIS view id)
      // It should NOT use 'grid' from the global/default view.
@@ -117,7 +117,7 @@ describe('ListView Persistence', () => {
           },
       };
       
-      renderWithProvider(<ListView schema={schema} />);
+      renderWithProvider(<ListView schema={schema} showViewSwitcher={true} />);
       
       // Should respect schema ('grid') because storage persistence is currently disabled
       const kanbanButton = screen.getByLabelText('Kanban');

package/src/index.tsx

@@ -10,10 +10,10 @@ import { ComponentRegistry } from '@object-ui/core';
 import { ListView } from './ListView';
 import { ViewSwitcher } from './ViewSwitcher';
 import { ObjectGallery } from './ObjectGallery';
-import type { ListViewSchema } from '@object-ui/types';
 
 export { ListView, ViewSwitcher, ObjectGallery };
 export type { ListViewProps } from './ListView';
+export type { ObjectGalleryProps } from './ObjectGallery';
 export type { ViewSwitcherProps, ViewType } from './ViewSwitcher';
 
 // Register ListView component
@@ -45,3 +45,25 @@ ComponentRegistry.register('list-view', ListView, {
     options: {},
   }
 });
+
+// Alias for generic view
+ComponentRegistry.register('list', ListView, {
+  namespace: 'view',
+  category: 'view',
+  label: 'List',
+  icon: 'LayoutList',
+  inputs: [
+    { name: 'objectName', type: 'string', label: 'Object Name', required: true },
+    { name: 'viewType', type: 'enum', label: 'Default View', enum: [
+      { label: 'Grid', value: 'grid' },
+      { label: 'List', value: 'list' },
+      { label: 'Kanban', value: 'kanban' },
+      { label: 'Calendar', value: 'calendar' },
+      { label: 'Chart', value: 'chart' }
+    ], defaultValue: 'grid' },
+    { name: 'fields', type: 'array', label: 'Fields' },
+    { name: 'filters', type: 'array', label: 'Filters' },
+    { name: 'sort', type: 'array', label: 'Sort' },
+    { name: 'options', type: 'object', label: 'View Options' },
+  ]
+});

package/src/registration.test.tsx

@@ -1,5 +1,4 @@
-import { describe, it, expect, beforeAll } from 'vitest';
-import { ComponentRegistry } from '@object-ui/core';
+import { describe, it, expect } from 'vitest';
 import { ListView } from './index';
 
 describe('Plugin List Registration', () => {

package/vitest.config.ts

@@ -1,7 +1,6 @@
 /// <reference types="vitest" />
 import { defineConfig } from 'vite';
 import react from '@vitejs/plugin-react';
-import path from 'path';
 
 export default defineConfig({
   plugins: [react()],