From 8cee97614d0dec7b24c5618438ec13e1266a8ef5 Mon Sep 17 00:00:00 2001 From: Matthew L McPeak Date: Tue, 14 Jul 2026 09:40:45 -0400 Subject: [PATCH] fix: use text-foreground for the Dialog close icon The close icon used the Button "secondary" variant's text-secondary-foreground, a near-white token fixed across both themes. That reads fine against the dark Hades popover but is almost invisible against the light Apollo popover since bg-transparent removes the variant's own background. text-foreground already adapts per-theme and is what a plain icon-only dismiss button should use. Pre-existing bug, unrelated to the reka-ui removal (same classes existed before) -- confirmed via computed-style + screenshot checks in both themes in headless Chromium. --- packages/library/src/components/ui/dialog/DialogContent.vue | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/library/src/components/ui/dialog/DialogContent.vue b/packages/library/src/components/ui/dialog/DialogContent.vue index cce7820..92bb848 100644 --- a/packages/library/src/components/ui/dialog/DialogContent.vue +++ b/packages/library/src/components/ui/dialog/DialogContent.vue @@ -52,7 +52,7 @@ watch(context.open, (value) => { isLocked.value = value }, { immediate: true }) -