Going mobile#34
Merged
Merged
Conversation
Align the left of items in the site nav. Reduce the size of the anchor icons and align them to the top of the text.
Improve the TOC handling. Only h2s on standard pages in the main section. Deeper rendering now we have the space for it in the nav toc.
If a bale has no manual index, create an auto one.
Better Naviagation: Index pages now show cards for their child pages, making it easier to navigate large documentation sets.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR improves the mobile navigation experience and introduces “card summaries” for bale index pages, enabling prominent links (with blurbs) to child pages/bales and adding a last-updated footer.
Changes:
- Add card summary generation in the renderer and display them as an index card grid in-page plus a compact child link list in the TOC sidebar.
- Adjust TOC rendering and add an inline
[TOC]wrapper/label, plus new inline breadcrumbs on mobile. - Capture page last-modified time and render a “Last updated” line; update CSS/JS to increase header hit areas and tweak nav behaviors.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/toc.rs | Adds a Nodes::len() helper used by new TOC rendering logic. |
| src/render/layout/html.rs | Adds inline breadcrumbs, card rendering, last-updated footer, and refactors TOC rendering behavior. |
| src/render.rs | Introduces CardSummary, changes PageKind::Index to carry summaries, and generates synthetic index pages when missing. |
| src/doctree.rs | Tracks modified time per page and adds Page::synthetic() + first_paragraph() extraction for blurbs. |
| assets/style.css | Updates header button styling for mobile, adds card styles, breadcrumbs tweaks, and last-updated styling. |
| assets/dark.js | Applies shared header icon button styling and adjusts SVG sizing attributes. |
| AGENTS.md | Documents new TOC and auto-index behavior and adds test/preview guidance. |
| .gitignore | Ignores docs/examples/ scratch test pages. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Reduce the "glare" from the accent colour by dimming it a little on dark mode renders.
iwillspeak
commented
Mar 7, 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.
Improved Mobile use. Makes hit areas bigger, and tweaks some nav elements to make moving around simpler.
Introduces "Card Summaries" which link from a bale to the items within it.