Skip to content

docs(website): hide the film transcript from sighted visitors - #55

Merged
yannelli merged 1 commit into
mainfrom
fix/remove-film-transcript
Oct 31, 2026
Merged

yannelli merged 1 commit into
mainfrom
fix/remove-film-transcript

Conversation

@yannelli

Copy link
Copy Markdown
Owner

Removes the visible "Read the film transcript" disclosure under the homepage film.

Summary

  • The film text now sits in a visually hidden figcaption, so screen readers still get it. It has no links, so nothing invisible takes keyboard focus. The video's aria-describedby points to its first paragraph.
  • The text is updated for the current film: five console screens, and the optional AI-writing checks plus the no-rewrite note kept.
  • Tests, WEBSITE.md, DESIGN.md and the design sidecar are updated to match.

Checks

  • Build (21 pages), Astro check 0/0/0, 24 tests. Homepage screenshots at 1440 and 390 show nothing under the film.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://fix-remove-film-transcript.concise.chat, https://fix-remove-film-transcript-be-concise.ryanyannelli.workers.dev (commit db655b3)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://72b33457.concise.chat, https://72b33457-be-concise.ryanyannelli.workers.dev db655b3 2026-10-31T07:20:19.461Z Visit the dashboard ↗

@yannelli
yannelli merged commit b24522a into main Oct 31, 2026
4 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