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
Make hyperlinks more visible when displayed in /wall #1907
Conversation
style/client.css
Outdated
@@ -1355,6 +1355,9 @@ p.or:after { | |||
padding: 0 4px 0 3px; | |||
font-style: normal; | |||
} | |||
.chat span a small { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
span
is probably not the correct match. Doesn't /wall
have a class on that span?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be next to the thing that makes the regular link text gray, too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see, it's right below this. This should be moved below .chat small
, rather than above. Also, the selector should be .chat .message-announce small
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, the contrast here is kind of poor. Probably the background on announcements should be darker.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should I make the background color of .message-announce
from battle-log.css darker ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can handle that, no worries.
Thanks! |
here's a small workaround regarding this : https://www.smogon.com/forums/threads/make-hyperlinks-more-visible-when-having-them-displayed-in-wall-or-announce.3693900/
(excuse the large amount of commits, I made a mess with my branches)