-
Notifications
You must be signed in to change notification settings - Fork 22.5k
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
Switch to <ol>
for breadcrumb
#37273
Conversation
The Breadcrumb Glossary page notes > _Note: While there's no dedicated semantic HTML element for breadcrumb navigation menus, the <ol> element is commonly used to represent their hierarchical structure._ I suggest following our own advice. From a semantically standpoint, it just makes to use an ordered list.
Can you make the corresponding CSS changes so that this modification works? |
Preview URLs Flaws (48)URL:
(comment last updated: 2024-12-20 17:11:48) |
Requested changes from #37273 to update corresponding CSS.
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.
Thank you.
Congratulations on your first merged MDN content PR.
Welcome to the team!
* Switch to `<ol>` for breadcrumb The Breadcrumb Glossary page notes > _Note: While there's no dedicated semantic HTML element for breadcrumb navigation menus, the <ol> element is commonly used to represent their hierarchical structure._ I suggest following our own advice. From a semantically standpoint, it just makes to use an ordered list. * Update corresponding CSS Requested changes from mdn#37273 to update corresponding CSS.
The Breadcrumb Glossary page notes
Description
I suggest following our own advice. From a semantically standpoint, it just makes to use an ordered list. This tiny PR replaces the
<ul>
to a<ol>
.Motivation
From a semantic point of view, it makes sense to switch to an ordered list.