-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Link colors changed and underlined recent change to tsugi-static #158
Comments
Hi @csev, this is probably because of a change in approach we took for dark mode since sometimes the coloring of links in dark mode made them a little hard to see (if the primary color ends up a bit dark). I ended up applying general styling to links in tsugi-static to use var(--text-color) and to be underlined to guarantee accessibility regardless of the theme color, but now that you mention it I think I should have just used a different theme variable (--text-light). I can open a PR for tsugi-static that should fix that. |
Google doesn't underline their links and has different colors for light or dark. Seems nice. On dark they use Webaim suggests a possibly a blue compatible with both but no requirement to underline. I'm not sure I personally like the "sea of underlined links" style as it's called in the last comment but this page looks like it turned into. It's seems better used when shorter links are underlined in a body of text as on the Webaim page.
|
Hey @andrewniekamp - The changes you did to tsugi-static seems like something changed. Here is a before and after look at my lessons pages on my production servers. I also merged in all of your tsugi code - but this happened when I did a shift refresh on a server with already upgraded Tsugi and the shift-refresh grabbed a new static - so it is likely static and not the PHP code.
The text was updated successfully, but these errors were encountered: