Redesign: engineering-console — our design on current infrastructure#53
Conversation
Rebuilt the branch as a hybrid on top of current main: keep all of
Ilyar's non-design work, apply our design on top.
From main (non-design, kept):
- ESLint flat config + lint script and deps; Lighthouse CI; VirusTotal
URL check; updated toolchain (TypeScript 6, Vite 8) and dep patches.
- WebGL feature-detection + lazy-loaded Three scene (graceful fallback).
- webp avatars; SEO/OG meta, JSON-LD and JS-less static fallback in
index.html; robots/sitemap/security infra.
Our design (applied over it):
- Self-hosted brandbook fonts (Tilt Warp / Hanken Grotesk / JetBrains
Mono); wider content grid (1450px, indexed body 3/-1).
- Frosted-glass cards (43px blur, 0.46 fill) with the backdrop-filter
fix (no scroll-reveal layer on sections).
- CTAs scroll to #contact ("Get in touch"); 6 service cards with lucide
icons; section labels in gold mono (no index numbers).
- Hero full-screen; particle field fades to 20% and scatters on scroll;
airier mobile field. Concise Team bios, en dashes, trimmed footer,
in@ email.
InfinityField ref access moved into an effect to satisfy lint.
Lint, tsc and production build are green.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
bf4da44 to
0c7480b
Compare
ilyar
left a comment
There was a problem hiding this comment.
Review notes:
-
src/App.tsx:2/src/App.tsx:14regresses the previous code-splitting for the Three/WebGL scene. ImportingSceneeagerly pulls@react-three/fiber/threeinto the initial app chunk; my local PR build produceddist/assets/index--xhoKKxg.jsat ~1.12 MB minified, whereas the current split build keeps the main app around ~240 KB and loads the Three code separately. This is a user-facing startup/perf regression on mobile and for scanners/no-WebGL environments. Please keepScenelazy-loaded fromAppand letScenehandle its own WebGL fallback internally. -
src/components/three/InfinityField.tsx:305-312now triggersreact-hooks/exhaustive-deps:useMemoreadsopacityandscrollbut only depends ongl. The runtime update path mostly masks this throughuseFrame, but the warning is valid and leaves reduced-motion / demand-frame behavior harder to reason about. Either include the missing deps or initialize those uniforms from stable defaults and update them explicitly after mount.
I also checked the infra/security files that mattered for the recent VirusTotal work: the static index.html fallback, robots.txt, sitemap.xml, .well-known/security.txt, custom 404.html, Pages workflow and VT workflow are present on this branch. Checks are green, and the PR renders correctly in a quick mobile smoke test.
ilyar
left a comment
There was a problem hiding this comment.
Approving so we can merge the redesign and handle the noted performance/code-splitting follow-ups separately.
Summary
Lands the engineering-console redesign (our version) on top of current
main.Our redesign branch and
mainhad diverged into two parallel versions of thesite. As agreed, this PR resolves the overlap in favour of our version — so
merging replaces main's conflicting parts with ours. The branch is based on the
latest
main, so the diff below is exactly what changes relative to it.What's in it
JetBrains Mono) under
/public/fonts; Google Fonts CDN dropped.1450px, indexed body3 / -1).43pxblur,0.46fill). Fixes thebroken
backdrop-filterby removing the scroll-reveal layer that promotedsections to a backdrop root.
#contact("Get in touch"), flat style.dropped, labels shown in gold mono.
the page scrolls (scroll-reactive), with an airier, lighter mobile field.
(
© 2026 Ever Guild), contact emailin@ever-guild.net.Notes for review
changes where they overlapped (e.g. avatar format, some infra tweaks). Worth
a look during review if any of those should be kept.
backup/redesign-pre-rebase.🤖 Generated with Claude Code