jake/Lumotia
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(settings-group): add optional icon prop and tighten title/description rhythm

Browse Source 
Title bumped to font-semibold so it scans first; description gets tracking-wide
plus mt-1 for cleaner rhythm. Chevron alignment shifted from mt-0.5 to mt-1 to
match the new spacing. Optional `icon` prop renders a Lucide component between
the chevron and the title block at 16x16, text-tertiary, no colour change on
open. Layout is preserved exactly when no icon is passed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jake
2026-05-07 10:07:39 +01:00
parent aebf6cd149
commit 3a27031eba
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/lib/components/SettingsGroup.svelte
View File

@@ -5,16 +5,18 @@
// (modern Chromium / WebKit), and falls back to instant open/close
// elsewhere. prefers-reduced-motion disables both.
import { ChevronRight } from "lucide-svelte";
import type { Component } from "svelte";
interface Props {
title: string;
description?: string;
open?: boolean;
onopen?: () => void;
icon?: Component<{ size?: number; class?: string; "aria-hidden"?: boolean | "true" | "false" }>;
children?: import("svelte").Snippet;
}
let { title, description = "", open = false, onopen, children }: Props = $props();
let { title, description = "", open = false, onopen, icon: Icon, children }: Props = $props();
// Fires once on the first transition from closed → open. Used by
// sections that lazy-load expensive data (e.g. TTS voice enumeration).
@@ -47,13 +49,16 @@
class="flex items-start gap-2 cursor-pointer list-none py-3 px-1 rounded hover:bg-hover focus-visible:outline focus-visible:outline-2 focus-visible:outline-accent"
>
<ChevronRight
class="chevron size-4 mt-0.5 shrink-0 text-text-tertiary"
class="chevron size-4 mt-1 shrink-0 text-text-tertiary"
aria-hidden="true"
/>
{#if Icon}
<Icon size={16} class="mt-1 shrink-0 text-text-tertiary" aria-hidden="true" />
{/if}
<div class="flex-1">
<p class="text-sm text-text font-medium">{title}</p>
<p class="text-sm text-text font-semibold">{title}</p>
{#if description}
<p class="text-[11px] text-text-tertiary mt-0.5">{description}</p>
<p class="text-[11px] text-text-tertiary tracking-wide mt-1">{description}</p>
{/if}
</div>
</summary>