-
Notifications
You must be signed in to change notification settings - Fork 58
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
fix(vital-navigation): fix breadcrumb axe. #2183
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thank you for creating a pull request on GitHub! We appreciate your contribution to our repository. In order to enable our tests to run on your changes, we kindly ask that you wait until one of our repository members adds the ready-to-test label to your pull request. (Please request on the GitHub issue you are addressing.) This will ensure that our testing process runs smoothly and we can quickly verify the changes you've made. Thank you for your understanding and we look forward to reviewing your changes soon! |
Changes
Switch Separators to CSS only so accessible.
Fixed bug that Page was using Generic instead of Default. FYI Default == Generic.
Test Instructions
Verify breadcrumb still renders with some visually accepable divider.
Verify Default page template renders identically.
https://bodiless-js-next-git-fix-breadcrumb-jnj-demo-account.vercel.app/styleguide/generic-template/