-
Notifications
You must be signed in to change notification settings - Fork 18
fix a11y breadcrumb #465
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
base: master
Are you sure you want to change the base?
fix a11y breadcrumb #465
Conversation
Reviewer's GuideIntroduces a Breadcrumb service to overhaul Yoast SEO breadcrumbs for accessibility, outputting them as an ordered list with separators inside the list items and disabling the external separator, and registers this service in the theme’s framework. Sequence diagram for Yoast breadcrumb rendering with new Breadcrumb servicesequenceDiagram
participant YoastSEO as Yoast SEO
participant Breadcrumb as Breadcrumb Service
participant Theme as Theme
YoastSEO->>Breadcrumb: apply 'wpseo_breadcrumb_output_wrapper' filter
YoastSEO->>Breadcrumb: apply 'wpseo_breadcrumb_single_link_wrapper' filter
YoastSEO->>Breadcrumb: apply 'wpseo_breadcrumb_single_link' filter
YoastSEO->>Breadcrumb: apply 'wpseo_breadcrumb_separator' filter
Breadcrumb-->>YoastSEO: returns modified wrappers and separator
YoastSEO-->>Theme: outputs accessible breadcrumb as <ol><li>...</li></ol> with separator inside <li>
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
@MarieComet va proposer une modification/correction |
@MarieComet peux-tu faire le fix stp ? Besoin de merger en vu d'un nouveau projet |
Fix du breadcrumb yoast pour ajouter une structure liste ordonnée au lieu des
<span>
+ inclure le séparateur à l'intérieur des<li>
car sinon il se trouve entre chaque<li>
On perd donc la main sur la configuration du séparateur en BO, mais pas gênant.
Summary by Sourcery
Introduce an accessibility-focused Breadcrumb service that replaces the default Yoast breadcrumb spans with semantic list markup and hides separators within list items.
New Features:
Enhancements:
) with list items (- )
- Include the separator inside each
- with aria-hidden attribute
- Remove the default external breadcrumb separator
Chores: