Fix video overlay click handling

[jnptzl]

Motivation

• The existing overlay prevented clicks from reaching the video element, blocking pause interactions and breaking the expected "click to play" UX.
• The change aims to reveal the player without leaving a hidden overlay that intercepts events and to restore the overlay when playback is paused.

Description

• Replace the static overlay with an attachOverlay helper that creates a .be-video-overlay only when absent and returns it for later control.
• Introduce an overlayRevealed flag to track whether the overlay has been revealed and should no longer block playback interactions.
• On overlay click the handler now calls video.play(), sets overlayRevealed = true, and removes the overlay with overlay.remove() instead of calling stopPropagation() or leaving a hidden element.
• Add a pause listener that resets overlayRevealed and re-attaches the overlay via attachOverlay() (unless the video ended), and update the play handler to use the overlayRevealed flag to prevent unwanted autoplay.

Testing

• No automated tests were run.

---

Codex Task

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e2b7b67937

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Stop overlay click from bubbling to page handlers

The click handler no longer calls stopPropagation(), so the overlay click now bubbles up to ancestor handlers. On pages like X/Twitter where the tweet container uses click handlers to open the tweet detail view, this means “click to play” can also navigate/open the tweet instead of just starting playback. This regression happens whenever the host page registers click handlers on ancestors of the video container, so consider restoring stopPropagation() (or otherwise preventing host clicks) to keep overlay clicks from triggering page-level navigation.

Useful? React with 👍 / 👎.