@lyeve-labs/ui-kit 0.22.1 → 0.23.0

package/dist/components/Button.svelte

@@ -2,6 +2,7 @@
   import type { Snippet } from 'svelte';
   import type { HTMLAnchorAttributes, HTMLButtonAttributes } from 'svelte/elements';
   import Spinner from './Spinner.svelte';
+  import Tooltip from './Tooltip.svelte';
   import { safeHref } from '../internal/href.js';
 
   type Variant = 'primary' | 'secondary' | 'danger' | 'ghost' | 'outline' | 'violet';
@@ -25,6 +26,18 @@
     type?: 'button' | 'submit' | 'reset';
     href?: string;
     full?: boolean;
+    /**
+     * The words shown on hover and focus. Defaults to the button's own
+     * aria-label, so an icon-only button names itself to a sighted reader the
+     * way it already does to a screen reader; a string replaces those words,
+     * and `false` keeps the button silent.
+     *
+     * Thirty-five icon-only actions across one console had a name and no
+     * hint, so a pointer user learned what the trash can did by pressing it.
+     * The hint lives here rather than on each caller because the rule that
+     * makes a row action icon-only is the kit's, and so is the fix.
+     */
+    hint?: string | false;
     class?: string;
     onclick?: (e: MouseEvent) => void;
     children: Snippet;
@@ -38,12 +51,18 @@
     type = 'button',
     href = undefined,
     full = false,
+    hint = undefined,
     class: klass = '',
     onclick,
     children,
     ...rest
   }: Props = $props();
 
+  /** The hint text, or nothing: an explicit string, else the aria-label, unless switched off. */
+  const hintText = $derived(
+    hint === false ? undefined : (hint ?? (rest['aria-label'] || undefined)),
+  );
+
   /**
    * Colour per variant, in three states: at rest, under a pointer, and held.
    *
@@ -104,27 +123,40 @@
   const inert = $derived(disabled || loading);
 </script>
 
-{#if href}
-  <a
-    href={inert ? undefined : resolvedHref}
-    class="{cls} {inert ? 'pointer-events-none opacity-50' : ''}"
-    aria-disabled={inert ? 'true' : undefined}
-    tabindex={inert ? -1 : undefined}
-    {...rest}
-  >
-    {#if loading}<Spinner size={spinnerSize} />{/if}
-    {@render children()}
-  </a>
+{#snippet control()}
+  {#if href}
+    <a
+      href={inert ? undefined : resolvedHref}
+      class="{cls} {inert ? 'pointer-events-none opacity-50' : ''}"
+      aria-disabled={inert ? 'true' : undefined}
+      tabindex={inert ? -1 : undefined}
+      {...rest}
+    >
+      {#if loading}<Spinner size={spinnerSize} />{/if}
+      {@render children()}
+    </a>
+  {:else}
+    <button
+      {type}
+      disabled={disabled || loading}
+      aria-busy={loading ? 'true' : undefined}
+      {onclick}
+      class={cls}
+      {...rest}
+    >
+      {#if loading}<Spinner size={spinnerSize} />{/if}
+      {@render children()}
+    </button>
+  {/if}
+{/snippet}
+
+<!-- The hint repeats the name, so it does not describe the control a second
+     time; the aria-label already announces it. The wrapper takes the full
+     width along with the button, or a full button would shrink to its text. -->
+{#if hintText}
+  <Tooltip text={hintText} describe={false} class={full ? 'w-full' : ''}>
+    {@render control()}
+  </Tooltip>
 {:else}
-  <button
-    {type}
-    disabled={disabled || loading}
-    aria-busy={loading ? 'true' : undefined}
-    {onclick}
-    class={cls}
-    {...rest}
-  >
-    {#if loading}<Spinner size={spinnerSize} />{/if}
-    {@render children()}
-  </button>
+  {@render control()}
 {/if}

package/dist/components/Button.svelte.d.ts

@@ -16,6 +16,18 @@ interface Props extends Forwarded {
     type?: 'button' | 'submit' | 'reset';
     href?: string;
     full?: boolean;
+    /**
+     * The words shown on hover and focus. Defaults to the button's own
+     * aria-label, so an icon-only button names itself to a sighted reader the
+     * way it already does to a screen reader; a string replaces those words,
+     * and `false` keeps the button silent.
+     *
+     * Thirty-five icon-only actions across one console had a name and no
+     * hint, so a pointer user learned what the trash can did by pressing it.
+     * The hint lives here rather than on each caller because the rule that
+     * makes a row action icon-only is the kit's, and so is the fix.
+     */
+    hint?: string | false;
     class?: string;
     onclick?: (e: MouseEvent) => void;
     children: Snippet;

package/dist/components/Tooltip.svelte

