-
Notifications
You must be signed in to change notification settings - Fork 40
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 breadcrumbs #136
base: main
Are you sure you want to change the base?
Fix breadcrumbs #136
Conversation
@Rick-Anderson @gewarren Please review, thanks! |
@raprice I'm not sure that this is working how you want it to work. For example compare these two pages: |
@gewarren You're absolutely right, something is funky. I will dig into this, putting back in draft until I get this rendering correctly. Thanks for catching it! |
dotnet/entityframework.apidocs |
@gewarren Please take a look - the breadcrumbs from this PR are now rendering correctly (Learn / .NET / Entity Framework / .NET API browser) https://review.learn.microsoft.com/en-us/dotnet/api/microsoft.entityframeworkcore?view=efcore-8.0&branch=pr-en-us-136 |
This PR brings breadcrumb implementation into alignment with platform architecture requirements. This PR is part of a previously announced batch of breadcrumb fixes across the Learn platform and will be auto-merged if there are no build warnings. This PR may include removing the “extend breadcrumb” feature from any docfx files that are still using it, fixing breadcrumb file references in the docfx file, and rewriting breadcrumb files to match the approved breadcrumb pattern for a given product’s documentation.