@stevederico/skateboard-ui 2.10.0 → 2.11.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # CHANGELOG
 
+2.11.0
+
+  Fix DynamicIcon lazy loading
+  Remove barrel import lucide-react
+  Hoist LandingView static values
+  Add FeatureIcon emoji fallback
+
+2.10.1
+
+  Add Geist font files self-hosted
+  Add @font-face declarations
+
 2.10.0
 
   Fix landing page feature icons

package/core/DynamicIcon.jsx

@@ -1,28 +1,11 @@
-import React from "react";
-import * as LucideIcons from "lucide-react";
+import React, { useState, useEffect } from "react";
 import { cn } from "../shadcn/lib/utils.js";
 
 /**
- * Render a Lucide icon by name string.
+ * Convert a kebab-case, snake_case, or space-separated string to PascalCase.
  *
- * Accepts kebab-case, snake_case, or PascalCase icon names and resolves
- * them to the matching lucide-react component. Supports className merging
- * via shadcn cn() utility.
- *
- * @param {Object} props
- * @param {string} props.name - Icon name (e.g. "home", "arrow-right", "Settings")
- * @param {number} [props.size=24] - Icon size in pixels
- * @param {string} [props.color='currentColor'] - Icon stroke color
- * @param {number} [props.strokeWidth=2] - Stroke width
- * @param {string} [props.className] - Additional CSS classes
- * @returns {JSX.Element|null} Rendered icon or null if name not found
- *
- * @example
- * import DynamicIcon from '@stevederico/skateboard-ui/DynamicIcon';
- *
- * <DynamicIcon name="home" size={24} />
- * <DynamicIcon name="arrow-right" size={20} color="red" />
- * <DynamicIcon name="settings" className="text-muted-foreground" />
+ * @param {string} str - Input string
+ * @returns {string} PascalCase version
  */
 function toPascalCase(str) {
   return str
@@ -31,28 +14,112 @@ function toPascalCase(str) {
     .join("");
 }
 
-function resolveIcon(name) {
-  const candidates = [name, toPascalCase(name), name.charAt(0).toUpperCase() + name.slice(1)];
+let iconsModule = null;
+let iconsPromise = null;
+
+/**
+ * Lazily load the lucide-react module on first use.
+ * Subsequent calls return the cached module immediately.
+ *
+ * @returns {Promise<Object>} The lucide-react module
+ */
+function loadIcons() {
+  if (iconsModule) return Promise.resolve(iconsModule);
+  if (!iconsPromise) {
+    iconsPromise = import("lucide-react").then((mod) => {
+      iconsModule = mod;
+      return mod;
+    });
+  }
+  return iconsPromise;
+}
+
+/**
+ * Resolve icon name to a Lucide component from the cached module.
+ *
+ * @param {Object} mod - The lucide-react module
+ * @param {string} name - Icon name (kebab-case, snake_case, or PascalCase)
+ * @returns {React.ComponentType|null} Icon component or null
+ */
+function resolveIcon(mod, name) {
+  if (!mod) return null;
+  const candidates = [
+    name,
+    toPascalCase(name),
+    name.charAt(0).toUpperCase() + name.slice(1),
+  ];
   for (const candidate of candidates) {
-    if (LucideIcons[candidate]) return LucideIcons[candidate];
+    if (mod[candidate]) return mod[candidate];
   }
   return null;
 }
 
-const DynamicIcon = ({ name, size = 24, color = "currentColor", strokeWidth = 2, className, ...props }) => {
-  const Icon = resolveIcon(name);
-  if (!Icon) return null;
-  return <Icon size={size} color={color} strokeWidth={strokeWidth} className={cn(className)} {...props} />;
-};
-
 /**
  * Check if a name string resolves to a valid Lucide icon.
+ * Returns false until the icon module has been loaded.
  *
  * @param {string} name - Icon name to check
  * @returns {boolean} True if name resolves to an icon
  */
 export function canResolveIcon(name) {
-  return resolveIcon(name) !== null;
+  if (!iconsModule) return false;
+  return resolveIcon(iconsModule, name) !== null;
 }
 
+/**
+ * Render a Lucide icon by name string with lazy loading.
+ *
+ * Loads lucide-react on first render via dynamic import, then caches the
+ * module for all subsequent uses. Shows nothing while loading (typically
+ * under 50ms from cache).
+ *
+ * @param {Object} props
+ * @param {string} props.name - Icon name (e.g. "home", "arrow-right", "Settings")
+ * @param {number} [props.size=24] - Icon size in pixels
+ * @param {string} [props.color='currentColor'] - Icon stroke color
+ * @param {number} [props.strokeWidth=2] - Stroke width
+ * @param {string} [props.className] - Additional CSS classes
+ * @returns {JSX.Element|null} Rendered icon or null if loading/not found
+ *
+ * @example
+ * import DynamicIcon from '@stevederico/skateboard-ui/DynamicIcon';
+ *
+ * <DynamicIcon name="home" size={24} />
+ * <DynamicIcon name="arrow-right" size={20} color="red" />
+ * <DynamicIcon name="settings" className="text-muted-foreground" />
+ */
+const DynamicIcon = ({
+  name,
+  size = 24,
+  color = "currentColor",
+  strokeWidth = 2,
+  className,
+  ...props
+}) => {
+  const [Icon, setIcon] = useState(() => {
+    if (iconsModule) return resolveIcon(iconsModule, name);
+    return null;
+  });
+
+  useEffect(() => {
+    if (Icon) return;
+    loadIcons().then((mod) => {
+      const resolved = resolveIcon(mod, name);
+      setIcon(() => resolved);
+    });
+  }, [name, Icon]);
+
+  if (!Icon) return null;
+
+  return (
+    <Icon
+      size={size}
+      color={color}
+      strokeWidth={strokeWidth}
+      className={cn(className)}
+      {...props}
+    />
+  );
+};
+
 export default DynamicIcon;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stevederico/skateboard-ui",
   "private": false,
-  "version": "2.10.0",
+  "version": "2.11.0",
   "type": "module",
   "exports": {
     "./Sidebar": {

package/styles.css

@@ -1,3 +1,19 @@
+@font-face {
+  font-family: 'Geist';
+  src: url('./fonts/GeistVF.woff2') format('woff2');
+  font-weight: 100 900;
+  font-style: normal;
+  font-display: swap;
+}
+
+@font-face {
+  font-family: 'Geist Mono';
+  src: url('./fonts/GeistMonoVF.woff2') format('woff2');
+  font-weight: 100 900;
+  font-style: normal;
+  font-display: swap;
+}
+
 @import "tailwindcss";
 
 @source '../';

package/views/LandingView.jsx

@@ -2,23 +2,42 @@ import React from 'react';
 import { useNavigate } from 'react-router-dom';
 import { useTheme } from 'next-themes';
 import { getState } from "../core/Context.jsx";
-import DynamicIcon, { canResolveIcon } from '../core/DynamicIcon.jsx';
+import DynamicIcon from '../core/DynamicIcon.jsx';
 import { Sun, Moon, Check } from 'lucide-react';
 import { Button } from '../shadcn/ui/button.jsx';
 import { Card, CardHeader, CardTitle, CardDescription, CardContent, CardFooter } from '../shadcn/ui/card.jsx';
 import { Badge } from '../shadcn/ui/badge.jsx';
 import { Separator } from '../shadcn/ui/separator.jsx';
 
+const CURRENT_YEAR = new Date().getFullYear();
+
+const DEFAULT_NAV_LINKS = [
+  { label: 'Features', href: '#features' },
+];
+
+const DEFAULT_NAV_LINKS_SUFFIX = [
+  { label: 'Terms', href: '/terms' },
+];
+
+const PRICING_LINK = { label: 'Pricing', href: '#pricing' };
+
+const PRIVACY_LINK = { label: 'Privacy', href: '/privacy' };
+const TERMS_LINK = { label: 'Terms', href: '/terms' };
+const EULA_LINK = { label: 'EULA', href: '/eula' };
+
 /**
  * Renders a feature icon from a name string or emoji.
- * Resolves Lucide icon names (kebab-case), falls back to raw text (emoji).
+ * If the name looks like an icon identifier (ASCII letters, digits, hyphens),
+ * renders it via DynamicIcon with lazy loading. Otherwise treats it as
+ * raw text (e.g. an emoji).
  *
  * @param {Object} props
  * @param {string} props.name - Icon name (kebab-case) or emoji string
  * @returns {JSX.Element} Icon component or text span
  */
 function FeatureIcon({ name }) {
-  if (canResolveIcon(name)) return <DynamicIcon name={name} size={24} />;
+  const isIconName = /^[a-z][a-z0-9-]*$/i.test(name);
+  if (isIconName) return <DynamicIcon name={name} size={24} />;
   return <span>{name}</span>;
 }
 
@@ -57,9 +76,9 @@ export default function LandingView() {
 
           <div className="hidden md:flex gap-6">
             {(constants.navLinks || [
-              { label: 'Features', href: '#features' },
-              ...(constants.stripeProducts?.length > 0 ? [{ label: 'Pricing', href: '#pricing' }] : []),
-              { label: 'Terms', href: '/terms' },
+              ...DEFAULT_NAV_LINKS,
+              ...(constants.stripeProducts?.length > 0 ? [PRICING_LINK] : []),
+              ...DEFAULT_NAV_LINKS_SUFFIX,
             ]).map((link, index) => (
               <a key={index} href={link.href} className="text-muted-foreground hover:text-foreground transition-colors font-semibold">{link.label}</a>
             ))}
@@ -171,14 +190,14 @@ export default function LandingView() {
           <Separator className="mb-8" />
           <div className="flex justify-center gap-8 mb-6">
             {(constants.footerLinks || [
-              ...(constants.privacyPolicy ? [{ label: 'Privacy', href: '/privacy' }] : []),
-              ...(constants.termsOfService ? [{ label: 'Terms', href: '/terms' }] : []),
-              ...(constants.EULA ? [{ label: 'EULA', href: '/eula' }] : []),
+              ...(constants.privacyPolicy ? [PRIVACY_LINK] : []),
+              ...(constants.termsOfService ? [TERMS_LINK] : []),
+              ...(constants.EULA ? [EULA_LINK] : []),
             ]).map((link, index) => (
               <a key={index} href={link.href} className="text-muted-foreground hover:text-foreground transition-colors font-semibold">{link.label}</a>
             ))}
           </div>
-          <p className="text-center text-muted-foreground">&copy; {new Date().getFullYear()} {constants.companyName}. {constants.copyrightText || 'All rights reserved.'}</p>
+          <p className="text-center text-muted-foreground">&copy; {CURRENT_YEAR} {constants.companyName}. {constants.copyrightText || 'All rights reserved.'}</p>
         </div>
       </footer>
     </div>