jfraeys/coaching-site
Watch
1
0
Fork
You've already forked coaching-site
 0
Code Issues 10 Pull requests 3 Projects Releases Packages Wiki Activity Actions

design: blue page headers on inner pages; version the service-worker cache #11

Open
jfraeys wants to merge 2 commits from design/inner-page-headers into main
pull from: design/inner-page-headers
merge into: jfraeys:main
Conversation 0 Commits 2 Files changed 3 +15 -18
jfraeys commented 2026-10-19 17:37:15 -04:00
Owner
Copy link

Inner pages had a flat cream header while the home page is blue.

  • Header: .page-header-section now uses the --hero-* gradient with a 3px --primary rule, in styles.css and the inlined critical CSS. Title and lead use --hero-text. The dark-mode override is dropped; the dark tokens supply the gradient. Token-only, so client re-skins carry through.
  • Service worker: cache names are versioned (CACHE_VERSION), so returning visitors drop their cached HTML on the next deploy instead of seeing the old page for up to 7 days. Bump it when a redesign ships.

Checked: hugo --minify builds and go test ./tests/site/... passes. Not checked visually (no screenshot tooling available).

🤖 Generated with Claude Code

https://claude.ai/code/session_01B1FuEx3w8ax6P8QvdxxADM

Inner pages had a flat cream header while the home page is blue. - **Header:** `.page-header-section` now uses the `--hero-*` gradient with a 3px `--primary` rule, in `styles.css` and the inlined critical CSS. Title and lead use `--hero-text`. The dark-mode override is dropped; the dark tokens supply the gradient. Token-only, so client re-skins carry through. - **Service worker:** cache names are versioned (`CACHE_VERSION`), so returning visitors drop their cached HTML on the next deploy instead of seeing the old page for up to 7 days. Bump it when a redesign ships. Checked: `hugo --minify` builds and `go test ./tests/site/...` passes. Not checked visually (no screenshot tooling available). 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01B1FuEx3w8ax6P8QvdxxADM
Inner pages used a flat cream header while the home page is blue. The page
header now uses the --hero-* gradient with a --primary rule, in both
styles.css and the inlined critical CSS; dark mode follows the dark tokens.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B1FuEx3w8ax6P8QvdxxADM
sw: version the cache names so a redesign flushes returning visitors
Some checks failed
CI / Test & build (pull_request) Successful in 5m6s
Details
CI / golangci-lint (pull_request) Successful in 2m35s
Details
CI / Build artifacts (pull_request) Successful in 1m43s
Details
CI / Staging deploy check (pull_request) Failing after 1s
Details
81c8983c90
 
The worker serves cached HTML stale-while-revalidate for up to 7 days and the
cache names never changed, so visitors kept the old page after a deploy.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01B1FuEx3w8ax6P8QvdxxADM
Some checks failed
CI / Test & build (pull_request) Successful in 5m6s
Details
CI / golangci-lint (pull_request) Successful in 2m35s
Details
CI / Build artifacts (pull_request) Successful in 1m43s
Details
CI / Staging deploy check (pull_request) Failing after 1s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin design/inner-page-headers:design/inner-page-headers
git switch design/inner-page-headers

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff design/inner-page-headers
git switch design/inner-page-headers
git rebase main
git switch main
git merge --ff-only design/inner-page-headers
git switch design/inner-page-headers
git rebase main
git switch main
git merge --no-ff design/inner-page-headers
git switch main
git merge --squash design/inner-page-headers
git switch main
git merge --ff-only design/inner-page-headers
git switch main
git merge design/inner-page-headers
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
jfraeys/coaching-site!11
No description provided.