Skip to content

Fix copy controls and add field exports - #34

Merged
erinepshovel-code merged 1 commit into
mainfrom
agent/fix-copy-control-deployment
Aug 3, 2026
Merged

erinepshovel-code merged 1 commit into
mainfrom
agent/fix-copy-control-deployment

Conversation

@erinepshovel-code

@erinepshovel-code erinepshovel-code commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

What changed

Repairs the deployment gate for the copy-control feature and extends every ordinary substantive field with one compact action group:

  • Copy copies that field’s visible text.
  • .md downloads the field as a title-based Markdown file.
  • PDF opens a print-ready version so the browser can save that exact field as a PDF.

Linked cards retain sibling controls, avoiding nested interactive elements. Four Cuts uses the same controls.

Root cause

PR #33 added the copy script to both the public splash and the emergency fallback, while the established contracts require the fallback to contain no script. The splash contract was also still asserting no script, so the required build stopped before Pages could publish the merged feature.

Validation

  • npm test — 44/44 passed
  • JavaScript syntax checks — passed
  • git diff --check — passed

hmmm

The emergency fallback deliberately remains copy-control-free because it must remain usable with no JavaScript or external dependencies. The local rendered-site build could not complete after an external textbook-refresh timeout; CI is the authoritative full build and browser gate.

@erinepshovel-code
erinepshovel-code marked this pull request as ready for review August 3, 2026 06:28
@erinepshovel-code
erinepshovel-code merged commit a4c507f into main Aug 3, 2026
5 checks passed
@erinepshovel-code erinepshovel-code changed the title Fix copy control deployment checks Fix copy controls and add field exports Aug 3, 2026
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