Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions .claude/work/010-card-dual-links/brief.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

Update every project card in `index.html` to show two links: a live site link (GitHub Pages URL) and a GitHub repository link. Cards with no GitHub Pages URL show only the GitHub repository link.

- Status: done
- Status: archived
- Branch: feat/card-dual-links
- Priority: 8
- Blockers: None
Expand All @@ -21,15 +21,15 @@ Low. HTML-only change. Rollback: revert before merge.

## Definition of done

- [ ] Every card h2 is plain text (not a link)
- [ ] Every card has a `.card-links` section after the description paragraph
- [ ] Cards with a Pages URL show a "Live site" link and a "GitHub" link
- [ ] Cards without a Pages URL show only a "GitHub" link
- [ ] All links have descriptive `aria-label` attributes giving the project name in context
- [ ] External links (GitHub repo, and any Pages URL not on projects.timdixon.net) carry `target="_blank" rel="noopener noreferrer"` and a visually-hidden "(opens in new window)" span
- [ ] `.card-links` CSS added to the inline style block
- [ ] WCAG 2.2 AAA passes with axe-core and Pa11y
- [ ] Carol signs off
- [x] Every card h2 is plain text (not a link)
- [x] Every card has a `.card-links` section after the description paragraph
- [x] Cards with a Pages URL show a "Live site" link and a "GitHub" link
- [x] Cards without a Pages URL show only a "GitHub" link
- [x] All links have descriptive `aria-label` attributes giving the project name in context
- [x] External links (GitHub repo, and any Pages URL not on projects.timdixon.net) carry `target="_blank" rel="noopener noreferrer"` and a visually-hidden "(opens in new window)" span
- [x] `.card-links` CSS added to the inline style block
- [x] WCAG 2.2 AAA passes with axe-core and Pa11y
- [x] Carol signs off

## Pre-approved GitHub actions

Expand Down
4 changes: 4 additions & 0 deletions .claude/work/010-card-dual-links/log.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,7 @@ Zero axe-core violations across all four themes. Pa11y zero errors. All 7 CI che

Token usage: Sean 41,042. Carol 52,100.
- [2026-06-09 10:17:46] subagent completed

## [2026-07-25] archive | Confirmed live and closed out

Verified PR #18 merged to main (2026-06-09) and dual-link markup present in the live `index.html` (`.card-links`, "Live site"/"GitHub" links, visually-hidden "opens in new window" spans). All definition-of-done items checked off. Status set to archived.
72 changes: 72 additions & 0 deletions .claude/work/011-emotion-wheel-card/brief.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
# Brief: 011-emotion-wheel-card

## Summary

Add a project card for Emotion Wheel to `index.html`, using the same dual-link card pattern (live site + GitHub repo) established in work folder 010.

- Status: active
- Branch: feat/emotion-wheel-card
- Mockup mode: D (no mockup — reuses the existing established card pattern)
- Priority: 8
- Blockers: None

## Requirements

No separate requirements document. Per Tim's direction (Q1 option B), this reuses the card pattern already implemented and tested in work folder 010-card-dual-links. Sean builds directly from that pattern; no Tad dispatch for this work.

Card data:

| Field | Value |
|---|---|
| Project name | Emotion Wheel |
| Description | Accessible emotion wheel selector with a local-only browser log (from repo description; Sean may lightly adapt wording to match the site's card copy style, consistent with other cards) |
| Live site URL | `https://emotionwheel.timdixon.net/` (confirmed live, HTTP 200) |
| GitHub repo URL | `https://github.com/timdixon82/Emotion-Wheel` |

## Routing plan

Sean builds the card on a branch, following the exact `.card-links` markup, ARIA labelling, and external-link pattern used for the other ten cards. Carol tests (functional and accessibility passes in parallel). Sonja reviews and takes to Tim for merge approval.

## Out of scope

- Any changes to CSS tokens, fonts, or design system files beyond what the existing `.card-links` block already provides
- Adding, removing, or reordering any other project card
- Changes to any file other than `index.html`
- Any new requirements-gathering pass by Tad

## Risk and rollback

Risk: the new card's markup could diverge from the established pattern (missing aria-label, missing "opens in new window" span, wrong link order) and fail accessibility checks.

Rollback: revert the branch commit before merge; no production impact until merged.

## Definition of done

- [ ] New card added to `index.html` with plain-text `<h2>` (not a link)
- [ ] Card has a `.card-links` section after the description paragraph, matching the existing pattern exactly
- [ ] Card shows both a "Live site" link (`https://emotionwheel.timdixon.net/`) and a "GitHub" link (`https://github.com/timdixon82/Emotion-Wheel`)
- [ ] Both links have descriptive `aria-label` attributes naming "Emotion Wheel" in context
- [ ] Both links carry `target="_blank" rel="noopener noreferrer"` and a visually-hidden "(opens in new window)" span
- [ ] No changes made to any file other than `index.html`
- [ ] WCAG 2.2 AAA passes with axe-core and Pa11y
- [ ] Carol signs off on functional and accessibility passes

## Approved GitHub actions

- [x] Create a branch
- [x] Commit to a branch
- [x] Push a branch other than the main branch
- [x] Open a pull request
- [x] Comment on a pull request or an issue
- [x] Create an issue

## Not pre-approved

These always pause for Tim, whatever is ticked above:

- Merging to the main branch. This always needs Tim's express approval at the time.
- Publishing to a blog or a social media account.

## Never allowed

The hard deny-list from `CLAUDE.md`. These are refused outright, whatever a brief says: force-push, branch deletion, history rewrite, repository deletion, repository visibility change, branch-protection edits, collaborator changes, release deletion, and disabling secret or code scanning.
9 changes: 9 additions & 0 deletions .claude/work/011-emotion-wheel-card/log.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
## [2026-07-25] open | Add Emotion Wheel project card

Work folder 011 opened. Triage: small feature.

Found Emotion Wheel missing from index.html while reviewing the project page after closing 010-card-dual-links. Repo `timdixon82/Emotion-Wheel` exists; live site `https://emotionwheel.timdixon.net/` confirmed reachable (HTTP 200).

Tim answered Q1 with option B: skip a separate requirements pass, build directly from the established dual-link card pattern.

Dispatching Sean to implement, following the pattern from work folder 010.
18 changes: 18 additions & 0 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -284,6 +284,24 @@ <h2>Clock Practice</h2>
</a>
</div>
</li>
<li class="card">
<h2>Emotion Wheel</h2>
<p>An accessible emotion wheel selector that keeps a log of your choices in your browser only.</p>
<div class="card-links">
<a href="https://emotionwheel.timdixon.net/"
class="card-link card-link--live"
aria-label="Emotion Wheel — live site"
target="_blank" rel="noopener noreferrer">
Live site<span class="visually-hidden"> (opens in new window)</span>
</a>
<a href="https://github.com/timdixon82/Emotion-Wheel"
class="card-link card-link--github"
aria-label="Emotion Wheel on GitHub"
target="_blank" rel="noopener noreferrer">
GitHub<span class="visually-hidden"> (opens in new window)</span>
</a>
</div>
</li>
<li class="card">
<h2>Image Colour Contrast Checker</h2>
<p>A tool for checking colour contrast ratios within images.</p>
Expand Down