diff --git a/web/src/components/ui/dialog.tsx b/web/src/components/ui/dialog.tsx index 93d7652dd..33b414bfb 100644 --- a/web/src/components/ui/dialog.tsx +++ b/web/src/components/ui/dialog.tsx @@ -77,7 +77,7 @@ const DialogContent = React.forwardRef< {children} {showCloseButton && ( - + Close diff --git a/web/src/components/ui/sheet.tsx b/web/src/components/ui/sheet.tsx index 4d98bc41d..54d236219 100644 --- a/web/src/components/ui/sheet.tsx +++ b/web/src/components/ui/sheet.tsx @@ -75,7 +75,7 @@ const SheetContent = React.forwardRef< {...props} > {children} - + Close diff --git a/web/src/index.css b/web/src/index.css index eb777ebcc..d200d24e4 100644 --- a/web/src/index.css +++ b/web/src/index.css @@ -12,7 +12,7 @@ @layer base { * { - @apply border-border outline-ring/50; + @apply border-border outline-none ring-0; } body { @apply bg-background text-foreground; diff --git a/web/src/themes/COLOR_GUIDE.md b/web/src/themes/COLOR_GUIDE.md index 2c9e1ef81..4197c858e 100644 --- a/web/src/themes/COLOR_GUIDE.md +++ b/web/src/themes/COLOR_GUIDE.md @@ -83,14 +83,12 @@ The color system supports both light and dark themes automatically through CSS c | `--accent-foreground` | Dark gray | Light gray | Text on accent backgrounds | | `--border` | Medium light | Medium dark | Dividers, input borders | | `--input` | Medium light | Medium dark | Form input backgrounds | -| `--ring` | Blue | Blue | Focus outlines | **When to use:** - Hover states (`--accent`) - Form field borders (`--border`) - Input field backgrounds (`--input`) -- Focus indicators (`--ring`) ### ⚠️ Feedback Colors @@ -134,7 +132,6 @@ The color system supports both light and dark themes automatically through CSS c | `--sidebar-accent` | Sidebar hover states | | `--sidebar-accent-foreground` | Text on sidebar hover states | | `--sidebar-border` | Sidebar dividers | -| `--sidebar-ring` | Sidebar focus indicators | ## Best Practices @@ -149,7 +146,6 @@ The color system supports both light and dark themes automatically through CSS c ``` 2. **Use semantic meaning:** - - Primary = main actions - Secondary = supporting actions - Destructive = dangerous/delete actions @@ -170,7 +166,6 @@ The color system supports both light and dark themes automatically through CSS c ``` 2. **Don't use colors outside their intended purpose:** - - Don't use destructive colors for positive actions - Don't use primary colors for secondary elements @@ -196,7 +191,6 @@ document.documentElement.classList.toggle("dark"); ## Accessibility - All color pairs meet WCAG contrast requirements -- Focus indicators use `--ring` for consistency - Color is never the only means of conveying information ## Implementation Examples @@ -235,10 +229,6 @@ document.documentElement.classList.toggle("dark"); color: var(--foreground); border: 1px solid var(--border); } - -.input:focus { - outline: 2px solid var(--ring); -} ``` ### Cards and Containers