@srcroot/ui 0.0.27 → 0.0.30

package/README.md

@@ -1,6 +1,6 @@
 # @srcroot/ui
 
-A **shadcn-like** React UI library for building accessible, polymorphic web applications.
+A UI library with polymorphic, accessible React components.
 This library provides a collection of re-usable components that you can copy and paste into your apps.
 
 ## Features

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@srcroot/ui",
-  "version": "0.0.27",
-  "description": "A shadcn-style CLI UI library with polymorphic, accessible React components",
+  "version": "0.0.30",
+  "description": "A UI library with polymorphic, accessible React components",
   "type": "module",
   "bin": {
     "srcroot-ui": "./dist/index.js"

package/registry/carousel.tsx

@@ -1,11 +1,14 @@
+'use client'
 import * as React from "react"
 import { cn } from "@/lib/utils"
 
 interface CarouselContextValue {
     currentIndex: number
-    setCurrentIndex: (index: number) => void
+    setCurrentIndex: React.Dispatch<React.SetStateAction<number>>
     itemsCount: number
-    setItemsCount: (count: number) => void
+    setItemsCount: React.Dispatch<React.SetStateAction<number>>
+    isTransitioning: boolean
+    setIsTransitioning: React.Dispatch<React.SetStateAction<boolean>>
 }
 
 const CarouselContext = React.createContext<CarouselContextValue | null>(null)
@@ -13,44 +16,42 @@ const CarouselContext = React.createContext<CarouselContextValue | null>(null)
 interface CarouselProps extends React.HTMLAttributes<HTMLDivElement> {
     /** Auto-play interval in ms (0 to disable) */
     autoPlay?: number
-    /** Loop back to start */
+    /** Loop back to start (ignored in this infinite implementation as it's always true-ish, but kept for API) */
     loop?: boolean
 }
 
 /**
- * Carousel/Slider component
- * 
- * @example
- * <Carousel>
- *   <CarouselContent>
- *     <CarouselItem>Slide 1</CarouselItem>
- *     <CarouselItem>Slide 2</CarouselItem>
- *   </CarouselContent>
- *   <CarouselPrevious />
- *   <CarouselNext />
- * </Carousel>
+ * Carousel/Slider component with Infinite Looping
  */
 const Carousel = React.forwardRef<HTMLDivElement, CarouselProps>(
     ({ className, children, autoPlay = 0, loop = true, ...props }, ref) => {
-        const [currentIndex, setCurrentIndex] = React.useState(0)
+        // Start at 1 because 0 is the clone of the last item
+        const [currentIndex, setCurrentIndex] = React.useState(1)
         const [itemsCount, setItemsCount] = React.useState(0)
+        const [isTransitioning, setIsTransitioning] = React.useState(true)
+        const intervalRef = React.useRef<NodeJS.Timeout | null>(null)
 
         React.useEffect(() => {
-            if (autoPlay > 0 && itemsCount > 0) {
-                const interval = setInterval(() => {
-                    setCurrentIndex((prev) => {
-                        if (prev >= itemsCount - 1) {
-                            return loop ? 0 : prev
-                        }
-                        return prev + 1
-                    })
+            if (autoPlay > 0 && itemsCount > 1) {
+                intervalRef.current = setInterval(() => {
+                    setIsTransitioning(true)
+                    setCurrentIndex((prev) => prev + 1)
                 }, autoPlay)
-                return () => clearInterval(interval)
+                return () => {
+                    if (intervalRef.current) clearInterval(intervalRef.current)
+                }
             }
-        }, [autoPlay, itemsCount, loop])
+        }, [autoPlay, itemsCount])
 
         return (
-            <CarouselContext.Provider value={{ currentIndex, setCurrentIndex, itemsCount, setItemsCount }}>
+            <CarouselContext.Provider value={{
+                currentIndex,
+                setCurrentIndex,
+                itemsCount,
+                setItemsCount,
+                isTransitioning,
+                setIsTransitioning
+            }}>
                 <div
                     ref={ref}
                     className={cn("relative", className)}
@@ -71,19 +72,59 @@ const CarouselContent = React.forwardRef<HTMLDivElement, React.HTMLAttributes<HT
         const context = React.useContext(CarouselContext)
         if (!context) throw new Error("CarouselContent must be used within Carousel")
 
-        const childrenArray = React.Children.toArray(children)
+        const items = React.Children.toArray(children)
 
         React.useEffect(() => {
-            context.setItemsCount(childrenArray.length)
-        }, [childrenArray.length, context])
+            context.setItemsCount(items.length)
+        }, [items.length, context])
+
+        // Clone first and last items for infinite loop illusion
+        const firstClone = items.length > 0 && React.isValidElement(items[0])
+            ? React.cloneElement(items[0] as React.ReactElement, { key: "clone-first" })
+            : null
+        const lastClone = items.length > 0 && React.isValidElement(items[items.length - 1])
+            ? React.cloneElement(items[items.length - 1] as React.ReactElement, { key: "clone-last" })
+            : null
+
+        // If we have items, prepend last-clone and append first-clone
+        const displayItems = items.length > 1 ? [lastClone, ...items, firstClone] : items
+
+        const handleTransitionEnd = () => {
+            if (items.length <= 1) return
+
+            // If reached the end clone (index = N + 1), snap back to first real item (index = 1)
+            if (context.currentIndex >= items.length + 1) {
+                context.setIsTransitioning(false)
+                context.setCurrentIndex(1)
+            }
+            // If reached the start clone (index = 0), snap forward to last real item (index = N)
+            else if (context.currentIndex <= 0) {
+                context.setIsTransitioning(false)
+                context.setCurrentIndex(items.length)
+            }
+        }
+
+        // Re-enable transition after a snap (on next frame)
+        React.useEffect(() => {
+            if (!context.isTransitioning) {
+                const timer = setTimeout(() => {
+                    context.setIsTransitioning(true)
+                }, 50)
+                return () => clearTimeout(timer)
+            }
+        }, [context.isTransitioning, context])
 
         return (
             <div ref={ref} className={cn("overflow-hidden", className)} {...props}>
                 <div
-                    className="flex transition-transform duration-300 ease-in-out"
-                    style={{ transform: `translateX(-${context.currentIndex * 100}%)` }}
+                    className="flex h-full w-full"
+                    style={{
+                        transform: `translateX(-${context.currentIndex * 100}%)`,
+                        transition: context.isTransitioning ? 'transform 300ms ease-in-out' : 'none'
+                    }}
+                    onTransitionEnd={handleTransitionEnd}
                 >
-                    {children}
+                    {displayItems}
                 </div>
             </div>
         )
@@ -97,7 +138,7 @@ const CarouselItem = React.forwardRef<HTMLDivElement, React.HTMLAttributes<HTMLD
             ref={ref}
             role="group"
             aria-roledescription="slide"
-            className={cn("min-w-0 shrink-0 grow-0 basis-full", className)}
+            className={cn("min-w-0 h-full shrink-0 grow-0 basis-full", className)}
             {...props}
         />
     )
@@ -111,19 +152,21 @@ const CarouselPrevious = React.forwardRef<
     const context = React.useContext(CarouselContext)
     if (!context) throw new Error("CarouselPrevious must be used within Carousel")
 
-    const canGoPrev = context.currentIndex > 0
+    const handlePrev = () => {
+        context.setIsTransitioning(true)
+        context.setCurrentIndex((prev) => prev - 1)
+    }
 
     return (
         <button
             ref={ref}
             type="button"
             className={cn(
-                "absolute left-4 top-1/2 -translate-y-1/2 h-8 w-8 rounded-full border bg-background shadow-md flex items-center justify-center",
+                "absolute left-4 top-1/2 -translate-y-1/2 h-8 w-8 rounded-full border bg-background shadow-md flex items-center justify-center cursor-pointer",
                 "hover:bg-accent disabled:opacity-50",
                 className
             )}
-            disabled={!canGoPrev}
-            onClick={() => context.setCurrentIndex(context.currentIndex - 1)}
+            onClick={handlePrev}
             aria-label="Previous slide"
             {...props}
         >
@@ -142,19 +185,21 @@ const CarouselNext = React.forwardRef<
     const context = React.useContext(CarouselContext)
     if (!context) throw new Error("CarouselNext must be used within Carousel")
 
-    const canGoNext = context.currentIndex < context.itemsCount - 1
+    const handleNext = () => {
+        context.setIsTransitioning(true)
+        context.setCurrentIndex((prev) => prev + 1)
+    }
 
     return (
         <button
             ref={ref}
             type="button"
             className={cn(
-                "absolute right-4 top-1/2 -translate-y-1/2 h-8 w-8 rounded-full border bg-background shadow-md flex items-center justify-center",
+                "absolute right-4 top-1/2 -translate-y-1/2 h-8 w-8 rounded-full border bg-background shadow-md flex items-center justify-center cursor-pointer",
                 "hover:bg-accent disabled:opacity-50",
                 className
             )}
-            disabled={!canGoNext}
-            onClick={() => context.setCurrentIndex(context.currentIndex + 1)}
+            onClick={handleNext}
             aria-label="Next slide"
             {...props}
         >

package/registry/sidebar.tsx

@@ -1,7 +1,7 @@
 "use client"
 
 import * as React from "react"
-import { LuPanelLeft } from "react-icons/lu"
+import { PanelLeft } from "lucide-react"
 import { cva, type VariantProps } from "class-variance-authority"
 
 import { cn } from "@/lib/utils"
@@ -168,7 +168,10 @@ const Sidebar = React.forwardRef<
         return (
             <div
                 ref={ref}
-                className="group peer hidden md:block text-sidebar-foreground"
+                className={cn(
+                    "group peer hidden md:block text-sidebar-foreground",
+                    className
+                )}
                 data-state={state}
                 data-collapsible={state === "collapsed" ? collapsible : ""}
                 data-variant={variant}
@@ -233,7 +236,7 @@ const SidebarTrigger = React.forwardRef<
             }}
             {...props}
         >
-            <LuPanelLeft />
+            <PanelLeft />
             <span className="sr-only">Toggle Sidebar</span>
         </Button>
     )