Skip to content

fix(site): step arrows beside the count, Quickstart comments, one code highlighting - #560

Merged
aicayzer merged 2 commits into
mainfrom
fix/paged-steps-nav
Oct 21, 2026
Merged

aicayzer merged 2 commits into
mainfrom
fix/paged-steps-nav

Conversation

@aicayzer

Copy link
Copy Markdown
Contributor

Summary

On a paged tutorial, Previous and Next sat below a panel as tall as the longest step, so they were often off screen and easy to miss, and the Next button was a heavy filled block. The arrows now sit beside the step count above the step, where they stay in view, and the step ends with a quiet "Next: " link. The Quickstart's code blocks gain short comments, and one Markdown block on Sync a folder no longer fails hydration.

Changes

  • steps.tsx: Previous and Next as icon buttons beside the count; a text link to the next step after it; no reserved height, so a short step has no empty space below it. Hidden steps still use hidden="until-found".
  • app.css: the paged-steps styles to match; the bulky button styles are gone.
  • Quickstart: comments in the multi-line blocks (what each part does, what to replace); the key's output drops its long powers line so the key stays visible instead of folded; "Download it and put it on your path".
  • source.config.ts: preload the YAML and Markdown grammars. Shiki loads a language on first use and colours Markdown frontmatter with YAML only once YAML is loaded, so the server and browser coloured Sync a folder's Notes/Reading list.md block differently and React re-rendered the page. This also happens on the live site.
  • STYLE.md: the Steps paged row describes the arrows.

Testing

  • pnpm test (366), pnpm build, pnpm build:status and pnpm types:check pass.
  • In the browser: the Quickstart at desktop and phone width, stepping forward with the arrows; all four paged pages at phone width with no sideways scroll.
  • Every docs page (68, from llms.txt, API operations aside) loads with no page errors; before the fix, Sync a folder had a hydration error.

Notes for reviewers

Checks only: site layout and wording, checked in the browser.

Checklist

  • Follows STYLE.md
  • Builds and tests pass

@aicayzer aicayzer left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: checks only

Summary. Moves the paged-steps arrows beside the count, comments the Quickstart code and preloads the YAML and Markdown grammars so server and browser highlight alike; checked in the browser, and all 68 docs pages load with no page errors.


Checked by the orchestrating session on claude-opus-5-5. This review may contain mistakes.

@aicayzer
aicayzer merged commit 55c1ae5 into main Oct 21, 2026
6 checks passed
@aicayzer
aicayzer deleted the fix/paged-steps-nav branch October 21, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant