- 
                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: