Skip to content

docs(specs): spec 061 restates 048 AC-1.8 at a phone's width (#288) - #292

Merged
plainprogrammer merged 17 commits into
mainfrom
061-place-line-wrap-at-phone-width
Nov 10, 2026
Merged

plainprogrammer merged 17 commits into
mainfrom
061-place-line-wrap-at-phone-width

Conversation

@plainprogrammer

Copy link
Copy Markdown
Owner

Summary

Closes #288 (the cause was found in #271, fixed by #272).

Spec 048 AC-1.8 said a five-character set code wraps the table row's phone line at 360px. That held only at a 345px content width (a 360px frame less a classic scrollbar). A real phone draws overlay scrollbars, has 360px of content, and fits that line with about 8px to spare; the example passed by about 7px. The CSS is right and does not change; the criterion and its example were wrong.

New spec 061 (supersedes spec 048 AC-1.8, recorded in both directions in the specs index; spec 048 is not edited) restates it, as the maintainer ruled (option a):

  • AC-1.1 / AC-1.2: a no-location lot of PLST · PCY-132 · Etched · NM · EN (184px) wraps the phone line at 360px, no sideways scroll, no Location column, no place; the example asserts the unwrapped width exceeds room + scrollbar width by at least 20px, read from the frame, so it holds with classic or overlay scrollbars (margin 23.15px in both).
  • AC-1.3: spec 048 AC-1.7 and its 360px and 390px examples stand, unchanged.
  • AC-1.4: the example was also run once with overlay scrollbars (temporary, reverted edit of the pin): scrollbar 0, room 161, unwrapped 184.15, margin 23.15, passed; recorded in verification.md.
  • Documents: the area doc's place line bullet, Sources and issues list; the specs index; a lesson.

No stylesheet, view, helper, model, controller, migration or export change, so no screenshots (the maintainer, 2026-11-10).

Reviews

  • Spec: two read-only Opus rounds (round 1: 1 blocking, 2 major, 7 minor, all taken; round 2: APPROVE, 4 minor, taken).
  • Plan: one read-only Opus round, APPROVE with fixes (1 major, 7 minor, all taken).
  • Implementation review (sdd-review Mode B, Opus, read-only): SPEC-ALIGNED, 3 minor, all taken; recorded in docs/specs/061-place-line-wrap-at-phone-width/verification.md.

Test plan

  • bin/rspec spec/system/collection_row_place_spec.rb: 18 examples, 0 failures (the red run on the old PMH3A line failed with margin -7.6: unwrapped 153.4, room 146, scrollbar 15)
  • Overlay-scrollbar run recorded (AC-1.4)
  • Pre-push bin/ci passed locally
  • GitHub CI green

🤖 Generated with Claude Code

plainprogrammer and others added 17 commits November 10, 2026 11:40
…#288)

Restates spec 048 AC-1.8, which held only at a 345px content width (a classic scrollbar). The maintainer ruled option (a): a PLST, PCY-132, Etched line with a measured 20px margin.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…dth (#288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…width, red (spec 061, #288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…line (spec 061 AC-1.1, AC-1.2, #288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
#288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…a doc sentence (#288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ture its lesson (#288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… (spec 027 D4)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…#288)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@plainprogrammer plainprogrammer self-assigned this Nov 10, 2026
@plainprogrammer
plainprogrammer marked this pull request as ready for review November 10, 2026 17:35
@plainprogrammer
plainprogrammer merged commit 33d6af8 into main Nov 10, 2026
9 checks passed
@plainprogrammer
plainprogrammer deleted the 061-place-line-wrap-at-phone-width branch November 10, 2026 17:36
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.

Restate spec 048 AC-1.8: the 360px place-line wrap only holds with a classic scrollbar

1 participant