Skip to content

Reveal canon text when Way rows are opened - #27

Merged
erinepshovel-code merged 1 commit into
mainfrom
agent/way-inline-text-disclosure
Jul 31, 2026
Merged

erinepshovel-code merged 1 commit into
mainfrom
agent/way-inline-text-disclosure

Conversation

@erinepshovel-code

Copy link
Copy Markdown
Contributor

What changed

  • replaces each /way/ heading link with a native <details>/<summary> disclosure
  • reveals the exact generated unit.content inline when the row is tapped
  • keeps separate links to the Unit page, Article Lab, and exact Source/provenance page
  • preserves section, subheading, and nested-heading indentation
  • adds responsive mobile styling and open/closed state labels

Root cause

The canon-derived tree rendered each row only as a navigation link. A click left the tree for a companion page, so the map itself never showed the text the row represented.

User impact

Every row now works as a one-tap reading surface, including without JavaScript. Opening a row does not change the /way/ URL. Deeper interpretation and provenance paths remain explicit rather than being collapsed into the disclosure.

Validation

  • 31 repository/static tests passed
  • exact-text Chromium interaction test passed
  • /way/ automated accessibility check passed
  • mobile disclosure render inspected at 393 × 851
  • branch is one commit ahead of main, zero behind, with exactly five intended files

hmmm

The local production aggregate gate could not complete because external canon/repository/textbook requests entered fallback twice; the validator correctly rejected fallback production data. The checked-in recovery canon is also stale against current Article drafts. GitHub CI is therefore the authoritative live-source gate for this PR.

@erinepshovel-code
erinepshovel-code marked this pull request as ready for review July 31, 2026 11:36
@erinepshovel-code
erinepshovel-code merged commit eebf7b5 into main Jul 31, 2026
4 of 5 checks passed
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.

1 participant