Skip to content

Simplify the Way map to title-and-text disclosures - #28

Merged
erinepshovel-code merged 2 commits into
mainfrom
agent/simplify-way-disclosures
Jul 31, 2026
Merged

erinepshovel-code merged 2 commits into
mainfrom
agent/simplify-way-disclosures

Conversation

@erinepshovel-code

@erinepshovel-code erinepshovel-code commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

What changed

  • replaces the section wrappers and duplicated section headings with one continuous canon outline
  • makes every row a native title-only <details>/<summary> disclosure
  • renders only the exact generated unit.content beneath the selected title
  • removes line/type metadata, canon badges, open-state labels, action buttons, nested scroll regions, and row-level links
  • preserves canonical order and existing level-based indentation

Root cause

Merged PR #27 made rows expandable, but retained the prior page hierarchy inside each row and added another action interface beneath the text. That made the map larger and more complicated than the requested title → text interaction.

User impact

The /way/ page is now one compact reading list. Selecting a title opens its canon text in place; selecting it again closes the text. The interaction is native HTML and requires no JavaScript.

Validation

  • GitHub Pull Request workflow Fix SITREP browser checks for supported fallback data #67 passed with live canon, repository, textbook, and works sources
  • the live-source build validated all 29 canon units and the generated outline
  • 31 repository/static tests passed
  • the template contract proves every disclosure renders exact generated unit.content
  • Chromium verifies closed → open → visible text, no row links, and an unchanged /way/ URL
  • /way/ automated accessibility checks passed
  • mobile open-state render inspected at 393 × 851
  • branch is two focused commits ahead of current main, zero behind, with exactly five intended files
  • checked against skill-lib the-interdependency, canon, and skill-usage: exact canon remains source-derived; no new module or doctrine was introduced

hmmm

No validation failure remains. The public site will retain PR #27's larger interface until this draft is reviewed, merged, and deployed.

@erinepshovel-code
erinepshovel-code marked this pull request as ready for review July 31, 2026 13:20
@erinepshovel-code
erinepshovel-code merged commit 94d5a46 into main Jul 31, 2026
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