Skip to content

Commit b7601f4

Browse files
TMHSDigitalclaude
andcommitted
docs: record the Search button fix in the ledger
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wc67NX9MM964uaVGn4bpZJ
1 parent 4c4a111 commit b7601f4

1 file changed

Lines changed: 20 additions & 1 deletion

File tree

‎docs/review-ledger.md‎

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,24 @@ Newest section at the top. One section per review run. This is the record of wha
44
run measured, what it changed, what it deliberately did not change, and what the next
55
run should not spend time re-deciding.
66

7+
## 2026-09-23 (Search button)
8+
9+
The previous section's next action: make the floating button land where it focuses. One
10+
commit, `4c4a111`.
11+
12+
- The button said "Top", scrolled to the page header, then focused the search box about
13+
1,600px lower. It is now **"Search"**: it scrolls to the catalog section and focuses the
14+
box with `preventScroll`, so there is one scroll and the focus is on screen.
15+
- Checked at 1280x900 with and without reduced motion, 375x800 and 800x400: search box on
16+
screen and focused every time, and typing filters immediately ("kenney", 54 of 305).
17+
- Not changed: the button stays visible while the search box is on screen, because it
18+
shows by scroll depth (600px) and the catalog starts below that. Cosmetic.
19+
- Next highest-value action: **settle the five plural subcategory pairs**
20+
(character/characters, environment/environments, interior/interiors, tileset/tilesets,
21+
vector/vectors) by majority use, then let V13 fold a trailing "s" so a new variant
22+
cannot return. Check the check against the whole catalog before trusting it: plural
23+
folding is where false positives would come from.
24+
725
## 2026-09-23 (after the second full run)
826

927
The second full run's next action: the sticky filter panel on phones. One commit
@@ -22,7 +40,8 @@ The second full run's next action: the sticky filter panel on phones. One commit
2240
- **Found, not fixed (pre-existing, same on the live site):** on desktop, Top scrolls to
2341
the page header but focuses the search box about 1,600px further down, off screen. A
2442
keyboard user who then types is typing into a field they cannot see.
25-
- Next highest-value action: **make Top land where it focuses.** Either scroll to the
43+
- Next highest-value action: ~~**make Top land where it focuses.**~~ Done in the Search
44+
button section above. Either scroll to the
2645
catalog controls rather than the page top, or stop moving focus. A few lines in
2746
`app.js`, and it is the last interaction defect on record. After that, the plural
2847
subcategory convention (five pairs) from the second full run.

0 commit comments

Comments
 (0)