@@ -6,11 +6,21 @@
   interface Props {
     text: string;
     position?: Position;
+    /**
+     * Whether the text is the trigger's accessible description.
+     *
+     * On by default, because a tooltip that nothing points at is read by
+     * nobody. Off for a trigger that already carries the same words as its
+     * name: an icon-only button named by aria-label would otherwise be
+     * announced as "Delete, Delete", so the hint stays visual there and the
+     * name does the announcing.
+     */
+    describe?: boolean;
     class?: string;
     children: Snippet;
   }
 
-  let { text, position = 'top', class: cls = '', children }: Props = $props();
+  let { text, position = 'top', describe = true, class: cls = '', children }: Props = $props();
 
   let visible = $state(false);
   let wrapper = $state<HTMLElement>();
@@ -32,6 +42,7 @@
    * text was visible to a sighted mouse user and to nobody else.
    */
   $effect(() => {
+    if (!describe) return;
     const trigger = wrapper?.querySelector<HTMLElement>(
       'a[href], button, input, select, textarea, [tabindex]:not([tabindex="-1"])',
     );
@@ -70,7 +81,8 @@
        moving onto it does not dismiss it, which SC 1.4.13 also requires. -->
   <span
     {id}
-    role="tooltip"
+    role={describe ? 'tooltip' : undefined}
+    aria-hidden={describe ? undefined : 'true'}
     hidden={!visible}
     class="absolute {pos[position]} z-tooltip max-w-[min(16rem,calc(100vw-2rem))]
       rounded-lg border border-line bg-surface-2 px-2.5 py-1.5 text-xs text-fg shadow-xl"

package/dist/components/Tooltip.svelte.d.ts

@@ -3,6 +3,16 @@ type Position = 'top' | 'bottom' | 'left' | 'right';
 interface Props {
     text: string;
     position?: Position;
+    /**
+     * Whether the text is the trigger's accessible description.
+     *
+     * On by default, because a tooltip that nothing points at is read by
+     * nobody. Off for a trigger that already carries the same words as its
+     * name: an icon-only button named by aria-label would otherwise be
+     * announced as "Delete, Delete", so the hint stays visual there and the
+     * name does the announcing.
+     */
+    describe?: boolean;
     class?: string;
     children: Snippet;
 }

package/dist/components/dialog/ConfirmDialog.svelte

@@ -42,6 +42,9 @@
   let confirmLabel = $derived(
     typeof entry.meta?.confirmLabel === 'string' ? entry.meta.confirmLabel : 'Confirm',
   );
+  // An identifier the message refers to. Set apart in monospace on its own
+  // line rather than inside the sentence, where a UUID wraps mid-string.
+  let detail = $derived(typeof entry.meta?.confirmDetail === 'string' ? entry.meta.confirmDetail : '');
 
   async function handleConfirm() {
     if (loading) return;
@@ -80,12 +83,16 @@
       {#if message}
         <p class="text-sm text-muted mt-1">{message}</p>
       {/if}
+      {#if detail}
+        <p class="mt-2 break-all font-mono text-xs text-faint">{detail}</p>
+      {/if}
     </div>
   </div>
 
-  <!-- Actions -->
+  <!-- Actions. Cancel takes focus first: a destructive dialog must not
+       accept an Enter pressed before it was read. -->
   <div class="flex items-center justify-end gap-3 pt-2">
-    <Button variant="secondary" size="sm" onclick={handleCancel}>
+    <Button variant="secondary" size="sm" onclick={handleCancel} data-initial-focus>
       {cancelLabel}
     </Button>
     <Button variant="danger" size="sm" onclick={handleConfirm} {loading}>

package/dist/components/dialog/Dialog.svelte

@@ -137,34 +137,39 @@
     tabindex="-1"
     onkeydown={handleKeydown}
   >
-    <!-- Header -->
-    <div class="flex items-center justify-between px-6 pt-6 pb-2">
-      <div class="flex-1 min-w-0">
-        {#if entry.options.title && entry.meta?.confirmTitle === undefined}
-          {#if typeof entry.options.title === 'string'}
-            <h2 class="text-lg font-semibold text-fg truncate">
-              {entry.options.title}
-            </h2>
-          {:else}
-            {@render entry.options.title()}
+    <!-- Header. A confirm dialog has none: it renders its own title beside
+         the warning icon and offers Cancel, so a header here was an empty row
+         with a close control in it, and that control was the first thing
+         focused. Escape still dismisses. -->
+    {#if entry.meta?.confirmTitle === undefined}
+      <div class="flex items-center justify-between px-6 pt-6 pb-2">
+        <div class="flex-1 min-w-0">
+          {#if entry.options.title}
+            {#if typeof entry.options.title === 'string'}
+              <h2 class="text-lg font-semibold text-fg truncate">
+                {entry.options.title}
+              </h2>
+            {:else}
+              {@render entry.options.title()}
+            {/if}
           {/if}
-        {/if}
-      </div>
+        </div>
 
-      {#if !entry.options.persistent}
-        <button
-          class="inline-flex items-center justify-center w-8 h-8 -me-2 rounded-lg
+        {#if !entry.options.persistent}
+          <button
+            class="inline-flex items-center justify-center w-8 h-8 -me-2 rounded-lg
 						text-muted hover:text-fg hover:bg-surface-2 transition-colors duration-150 shrink-0"
-          onclick={() => handleClose()}
-          aria-label="Close"
-        >
-          <X class="w-4 h-4" />
-        </button>
-      {/if}
-    </div>
+            onclick={() => handleClose()}
+            aria-label="Close"
+          >
+            <X class="w-4 h-4" />
+          </button>
+        {/if}
+      </div>
+    {/if}
 
     <!-- Body -->
-    <div class="px-6 py-2">
+    <div class={entry.meta?.confirmTitle === undefined ? 'px-6 py-2' : 'px-6 pt-6 pb-2'}>
       {#if entry.options.body}
         {@render entry.options.body()}
       {:else if entry.meta?.confirmTitle !== undefined}

package/dist/components/dialog/dialog-manager.svelte.d.ts

@@ -55,6 +55,12 @@ export interface ConfirmOptions {
     confirmLabel?: string;
     /** The verb on the dismissing button. Defaults to `Cancel`. */
     cancelLabel?: string;
+    /**
+     * An identifier the message refers to, such as the id of the record about
+     * to be deleted. Rendered on its own line in monospace, so a UUID never
+     * wraps inside the sentence.
+     */
+    detail?: string;
 }
 /**
  * Convenience: confirm dialog.

package/dist/components/dialog/dialog-manager.svelte.js

@@ -151,6 +151,7 @@ export function confirm(title, message, options) {
         // would be indistinguishable from an override to the empty case.
         ...(options?.confirmLabel !== undefined ? { confirmLabel: options.confirmLabel } : {}),
         ...(options?.cancelLabel !== undefined ? { cancelLabel: options.cancelLabel } : {}),
+        ...(options?.detail !== undefined ? { confirmDetail: options.detail } : {}),
     });
     // Cancelling a dialog dismisses it, and dismissal rejects. Callers write
     // `if (await confirm(...))`, so a rejection on Cancel is an unhandled

package/dist/index.d.ts

@@ -89,4 +89,4 @@ export type { ConfirmOptions } from './components/dialog/dialog-manager.svelte.j
 export type { DialogOptions, DialogEntry, DialogSize } from './components/dialog/types.js';
 export { cn, type ClassValue } from './utils/cn.js';
 export { getTheme, getThemePreference, nextThemePreference, resolveTheme, setTheme, setThemePreference, systemTheme, themeBootScript, toggleTheme, watchSystemTheme, THEME_PREFERENCES, type Theme, type ThemePreference, } from './utils/theme.js';
-export declare const VERSION = "0.22.1";
+export declare const VERSION = "0.23.0";

package/dist/index.js

@@ -96,4 +96,4 @@ export { getTheme, getThemePreference, nextThemePreference, resolveTheme, setThe
 // ── Version ────────────────────────────────────────────────────────────────
 // Generated from package.json by `pnpm version:sync`. Bump package.json, never
 // this line; the build and the test suite fail when the two disagree.
-export const VERSION = '0.22.1';
+export const VERSION = '0.23.0';

package/dist/internal/overlay.js

@@ -50,9 +50,13 @@ function focusable(node) {
 export function overlay(node) {
     const previous = document.activeElement instanceof HTMLElement ? document.activeElement : null;
     lockBodyScroll();
-    // A panel with nothing focusable still has to receive focus, or the screen
-    // reader stays on the element behind the overlay and reads the wrong thing.
-    const first = focusable(node)[0];
+    // A panel may name where focus lands with data-initial-focus: a confirm
+    // dialog puts it on Cancel, so that Enter on a dialog a person has not read
+    // yet does nothing, and the close control is never the first thing focused.
+    // Otherwise the first focusable element takes it. A panel with nothing
+    // focusable still has to receive focus, or the screen reader stays on the
+    // element behind the overlay and reads the wrong thing.
+    const first = node.querySelector('[data-initial-focus]') ?? focusable(node)[0];
     if (first) {
         first.focus();
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lyeve-labs/ui-kit",
-  "version": "0.22.1",
+  "version": "0.23.0",
   "description": "A clean, accessible, palette-aware Svelte 5 component library. The design system behind LyEve.",
   "license": "MIT",
   "author": "LyEve Labs <hello@lyeve.com>",