1
0
Fork 0

fix: use span for close button to ensure true transparency

- Browser button elements have default background/border
- Span with onClick provides the same functionality with no defaults
- Maintains keyboard focus states and accessibility
This commit is contained in:
Jason Fraley 2026-07-28 17:33:20 -04:00
parent fbaad2e4bb
commit 9651ad83ae

View file

@ -249,13 +249,13 @@ export const NodeEditDialog: FC<NodeEditDialogProps> = ({
<DialogHeader>
<DialogTitle className="flex items-center justify-between">
<span>Edit {typeLabel}</span>
<button
<span
onClick={() => onOpenChange(false)}
className="hover:bg-zinc-100 dark:hover:bg-zinc-800 transition-colors"
className="hover:bg-zinc-100 dark:hover:bg-zinc-800 transition-colors focus:outline-none focus-visible:ring-2 focus-visible:ring-zinc-400 rounded-full"
aria-label="Close"
>
<XIcon className="size-4 text-zinc-500" />
</button>
<XIcon className="size-8 text-zinc-500" />
</span>
</DialogTitle>
</DialogHeader>
{renderFields()}