Skip to content

site: faster, clearer portfolio with real project schematics - #3

Merged
grimgrimberg merged 4 commits into
devfrom
claude/website-repo-review-0056ec
Nov 8, 2026
Merged

site: faster, clearer portfolio with real project schematics#3
grimgrimberg merged 4 commits into
devfrom
claude/website-repo-review-0056ec

Conversation

@grimgrimberg

@grimgrimberg grimgrimberg commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

The live site worked, but it undersold the engineering. The project cards were emoji on gradients. The scrolled nav let page text show through. Floating buttons covered card links. About 2 MB of three.js and Clippy loaded before anything else. The site had no 404 page, no link-preview image and no sitemap.

This PR fixes those things and keeps the personality.

What a visitor notices

  • Each project card now shows a line-art schematic of how the project actually works: the pure-pursuit path, the CW rendezvous hops, the UGV watchdog trace, the VGGT camera frustums.
  • Technical Expertise is now a set of spec cards, and each skill links to the project that proves it.
  • The professional copy is tighter and every claim matches the CV. "Led five student developers" now appears on the site.
  • Retro Classics drops from about 4,200 px to about 1,600 px on phones (a swipeable shelf). All the jokes are kept.
  • The nav is opaque once you scroll. The floating Guide / Goose / Command buttons are now compact icons on desktop. Brass, green and coral buttons pass AA contrast.

New: Fly101 project page

  • fly101.html documents the fly-brain robot arm project, with a rotating 3D view of real MaleCNS neuron positions and renders of the fly and SO-101 models.
  • Headline results come first; full results, baselines and open questions are in collapsible technical notes.
  • A new Connectome lab in Take the controls rotates the same data and switches brain regions on and off. It is anatomy only, and its data loads on request.
  • The Fly101 repo is private, so the page has no repo link. Models and data are credited on the page.

Under the hood

  • three.js loads when a lab gets near the screen or the page goes idle. Clippy loads after page load on desktop, and on phones only when someone taps Guide.
  • Keyboard and screen readers: the skip link and in-page anchors move focus and update the URL. The command palette keeps focus inside and returns it on close. The retro pop-ups are real dialogs. The role tabs support arrow keys. Reduced motion is respected site-wide.
  • Added 404.html, robots.txt, sitemap.xml, a 1200×630 link-preview card, JSON-LD for the Person, and an Apple touch icon. All pages now use the same LinkedIn URL.
  • about, projects, vision and thank-you are now noindex redirects into homepage sections. The old projects.html described work that never happened, and thank-you.html promised replies from a backend that doesn't exist.
  • Refreshed the repo snapshot: the FPV lab and five other public repos were missing. Removed the dead scripts.js and CV-gate code.
  • New CI workflow: CSS build check, lint, lab model tests and the Playwright smoke suite.

Tests
Static lint, npm run test:models, the hygiene suite (6 passed) and the smoke suite (13 passed, 17 skipped by design) all pass locally. They ran on Chromium 1248, because the pinned 1217 build was still downloading.

The reconstruction smoke test needed one fix. Its second width called goto with the same URL and hash, which is an in-page jump, not a reload. The old version only passed because the smooth scroll was cut short. It now goes to about:blank first.

Not in this PR (waiting on Yuval)

  • Deleting the 32 MB of unreferenced original photos.
  • Generating proper thumbnails.
  • Fixing the photo page (its gallery arrows are invisible and can't be reached with the keyboard).

Created with Claude Opus 5.5 in Claude Code.

@grimgrimberg @claude
site: faster, clearer portfolio with real project schematics
d48be5e 
- Lazy-load three.js labs and Clippy (~2 MB off first load; no Clippy download on phones)
- Opaque scrolled nav, compact desktop dock, AA-contrast buttons, reduced-motion support
- Skip link and in-page anchors move focus; palette, retro dialogs and role tabs are keyboard-safe
- Line-art project schematics and spec-sheet expertise cards replace emoji tiles
- Tighter professional copy, compact retro shelf, real footer icons
- 404 page, robots.txt, sitemap, link-preview card, JSON-LD, touch icon
- Legacy pages redirect into homepage sections; drop dead scripts.js and CV-gate code
- Refresh repo snapshot, add CI for build/lint/model/smoke checks

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 2, 2026 21:37
Copilot AI reviewed Nov 2, 2026

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-02T21:39:15.590529Z d48be5e PR opened
🔒 Security Review ✅ Completed 2026-11-02T21:39:57.361499Z d48be5e PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

grimgrimberg and others added 3 commits November 6, 2026 13:39
@grimgrimberg @claude
ci: pin site checks to ubuntu-24.04
5ae968c 
ubuntu-latest moved to 26.04, where Playwright 1.59 cannot install browsers.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@grimgrimberg @claude
site: add Fly101 project page with real connectome renders
4306801 
- New fly101.html with a rotating 3D view of real MaleCNS neuron positions
- MuJoCo renders of the flybody fly and the SO-101 arm, and a nervous-system figure
- Full results, baselines and open questions in collapsible technical notes
- Homepage project card, footer link, sitemap, lint and hygiene coverage

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@grimgrimberg @claude
site: add connectome lab to Take the controls
1ec1183 
Rotate real MaleCNS neuron positions and switch regions on and off.
three.js and the point data load only when the lab opens.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@grimgrimberg
grimgrimberg merged commit 73a91a7 into dev Nov 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@grimgrimberg