-
Notifications
You must be signed in to change notification settings - Fork 2k
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
#1160 add styling to popular tags #3460
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.
I like where this is heading - my only question is about the DOM changes that support this styling...
_includes/scripts.html
Outdated
<% _.each(popularTags, function(entry, key){ %> | ||
<li><a><%-entry.name%></a> (<%-entry.frequency%>)</li> | ||
<a title="Popular Tag: <%-entry.name%>" tabindex="0"><%-entry.name%> <span class="popular-tags__frequency">(<%-entry.frequency%>)</span></a> |
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.
🤔 Can we do these styling changes without losing the ul
/li
semantics of these elements?
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 comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
… from div restructure)
Checking the latest changes to the pull request... 352 projects without issues ✅Everything should be good to merge! |
255c99e
to
d6fad2b
Compare
@cweave Thanks |
#1160 add styling to popular tags
Light mode, python hover/active state:
Dark mode, python hove/active state:
iPhone 12/13 screensize (375px wide):
popular-tag
list to adiv
with links. Addstitles
and tabbing order to the links for a11y.container-border
color to rbg value.container-border-alt
color for darkmode