Switch hover back to default; minor styles adjustment#2431
Switch hover back to default; minor styles adjustment#2431josh-collinsworth merged 6 commits intomainfrom
Conversation
| @import "tailwindcss"; | ||
|
|
||
| @theme { | ||
| @theme static { |
There was a problem hiding this comment.
This ensures our default theme color variables are not tree-shaken. Adds a tiny bit of extra CSS to the final output, but lets you play with all the variables in the browser as needed.
| @apply text-deploy-500 decoration-deploy-500/20 hover:decoration-deploy-500; | ||
| } | ||
|
|
||
| .learn-link { |
There was a problem hiding this comment.
These are actually changes that got missed from a separate PR but that have no visible effect.
We don't need the !s anymore now that we've got layers, and the .learn-link class isn't in the markup anymore (since it's only rendered in one place, so it made more sense to just move the classes there instead of having a separate @apply block).
| <img | ||
| src="/img/light.svg" | ||
| class="w-full block -translate-y-full dark:translate-y-0 transition-transform duration-200 easing-[0.86,0,0.07,1]" | ||
| alt="" |
There was a problem hiding this comment.
should there be some text in the alts? Does it matter?
There was a problem hiding this comment.
There's accessible text inside the button to describe its function already, so I don't think adding image descriptions here is probably beneficial. The empty alt just lets the images be ignored.
2 participants
Mostly just restores the default
:hoverbehavior, so that hover effects trigger on both fine and coarse pointer devices. Also includes a couple other small CSS fixups.