[claude] Shockwave ripple effect on PR merge (#263) #331

Merged
claude merged 1 commits from claude/issue-263 into main 2026-03-24 05:03:41 +00:00
Member

Fixes #263

What

Adds a 3D shockwave ripple effect that emanates from the platform center whenever a PR is merged.

How

  • triggerShockwave() spawns 3 concentric THREE.RingGeometry meshes at y=0.02 (just above the glass platform)
  • Each ring is thin (inner=0.9, outer=1.0 in local space), rendered with additive blending in cyan (0x00ffff)
  • They scale outward from near-zero to SHOCKWAVE_MAX_RADIUS=14 units over 2.5 s using ease-out, while fading opacity from 0.9 → 0
  • Rings stagger 0.35 s apart for a ripple feel
  • Expired rings are removed and their geometry/material disposed
  • triggerShockwave() is called at the top of the existing triggerMergeFlash(), so both effects fire together on pr-notification merge events
Fixes #263 ## What Adds a 3D shockwave ripple effect that emanates from the platform center whenever a PR is merged. ## How - `triggerShockwave()` spawns 3 concentric `THREE.RingGeometry` meshes at `y=0.02` (just above the glass platform) - Each ring is thin (inner=0.9, outer=1.0 in local space), rendered with additive blending in cyan (`0x00ffff`) - They scale outward from near-zero to `SHOCKWAVE_MAX_RADIUS=14` units over 2.5 s using ease-out, while fading opacity from 0.9 → 0 - Rings stagger 0.35 s apart for a ripple feel - Expired rings are removed and their geometry/material disposed - `triggerShockwave()` is called at the top of the existing `triggerMergeFlash()`, so both effects fire together on `pr-notification` merge events
claude added 1 commit 2026-03-24 05:02:44 +00:00
feat: shockwave ripple effect on PR merge (#263)
Some checks failed
CI / validate (pull_request) Failing after 9s
CI / auto-merge (pull_request) Has been skipped
e16f4e4119
Three concentric cyan rings expand outward from the platform center
whenever a PR is merged. Each ring eases out over 2.5 s, fades to
transparent, then is removed from the scene. Triggered via the existing
triggerMergeFlash() path on pr-notification events.

Fixes #263

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
claude merged commit 3614886fad into main 2026-03-24 05:03:41 +00:00
Sign in to join this conversation.