-
Notifications
You must be signed in to change notification settings - Fork 4
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
Refacto menu + mod-vertical + scrollBox #2565
Conversation
3a83fc1
to
426ff04
Compare
3d3f58b
to
79bd4d3
Compare
<span class="menu-list-item-action is-disabled"> | ||
Menu 3 | ||
</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.
Just out of curiosity: Why not only removing the href
attribute? So that we don’t inherit some styles?
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 haven't even tried it, actually... due to ignorance.
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'm afraid including anchor tags without the href
attribute in the docs will be confusing for some users.
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 can be documented if required. Both work for me.
Description
a
are now withouthref
.mod-S
is now automatically applied to any embedded digital badges.mod-vertical
has been added..scrollBox
) now displays a scrollbar and shadows when necessary.