@adea-ai/ui 0.110.1 → 0.111.0

package/src/components/ui/select/select.tsx

@@ -2,7 +2,13 @@ import { Select as KobalteSelect } from '@kobalte/core/select'
 import { Check, ChevronDown, ChevronsUpDown } from 'lucide-solid'
 import type { ComponentProps } from 'solid-js'
 import { Show, splitProps } from 'solid-js'
-import { menuContentPadding, menuItem, popoverArrow, topLayerProps } from '#lib/overlay'
+import {
+  menuContentPadding,
+  menuItem,
+  menuSwatchTone,
+  type MenuSwatchTone,
+  topLayerProps,
+} from '#lib/overlay'
 import { cn } from '#lib/utils'
 
 /**
@@ -21,7 +27,12 @@ import { cn } from '#lib/utils'
 export function Select<Option, OptGroup = never>(
   props: ComponentProps<typeof KobalteSelect<Option, OptGroup>>
 ) {
-  return <KobalteSelect {...props} />
+  // Kobalte's own default gutter is 8px; without the arrow (removed below) the
+  // popper no longer folds half an arrow box into that offset, and 8px read as
+  // a detached list. Four pixels is shadcn's `sideOffset` — attached, with
+  // room for the trigger's own border.
+  const [local, rest] = splitProps(props, ['gutter'])
+  return <KobalteSelect gutter={local.gutter ?? 4} {...rest} />
 }
 
 export type SelectTriggerProps = ComponentProps<typeof KobalteSelect.Trigger> & {
@@ -153,20 +164,34 @@ export function SelectContent(props: SelectContentProps) {
         {...rest}
       >
         {local.children}
+        {/* No arrow: the list is the trigger's own value opened up, and Kobalte
+            reserves half the arrow's 30px box as popper gutter, which is where
+            the reported gap between a form trigger and its options came from. */}
         <KobalteSelect.Listbox class="flex flex-col" />
-        <KobalteSelect.Arrow aria-hidden="true" class={popoverArrow} />
       </KobalteSelect.Content>
     </KobalteSelect.Portal>
   )
 }
 
