AI Cookbook: match site breadcrumbs, relocate GitHub link - #5115
Merged
Conversation
AI Cookbook recipe pages bypass the default DocItem layout, so they never got the standard Home / section / page breadcrumb the rest of the site shows. Add a local breadcrumb built from the same global Infima classes and the theme's reusable Home crumb, and drop the now-redundant "Back to Cookbook" link. Move "Open in GitHub" out of the top action row into the recipe body, just below "Last updated," as a single link reading "View this recipe's code on GitHub." The link no longer renders at all when a recipe has no source frontmatter, instead of showing a disabled, dimmed dead end.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
📖 Docs PR preview linksThis PR does not change any pages in /docs. If you make updates, links to the modified pages will appear here. |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds site-consistent breadcrumbs to AI Cookbook recipes and improves source-link placement and behavior.
Changes:
- Adds Home / AI Cookbook / recipe breadcrumbs.
- Moves the GitHub source link below update metadata.
- Hides the source link when no URL exists and updates related styling.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
src/components/Cookbook/DocItem/CookbookDocItem.tsx |
Implements breadcrumbs and conditional GitHub linking. |
src/components/Cookbook/DocItem/CookbookDocItem.module.css |
Styles breadcrumbs and the relocated GitHub section. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Duncanma
marked this pull request as ready for review
August 14, 2026 18:04
Contributor
Author
Duncanma
enabled auto-merge (squash)
August 14, 2026 18:05
lennessyy
approved these changes
Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/ai-cookbook/*recipe pages matching the rest of the site (Home / AI Cookbook / page title), since these pages use a fully customdocItemComponentthat bypassed the default breadcrumb entirely.sourcefrontmatter, instead of showing a disabled, dimmed dead link.Test plan
yarn buildpasses (ran via pre-commit hook)/, AI Cookbook →/ai-cookbook, active crumb non-link)sourceURL and opens in a new tabsourceis absent (tested by temporarily removing it from a recipe's frontmatter)┆Attachments: EDU-6959 AI Cookbook: match site breadcrumbs, relocate GitHub link