Skip to content

Conversation

@nickdenardis
Copy link
Member

Clear up WCAG 2.2 A error

All Base sites have an WCAG 2.2 A error "Container element is empty" when a page doesn't have breadcrumbs visible.

Screenshot 2025-12-10 at 6 01 08 AM

Change

This updated code wraps the whole partial in a conditional to ensure the container and DOM elements are only displayed if there are breadcumbs.


Demo / Context

Before After
Screenshot 2025-12-10 at 5 57 28 AM Screenshot 2025-12-10 at 6 00 06 AM

Final Checklist

  • I have reviewed my code and it follows project standards
  • I have tested the changes locally
  • I have added or updated relevant documentation
  • I have added tests (if applicable)
  • I have communicated with the team about necessary post-deploy actions

@nickdenardis nickdenardis added this to the 8.16.0 Release milestone Dec 10, 2025
@nickdenardis nickdenardis self-assigned this Dec 10, 2025
@nickdenardis nickdenardis requested a review from a team as a code owner December 10, 2025 11:09
@coveralls
Copy link

Coverage Status

coverage: 99.083%. remained the same
when pulling da76b95 on feature/breadcrumbs
into abf2c1b on develop.

Copy link
Contributor

@KyleApsey KyleApsey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

@nickdenardis nickdenardis merged commit da76b95 into develop Dec 10, 2025
2 checks passed
@nickdenardis nickdenardis deleted the feature/breadcrumbs branch December 10, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants