-
Notifications
You must be signed in to change notification settings - Fork 25.6k
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
Use css padding for author urls to fix underline hover state #2472
Conversation
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.
Looks good, just one small naming change. Instead of .text
can we use .label
? It's slightly more semantic. With that change I can merge this in.
Makes sense. Pushed the fix. |
Looks good, thanks. |
@ali-farooq0 this bugged me for ages but I just today found the time to investigate this issue further. Now I found that you already created a PR. Thank you very much for that :) Can't wait for the next release. |
@d-Rickyy-b Anytime :). If you're using
|
…es#2472) * Use scss padding for sidebar author urls * rename .text to .label
…es#2472) * Use scss padding for sidebar author urls * rename .text to .label
…es#2472) * Use scss padding for sidebar author urls * rename .text to .label
…es#2472) * Use scss padding for sidebar author urls * rename .text to .label
This is a bug (style) fix.
Summary
Author profile page uses a literal
space character as padding between the social icon and the text. Which as a result causes the underline to show up on a blank space character on hover, causing it to look like the example below.
This PR wraps the text with a
span
block that can be used to customize spacing between the social icon and its relevant text, making it a breeze to customize as opposed to copyingauthor-profile.html
to your local repo and customizing it just to fix spacing, making upgrading the theme very difficult.About page - BUG
With fixed spacing