granite icon indicating copy to clipboard operation
granite copied to clipboard

Styling: Add `.destructive` class for `button.text-button`

Open wpkelso opened this issue 7 months ago • 3 comments

image

Image excluded for now because I don't believe there is a precedent style.

wpkelso avatar May 23 '25 18:05 wpkelso

I don't think this works in the dark style:

Screenshot from 2025-07-01 11 51 56

Compared to the old stylesheet:

Screenshot from 2025-07-01 11 53 18

danirabbit avatar Jul 01 '25 18:07 danirabbit

@danirabbit Ah, you're right, I totally didn't check that. Should probably just use SILVER_100 foreground for both cases? I'm afk right now so I can't verify that provides sufficient contrast for dark mode as well, but if it doesn't then we might need to modify the dark mode error color

wpkelso avatar Jul 01 '25 19:07 wpkelso

@wpkelso in the old stylesheet we just use white

danirabbit avatar Jul 01 '25 19:07 danirabbit