Skip to content

Marketing: find-in-page skips duplicated and hidden text - #2021

Merged
RhysSullivan merged 1 commit into
mainfrom
marketing-find-in-page
Sep 16, 2026
Merged

RhysSullivan merged 1 commit into
mainfrom
marketing-find-in-page

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

Cmd-F on the homepage matched every testimonial twice (the marquee's loop copy) and could land on hidden tab panes, hover panels, and the hidden download buttons.

  • The marquee's second copy, inactive tab panes, hover reveal panels, and hidden download buttons are now inert, so find-in-page, selection, tab focus, and screen readers skip them. They still render, so the loop and layout are unchanged.
  • Each testimonial now matches once.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 16, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing 84b79cf Commit Preview URL

Branch Preview URL
Sep 16 2026, 04:13 AM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 16, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud 84b79cf Sep 16 2026, 04:14 AM

@github-actions

github-actions Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@pkg-pr-new

pkg-pr-new Bot commented Sep 16, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@2021

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@2021

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@2021

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@2021

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@2021

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@2021

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@2021

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@2021

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@2021

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@2021

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@2021

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@2021

executor

npm i https://pkg.pr.new/executor@2021

commit: 84b79cf

@RhysSullivan
RhysSullivan merged commit f60ae01 into main Sep 16, 2026
44 checks passed
@RhysSullivan
RhysSullivan deleted the marketing-find-in-page branch September 16, 2026 04:15
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