@eintrek/erp-shell 0.1.39 → 0.1.41

package/dist/index.js

@@ -459,10 +459,22 @@ function SidebarProvider({ children, defaultOpen = true, }) {
             localStorage.setItem("sidebar-open", String(open));
         }
     }, [open]);
-    // Handle mobile detection
+    // Handle mobile detection.
+    //
+    // Crossing into a narrow viewport closes the sidebar. One `open` state
+    // serves both layouts, and on desktop it is remembered as true — so
+    // narrowing the window turned that into a drawer covering the whole page,
+    // which had to be dismissed before anything underneath could be reached.
+    // A remembered desktop preference is not a request to open a drawer.
     React.useEffect(() => {
+        let wasMobile = null;
         const checkMobile = () => {
-            setIsMobile(window.innerWidth < 768);
+            const nowMobile = window.innerWidth < 768;
+            setIsMobile(nowMobile);
+            if (nowMobile && wasMobile === false) {
+                setOpenState(false);
+            }
+            wasMobile = nowMobile;
         };
         checkMobile();
         window.addEventListener("resize", checkMobile);
@@ -783,13 +795,27 @@ function AppBreadcrumb({ labels = {}, homeLabel = "แดชบอร์ด", ro
     if (segments.length === 0) {
         return jsxRuntime.jsx("h1", { className: "text-xl font-semibold", children: homeLabel });
     }
-    return (jsxRuntime.jsx(erpTheme.Breadcrumb, { children: jsxRuntime.jsxs(erpTheme.BreadcrumbList, { children: [jsxRuntime.jsx(erpTheme.BreadcrumbItem, { className: "hidden md:block", children: jsxRuntime.jsx(erpTheme.BreadcrumbLink, { asChild: true, children: jsxRuntime.jsx(Link, { href: "/", className: "font-thai text-[14px] font-normal text-white/80 hover:text-white transition-colors", children: rootLabel }) }) }), segments.map((segment, index) => {
+    return (jsxRuntime.jsx(erpTheme.Breadcrumb, { children: jsxRuntime.jsxs(erpTheme.BreadcrumbList, { children: [jsxRuntime.jsx(erpTheme.BreadcrumbItem, { children: jsxRuntime.jsx(erpTheme.BreadcrumbLink, { asChild: true, children: jsxRuntime.jsx(Link, { href: "/", className: "font-thai text-[14px] font-normal text-white/80 hover:text-white transition-colors", children: rootLabel }) }) }), segments.map((segment, index) => {
                     const href = "/" + segments.slice(0, index + 1).join("/");
                     const isLast = index === segments.length - 1;
-                    const label = labels[segment] || segment;
-                    return (jsxRuntime.jsxs(React.Fragment, { children: [jsxRuntime.jsx(erpTheme.BreadcrumbSeparator, { className: "hidden md:block text-white/80", children: jsxRuntime.jsx(lucideReact.ChevronRight, { strokeWidth: 2, className: "h-4 w-4" }) }), jsxRuntime.jsx(erpTheme.BreadcrumbItem, { children: isLast ? (jsxRuntime.jsx(erpTheme.BreadcrumbPage, { className: "font-thai text-[14px] font-normal text-white/95 leading-none", children: decodeURIComponent(label) })) : (jsxRuntime.jsx(erpTheme.BreadcrumbLink, { asChild: true, children: jsxRuntime.jsx(Link, { href: href, className: "font-thai text-[14px] font-normal text-white/80 hover:text-white transition-colors leading-none", children: decodeURIComponent(label) }) })) })] }, `${segment}-${index}`));
+                    // A record id is not a place. Falling back to the raw segment put a
+                    // 36-character uuid in the trail, which pushed everything before it
+                    // off a phone and told the reader nothing they could use.
+                    const label = labels[segment] ?? (isRecordID(segment) ? "รายละเอียด" : segment);
+                    return (jsxRuntime.jsxs(React.Fragment, { children: [jsxRuntime.jsx(erpTheme.BreadcrumbSeparator, { className: "text-white/80", children: jsxRuntime.jsx(lucideReact.ChevronRight, { strokeWidth: 2, className: "h-4 w-4" }) }), jsxRuntime.jsx(erpTheme.BreadcrumbItem, { children: isLast ? (jsxRuntime.jsx(erpTheme.BreadcrumbPage, { className: "font-thai text-[14px] font-normal text-white/95 leading-none", children: decodeURIComponent(label) })) : (jsxRuntime.jsx(erpTheme.BreadcrumbLink, { asChild: true, children: jsxRuntime.jsx(Link, { href: href, className: "font-thai text-[14px] font-normal text-white/80 hover:text-white transition-colors leading-none", children: decodeURIComponent(label) }) })) })] }, `${segment}-${index}`));
                 })] }) }));
 }
+/**
+ * Whether a path segment is a record identifier rather than a page.
+ *
+ * Covers the uuid the APIs issue and the long opaque ids that show up in
+ * document routes. Deliberately narrow: a real page called "settings" must
+ * never be mistaken for one of these.
+ */
+function isRecordID(segment) {
+    return (/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(segment) ||
+        /^[0-9a-f]{24,}$/i.test(segment));
+}
 
 function isExternalHref(href) {
     return /^https?:\/\//i.test(href) || href.startsWith("//");