Repository navigation
Polish the Inference Day page and promote the event on the homepage - #42
Merged
Merged
Conversation
Follow-up to the merged Inference Day page (modelplaneai#40), rebased onto the lint fix (modelplaneai#41): - Make the hero and closing-CTA contrail full-bleed, sized in vw so the wave spans the section at every width (the fan reaches the lower-left, the lines run off the top-right) instead of being cut off; soften it on phones for legibility. - Add a dismissible homepage announcement bar linking to /inference-day. It is rendered client-only (its visibility depends on localStorage and the date), so there is no effect-driven state and no hydration mismatch; it hides itself once the event has passed. - Add an Inference Day link to the main nav (next to Slack, cyan accent) using next/link, and give the landing page a two-tier header: the main site Nav plus a slim event sub-nav for the on-page jump links. This replaces the standalone EventNav. npm run lint, build and validate-posts all pass. Signed-off-by: Charlie Hinojosa <charlie@sightbox.me>
The full-bleed wave crossed the hero's body text, date and buttons, and its glowing nodes landed on readable copy (looked odd on desktop, cluttered on mobile). Lift the hero contrail above the body copy (top -33vw) so it frames the wordmark and the nodes fall in open space, leaving the intro, date and buttons on clean dark. On phones/small tablets (<=768px) the copy is full-width with nowhere for the wave to go, so hide the decorative art there for a clean, legible hero and CTA. Signed-off-by: Charlie Hinojosa <charlie@sightbox.me>
The lockup was absolutely positioned in the hero's bottom-right, floating over the wave art, so it did not place predictably across breakpoints. Move it into the hero content flow, directly under the buttons, and centre it (full-width on small screens) so it sits consistently on desktop, tablet and mobile. Signed-off-by: Charlie Hinojosa <charlie@sightbox.me>
The earlier passes lifted the wave above the copy, then dropped it below the whole copy block; neither matched the handoff design, where the wave sweeps behind a compact copy block in the upper-left. Match the design's proportions so the composition works as drawn: - Give the hero a roomy min-height (~800px at 1440) with tighter padding, so the copy sits compact in the upper-left and leaves open space below and to the right for the wave. - Keep the wave full-bleed but anchor it near the top (top -9.2vw), so it sweeps from the upper-right down to the lower-left with its glowing nodes falling in the open area, clear of the text. - Set the wordmark on one line (white-space: nowrap, 108px cap) and widen the intro to two lines, matching the design's headline and copy. - Tighten the date and action gaps to the design's rhythm. - On phones/small tablets (<=768px) the wave is still hidden; let the wordmark wrap and drop the desktop min-height for a clean, legible stacked hero. npm run lint, build and validate-posts all pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016QU1xKJitT9UNpCfjXjDxt Signed-off-by: Charlie Hinojosa <charlie@sightbox.me>
charliehinojosa
force-pushed
the
inference-day-polish
branch
from
October 20, 2026 18:51
138fa1f to
15ca15e
Compare
Centred, the lockup landed over the busiest part of the wave and got lost in the pattern. Left-align it so it sits under the copy and buttons on clean dark, consistent with the rest of the hero's left-aligned content at every breakpoint. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016QU1xKJitT9UNpCfjXjDxt Signed-off-by: Charlie Hinojosa <charlie@sightbox.me>
3 tasks done
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
Follow-up to the Inference Day page (#40), rebased onto the lint fix (#41). Three improvements:
vw(the asset is ~108.5vw wide at 1440) so the whole flight-path composition stays full-bleed at every width — the fan reaches the lower-left, the lines run off the top-right — matching the design. Softened on phones so full-width copy stays legible./inference-day(gradient wordmark, date tag, hook, Register pill). Rendered client-only (next/dynamic,ssr:false) since its visibility depends onlocalStorageand the date — no effect-driven state, no hydration mismatch. It hides itself the day after the event, so no manual removal.next/link, and gives the landing page a two-tier header — the main site<Nav />plus a slim event sub-nav for the on-page jump links (Register · Agenda · Speakers). Replaces the standaloneEventNav.Scope stays contained: new rules are under
.id-pageor the.announce/.nav-eventclasses; the sharedNavgains one link and is otherwise unchanged.Checklist
npm run lint,npm run buildandnpm run validate-postslocally — all pass (lint clean; the remaining<img>warnings are pre-existing and match the rest of the site).git commit -s(DCO).Generated by Claude Code