Refactor AI section headers and unify report layouts #1262
Conversation
This change simplifies the header logic across the AI section by removing "magic" front matter variables (top_header, hide_title) in favor of standard Hugo variables (.Title, .Parent.Title) and explicit layouts. The previous implementation relied on complex conditional logic and custom front matter to manage header visibility and hierarchy. This was fragile and hard to maintain. By standardizing on core Hugo features, we make the templates more readable and predictable. ## Key Changes ### Layouts * `layouts/ai/single.html`: Refactored to standard H1 titles. Added logic to display .Parent.Title as an "eyebrow" for context (e.g., "AI Research Insights"). * `layouts/ai/report.html`: Created a new layout specifically for Report and Errata pages. This enforces an H2 styling for the page title, distinguishing reports from standard articles. * `layouts/ai/overview.html`: Updated to use .Title instead of top_header. ### Content * Removed `top_header` and `hide_title` from all AI content files. * Applied new `report` layout to the Capabilities Model and Gen AI Report pages. * Restructured gen-ai-report to match the capabilities-model pattern (moved content to `report/index.md`) to support the new hierarchy. * Renamed titles in Report sections to align with the new visual hierarchy requirements. ### Redirects Added redirects to support the gen-ai-report restructuring. ### Tests Updated playwright tests to reflect the new structure and headings ### Documentation Updated `styleguide.md` to discourage the use of magic front matter variables. Fixes #1223
|
Visit the preview URL for this PR (updated for commit 9a0ff8d): https://doradotdev--pr1262-drafts-off-xul1xe3v.web.app (expires Sun, 18 Jan 2026 19:08:56 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7ad2b3cf9cceb558b493931176f998ae46924361 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request is a great refactoring effort that simplifies the header logic for the AI section by removing custom front matter variables and unifying layouts. The changes make the templates more readable and maintainable, which is a significant improvement. The updates to content, redirects, and tests are all consistent with this goal.
I've added a suggestion on the new gen-ai-report page to improve consistency with the capabilities-model pattern by using relative paths for resources.
Overall, this is a solid and valuable change. Also, I noticed a distinct lack of coffee-related humor in these changes, so here's one to percolate on: Why did the coffee file a police report? It got mugged!
This change simplifies the header logic across the AI section by removing "magic" front matter variables (top_header, hide_title) in favor of standard Hugo variables (.Title, .Parent.Title) and explicit layouts.
The previous implementation relied on complex conditional logic and custom front matter to manage header visibility and hierarchy. This was fragile and hard to maintain. By standardizing on core Hugo features, we make the templates more readable and predictable.
Preview URLs
Key Changes
Layouts
layouts/ai/single.html: Refactored to standard H1 titles. Added logic to display .Parent.Title as an "eyebrow" for context (e.g., "AI Research Insights").layouts/ai/report.html: Created a new layout specifically for Report and Errata pages. This enforces an H2 styling for the page title, distinguishing reports from standard articles.layouts/ai/overview.html: Updated to use .Title instead of top_header.Content
top_headerandhide_titlefrom all AI content files.reportlayout to the Capabilities Model and Gen AI Report pages.report/index.md) to support the new hierarchy.Redirects
Added redirects to support the gen-ai-report restructuring.
Tests
Updated playwright tests to reflect the new structure and headings
Documentation
Updated
styleguide.mdto discourage the use of magic front matter variables.Fixes #1223