-export function SelectItem(props: ComponentProps<typeof KobalteSelect.Item>) {
-  const [local, rest] = splitProps(props, ['class', 'children'])
+export function SelectItem(
+  props: ComponentProps<typeof KobalteSelect.Item> & {
+    /** Draw a leading colour dot from the shared tone ladder — the accent or
+     * status role the option's card badge carries. `aria-hidden`; the label
+     * carries the meaning. */
+    swatch?: MenuSwatchTone
+  }
+) {
+  const [local, rest] = splitProps(props, ['class', 'children', 'swatch'])
 
   return (
     <KobalteSelect.Item class={cn(menuItem, 'pe-8', local.class)} {...rest}>
+      <Show when={local.swatch}>
+        <span aria-hidden="true" class={menuSwatchTone({ tone: local.swatch })} />
+      </Show>
       <KobalteSelect.ItemLabel>{local.children}</KobalteSelect.ItemLabel>
-      <KobalteSelect.ItemIndicator class="absolute end-2 flex size-4 items-center justify-center">
+      {/* The indicator reads in the accent, like every other selected state in
+          the system — the row's own foreground reads as decoration. */}
+      <KobalteSelect.ItemIndicator class="absolute end-2 flex size-4 items-center justify-center text-primary">
         <Check class="size-4" />
       </KobalteSelect.ItemIndicator>
     </KobalteSelect.Item>

package/src/components/ui/sheet/sheet.tsx

@@ -46,7 +46,7 @@ import { createDialogFocusRestoration, DialogOverlay } from '../dialog/dialog'
 // scrolled programmatically (focus, scrollIntoView, scrollTop), which slid the
 // pinned footer out of view. Only SheetBody scrolls.
 const sheetVariants = cva(
-  'bg-popover text-popover-foreground fixed z-(--z-dialog) flex flex-col overflow-clip border-border shadow-lg',
+  'bg-background text-foreground fixed z-(--z-dialog) flex flex-col overflow-clip border-border shadow-lg',
   {
     variants: {
       side: {

package/src/components/ui/tooltip/tooltip.tsx

@@ -1,11 +1,12 @@
 import { Tooltip as KobalteTooltip, useTooltipContext } from '@kobalte/core/tooltip'
-import type { ComponentProps } from 'solid-js'
+import type { ComponentProps, JSX } from 'solid-js'
 import {
   createContext,
   createEffect,
   createSignal,
   createUniqueId,
   onCleanup,
+  Show,
   splitProps,
   useContext,
 } from 'solid-js'
@@ -349,13 +350,24 @@ export function TooltipTrigger(props: ComponentProps<typeof KobalteTooltip.Trigg
   )
 }
 
-export type TooltipContentProps = ComponentProps<typeof KobalteTooltip.Content>
+export type TooltipContentProps = ComponentProps<typeof KobalteTooltip.Content> & {
+  /**
+   * A leading icon, repeating the trigger's glyph inside the tip — the side
+   * rail's icon+label treatment, offered so a toolbar or rail tooltip does not
+   * have to hand-roll it. Decorative by contract: the tip is `aria-hidden`
+   * while closed and never the accessible name, so the icon must not be the
+   * only carrier of meaning. Size it by leaving the svg bare; the slot fixes
+   * it to the system's control size.
+   */
+  icon?: JSX.Element
+}
 
 export function TooltipContent(props: TooltipContentProps) {
   const interaction = useContext(TooltipInteractionContext)
   const [local, rest] = splitProps(props, [
     'class',
     'children',
+    'icon',
     'aria-hidden',
     'onEscapeKeyDown',
     'onPointerDownOutside',
@@ -380,6 +392,9 @@ export function TooltipContent(props: TooltipContentProps) {
           'data-expanded:animate-in data-expanded:fade-in-0 data-expanded:zoom-in-95',
           'data-closed:animate-out data-closed:fade-out-0 data-closed:zoom-out-95',
           'data-expanded:duration-150 data-closed:duration-100',
+          // An icon+label tip is one row that reads as a unit, so the label
+          // never wraps under the glyph.
+          { 'flex items-center gap-2 whitespace-nowrap': !!local.icon },
           local.class
         )}
         // A tooltip is never interactive: the content is plain text, and the
@@ -404,6 +419,17 @@ export function TooltipContent(props: TooltipContentProps) {
         {...topLayerProps}
         {...rest}
       >
+        <Show when={local.icon}>
+          {(icon) => (
+            <span
+              data-slot="tooltip-icon"
+              aria-hidden="true"
+              class="flex size-4 shrink-0 items-center justify-center [&_svg]:size-4"
+            >
+              {icon()}
+            </span>
+          )}
+        </Show>
         {local.children}
       </KobalteTooltip.Content>
     </KobalteTooltip.Portal>

package/src/lib/overlay.ts

@@ -10,10 +10,29 @@
  * "an overlay looks like this" a fact rather than a convention.
  */
 
-/** The panel itself: fill, hairline edge, radius, elevation. */
+import { cva, type VariantProps } from '#lib/variants'
+
+/** The panel itself: fill, hairline edge, radius, elevation. Menus, popovers
+ * and the command palette float on the elevated overlay rung (`--popover`). */
 export const overlaySurface =
   'bg-popover text-popover-foreground rounded-xl border border-border shadow-lg'
 
+/**
+ * The modal panel: the same hairline edge, radius and elevation, painted on the
+ * *neutral* theme surface instead of the elevated overlay rung.
+ *
+ * `--popover` is the catalogue's `surfaceElevated`, whose derivation amplifies
+ * the canvas hue's chroma on the elevated step. On the dark default's violet
+ * canvas that makes the dialog the most chromatic surface in the ladder —
+ * `oklch(0.2406 0.0088 300.91)`, red above green in sRGB — and on a full dialog
+ * panel that cast reads as a warm tint the canvas never has. shadcn's own
+ * convention already splits the two: a Dialog paints `bg-background`, menus and
+ * popovers paint `bg-popover`. The veil, the hairline and the elevation shadow
+ * carry the separation the lighter fill no longer provides.
+ */
+export const dialogSurface =
+  'bg-background text-foreground rounded-xl border border-border shadow-lg'
+
 /**
  * Overlay titles have one fixed typography role. Keep this small recipe shared
  * without loading the configurable Heading component into every dialog.
@@ -85,6 +104,31 @@ export const menuLabel =
 /** The rule between menu groups. */
 export const menuSeparator = 'bg-border -mx-1 my-1 h-px'
 
+/**
+ * A leading colour swatch on a menu or select row: the sanctioned way to carry
+ * a card-tone colour into a list of options. The tones are the system's accent
+ * and status roles — the vocabulary a Badge already speaks — so the colour an
+ * option shows in a menu is the colour its badge carries on a card, and a
+ * caller never passes a raw colour into a list. The dot is `aria-hidden`; the
+ * label carries the meaning, so a tone is never the only signal.
+ */
+export const menuSwatchTone = cva('size-2 shrink-0 rounded-full', {
+  variants: {
+    tone: {
+      neutral: 'bg-muted-foreground',
+      primary: 'bg-primary',
+      success: 'bg-success',
+      warning: 'bg-warning',
+      destructive: 'bg-destructive',
+      info: 'bg-info',
+    },
+  },
+  defaultVariants: { tone: 'neutral' },
+})
+
+/** The tones a menu row's swatch can take. */
+export type MenuSwatchTone = NonNullable<VariantProps<typeof menuSwatchTone>['tone']>
+
 /** Padding shared by every menu-list surface. */
 export const menuContentPadding = 'p-1'