As a user I want a help page on the site so I can learn how to use the archive and share the guidance with family members before a scanning session #70
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Background
Family members who scan documents need to know two things before they start: how to name their files so metadata is auto-filled, and how to use the collaboration features once documents are in the archive. A
/helppage on the site serves both needs — it's always up to date, can be opened on the phone during a scanning session, and can be printed or linked in an email instead of attaching a static document.User Journey
A family member is about to scan a box of letters. They receive a link to
/help(or find the "Help" link in the nav). The page is structured into short sections they can read top to bottom or jump to. It covers:The page is print-friendly: clean typography, no sidebars, a "Print" link at the top. Each section has a short anchor link so individual sections can be linked directly from the UI (e.g. the drop zone hint links to
#filename-conventions).E2E Scenarios
Implementation notes
.sveltefile (no CMS needed at this scale).?icon or a text link in the footer area of the header).Content outline
Related: #66 (drop zone links here), #67 (enrichment queue links here), #69 (filename parser links here)
UX Review — @leonievoss
Overall: well-scoped and independent of the other notification issues. Two things to nail before implementation.
🔴 High — Print stylesheet must be specified, not implied
The issue mentions "print-friendly" but leaves the stylesheet to the implementer. This will be forgotten. Specify it explicitly:
The last rule is important: print-out recipients who receive a physical copy should still be able to see anchor URLs.
🔴 High —
?icon nav link fails for seniorsThe implementation note suggests a
?icon or a text link "in the footer area of the header". An icon-only link violates the redundant cues principle for the 60+ audience — they may not associate?with Help at all.Use a visible text link: "Hilfe" with a small
?icon alongside it, not instead of it. Minimum touch target 44×44px. Low visual weight is fine (same style as the subtle action links already in the nav), but the word must be present.🟡 Medium — Section headings need explicit
idattributes called outThe spec implies anchor links like
#filename-conventionswill work, but doesn't state that each<h2>must carry a matchingid. Without this, the deep links from the drop zone (#66) and filename parser (#69) will silently land at the top of the page. Add to implementation notes:🟡 Medium — Exclude
/helpfrom Paraglide route generationThe spec says "German only is fine for v1". Add a note that the
/helproute must be excluded from Paraglide's i18n route generation so it doesn't appear as a missing translation and doesn't throw a build warning forenandes.✅ What's good
QA Review — @saraholt
Simple feature, but two things will silently break if not encoded as tests.
🟡 Section anchors must be verified by a test, not just specified
Leonie flagged that each
<h2>must carry a matchingidattribute, otherwise the deep links from #66 and #69 silently land at the top of the page. Encode this as an E2E test so it can't break invisibly:Adjust the slugs to match whatever
idvalues are used in the implementation — the test enforces the contract.🟡 Nav link reachability — add to E2E suite
The issue spec lists this as an E2E scenario. It should be written explicitly:
checkA11yon the help page is low-cost and will catch Leonie's concern about the?icon — if the nav link is icon-only without accessible text, axe will flag it.🟡 Print stylesheet — manual checklist in the PR
Automated print testing in Playwright exists but is fragile and not worth the setup overhead for a static content page. Flag it as a manual checklist item in the PR description:
This makes it a conscious sign-off, not an assumption.
🟡 Paraglide exclusion — add a build check
Leonie noted the
/helproute must be excluded from Paraglide's i18n route generation. Verify this doesn't regress:Add this as a step in the CI integration-tests job so a future Paraglide config change doesn't silently reintroduce the warning.
✅ What's solid