@inshorts/glint-ui 0.5.1 → 0.5.3

package/README.md

@@ -48,8 +48,9 @@ import { Button, Modal, cn } from "@inshorts/glint-ui";
 import { Button } from "@inshorts/glint-ui/button";
 ```
 
-`ImageUploader` takes `uploadFile` / `uploadFromUrl` props — each panel passes
-its own BFF upload route. Absolute `https://` pastes work without a handler.
+`ImageUploader` keeps Upload and URL paste enabled by default. Pass
+`uploadFile` / `uploadFromUrl` for the panel BFF; without them, paste still
+applies any URL as-is and Upload surfaces a config error if a file is picked.
 
 ## Develop
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inshorts/glint-ui",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Glint Tailwind 4 UI kit for Inshorts Next panels.",
   "license": "UNLICENSED",
   "type": "module",

package/src/image-uploader.tsx

@@ -18,12 +18,14 @@ export type ImageUploaderProps = {
   accept?: string;
   /**
    * Upload a picked file and return its hosted URL.
-   * Required for the Upload button — without it, only URL paste works.
+   * Injected by the panel — Upload stays enabled either way; without a
+   * handler, picking a file surfaces a configuration error.
    */
   uploadFile?: (file: File) => Promise<string>;
   /**
-   * Resolve a non-http pasted value (relative path, etc.) into a hosted URL.
-   * Absolute `http(s)` values are applied as-is without calling this.
+   * Optional transform for pasted values (e.g. host a relative path).
+   * When omitted, any pasted non-empty string is applied as-is. Absolute
+   * `http(s)` URLs never call this.
    */
   uploadFromUrl?: (imageUrl: string) => Promise<string>;
 };
@@ -31,8 +33,9 @@ export type ImageUploaderProps = {
 /**
  * Reusable image uploader.
  *
- * Hosting is injected: each panel passes its own `uploadFile` / `uploadFromUrl`
- * so the kit stays free of panel-specific API routes.
+ * Upload and URL paste are both enabled by default. Hosting is injected:
+ * each panel passes `uploadFile` / `uploadFromUrl` so the kit stays free of
+ * panel-specific API routes.
  */
 export function ImageUploader({
   value = "",
@@ -48,6 +51,7 @@ export function ImageUploader({
   const inputRef = React.useRef<HTMLInputElement>(null);
   const [uploading, setUploading] = React.useState(false);
   const [error, setError] = React.useState<string | null>(null);
+  const busy = Boolean(disabled || uploading);
 
   const handleFile = async (file: File) => {
     if (!uploadFile) {
@@ -74,18 +78,14 @@ export function ImageUploader({
     }
     if (trimmed === value) return;
 
-    // Already-hosted absolute URLs can be used as-is.
-    if (/^https?:\/\//i.test(trimmed)) {
+    // Absolute URLs — and any paste when no transform is configured — apply
+    // as-is. Paste is never gated behind a disabled input.
+    if (!uploadFromUrl || /^https?:\/\//i.test(trimmed)) {
       setError(null);
       onChange(trimmed);
       return;
     }
 
-    if (!uploadFromUrl) {
-      setError("Paste an absolute image URL, or configure URL upload.");
-      return;
-    }
-
     setUploading(true);
     setError(null);
     try {
@@ -109,7 +109,7 @@ export function ImageUploader({
           type="file"
           accept={accept}
           className="hidden"
-          disabled={disabled || uploading || !uploadFile}
+          disabled={busy}
           onChange={(e) => {
             const file = e.target.files?.[0];
             if (file) void handleFile(file);
@@ -119,7 +119,7 @@ export function ImageUploader({
           type="button"
           variant="outline"
           size="sm"
-          disabled={disabled || uploading || !uploadFile}
+          disabled={busy}
           onClick={() => inputRef.current?.click()}
         >
           {uploading ? (
@@ -134,7 +134,7 @@ export function ImageUploader({
             type="button"
             variant="ghost"
             size="sm"
-            disabled={disabled || uploading}
+            disabled={busy}
             onClick={() => onChange("")}
             aria-label="Clear image"
           >
@@ -148,7 +148,7 @@ export function ImageUploader({
         <Input
           key={value || "empty"}
           defaultValue={value}
-          disabled={disabled || uploading}
+          disabled={busy}
           placeholder="Or paste image URL…"
           onBlur={(e) => {
             void handleUrl(e.target.value);