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
Code links #209
Comments
Definitely a bug if it's colored with the style disabled, looking into it. |
Well that was easy, but may affect other code tags, I must've put it there for a reason, but I don't remember right now. |
Hmm, underline still missing. |
Strange that it doesn't underline on hover, given that we don't touch @Mottie could you have a look in Chrome's DevTools? Firefox still shows the underline in computed styles, but doesn't render it. |
Hmm, I'm not seeing where the text-decoration is being changed, when I click on the a:-webkit-any-link {
color: -webkit-link;
text-decoration: underline;
cursor: auto;
} If I add this css (even without the a:hover code {
text-decoration: underline;
} But my question still is about the text color. There isn't any syntax highlighting, so should we leave the text off-white or the base color? |
Ok, sounds reasonable 😸 |
This underline thing smells like a Firefox bug to me, or a spec misbehaviour in Chrome, or both. I'm not sure if that property shall be inherited by child elements att all. Setting a:hover code {text-decoration: underline !important} does fix it, but I'm afraid this might have side-effects on other parts of the site. Hmm. |
I have an idea on how to fix this without causing too much side-effects. |
Please check if you still have the underline on hover in Chrome. |
Sorry, I've been doing a lot of running around today... yeah that is working just fine :P |
I was posting some code which linked back to documentation and noticed that the link was not being styled; there is no indication that it is a link except for hovering over it:
linked code
I'm not sure if making it match the base color would be a good idea, but maybe we could at least underline it? What do you guys think?
The text was updated successfully, but these errors were encountered: