Skip to content

feat(scratchnode): cohesive page-load entrance (the one real visual-pass gap)#462

Merged
HomenShum merged 1 commit into
mainfrom
feat/scratchnode-visual-delight
Jun 2, 2026
Merged

feat(scratchnode): cohesive page-load entrance (the one real visual-pass gap)#462
HomenShum merged 1 commit into
mainfrom
feat/scratchnode-visual-delight

Conversation

@HomenShum

Copy link
Copy Markdown
Owner

The visual delight pass is ~90% already on main — motion tokens, composer/private-mode polish, and Memory Wall tactile (rotate/pop-in/hover-lift) all exist (likely from the autonomous loop). The one structural gap a live audit found: the event room pops into final position with no orchestrated entrance.

This adds a transform-only settle on the main content (the feed's feedRowIn plays on top). Transform-only by design — no opacity in the keyframe — so a throttled/unsupported animation leaves content fully visible, never hidden. Landing excluded; reduced-motion disables it.

Verified live (worktree vs prod Convex): main.m opacity stays 1, content renders, entrance applies.

🤖 Generated with Claude Code

The home-v5 motion system (--motion-fast/med/slow, --ease-out/spring, glow
tokens), composer + private-mode polish (focus sheen, privateSeal, border glow,
dot pulse), and Memory Wall tactile (per-note rotate, snNotePop, hover-lift,
wallLaneIn) are ALREADY on main. The one structural gap a visual audit surfaced:
the event room pops into final position on first paint — no orchestrated
entrance.

This adds a transform-only "settle" on the main content so the room assembles
cohesively; the feed's per-row feedRowIn plays on top for a layered reveal.

TRANSFORM-ONLY by design: no opacity in the keyframe, so if the animation is
throttled (background tab) or unsupported, content stays fully visible (merely
offset a few px) — never hidden. Landing mode is excluded (it has its own
entrance); prefers-reduced-motion disables it.

Verified live (worktree served vs prod Convex demo room): main.m opacity stays
1 at all times, content renders (height 4047, 13 feed rows), entrance applies.

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

vercel Bot commented Jun 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
nodebench-ai Ready Ready Preview, Comment Jun 2, 2026 7:33am

Request Review

@augmentcode

augmentcode Bot commented Jun 2, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: Adds a transform-only first-paint entrance animation to the non-landing event-room main column so the page “settles” into place instead of popping.
Notes: Intended to remain fully visible (no opacity keyframes) and to respect reduced-motion preferences.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread public/proto/home-v5.html
top for a composed, layered reveal. Landing mode has its own entrance. */
@keyframes pageEnter { from { transform: translateY(10px); } to { transform: none; } }
body:not([data-page-mode="landing"]) main.m { animation: pageEnter var(--motion-slow) var(--ease-out) both; }
@media (prefers-reduced-motion: reduce) { main.m { animation: none; } }

@augmentcode augmentcode Bot Jun 2, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@media (prefers-reduced-motion: reduce) { main.m { animation: none; } } may not actually disable the entrance animation because the animated rule (body:not([data-page-mode="landing"]) main.m) has higher selector specificity, so it can win in the cascade even under reduced motion.

Severity: medium

Other Locations
  • public/proto/home-v5.html:173

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@HomenShum HomenShum merged commit 0b55980 into main Jun 2, 2026
16 checks passed
@HomenShum HomenShum deleted the feat/scratchnode-visual-delight branch June 2, 2026 07:44
@github-actions

github-actions Bot commented Jun 2, 2026

Copy link
Copy Markdown

Demo: walkthrough of the surfaces this PR changed is available as a workflow artifact (pr-demo-462) at https://github.com/HomenShum/nodebench-ai/actions/runs/26805895784

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