Skip to content

Ayobami h canvas playback toggle - #32

Merged
eddiejaoude merged 4 commits into
mainfrom
AyobamiH-canvas-playback-toggle
Aug 15, 2026
Merged

eddiejaoude merged 4 commits into
mainfrom
AyobamiH-canvas-playback-toggle

Conversation

@eddiejaoude

Copy link
Copy Markdown
Owner

Description

Related issue

Closes #

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Refactor / maintenance
  • Other (please describe)

How has this been tested?

Screenshots

Checklist

  • My code follows the style of this project
  • I have tested my changes locally
  • I have updated documentation where needed
  • My pull request targets the main branch
  • I have read the Contributing guide

AyobamiH and others added 4 commits August 10, 2026 07:39
PR #26 was authored against the DOM PreviewCanvas renderer and merged
after the WebGL-only consolidation deleted it, leaving App.tsx importing
a missing module (build broke on TS2307) and wrapping the toggle surface
around the retired renderer instead of WebGLScene.

- App: drop the dead PreviewCanvas import; wrap PreviewPlaybackSurface
  around the live WebGLScene + BrandOverlay.
- e2e/app: pausePreview now pauses idempotently via the "Preview
  playback" button's aria-pressed — clicking the surface now toggles
  playback, so the old focus-nudge click would have un-paused instead.
- e2e/playback: add a test asserting a surface click freezes/resumes the
  real clock (not just aria-pressed); extend the R-restart poll timeout
  so the starved-clock case under full-suite parallelism isn't flaky.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-reel Ready Ready Preview Aug 15, 2026 8:22am

Request Review

@eddiejaoude
eddiejaoude merged commit ba0034b into main Aug 15, 2026
3 of 4 checks passed
@eddiejaoude
eddiejaoude deleted the AyobamiH-canvas-playback-toggle branch August 15, 2026 08:22

This branch was successfully deployed

1 active deployment
Preview — 831ff4b9 Deployed Aug 15, 2026 by vercel[bot]
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.

2 participants