Repository navigation
fix(web): reduce rendering overhead and repair mobile navigation - #99
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
cryptasian
marked this pull request as ready for review
September 18, 2026 05:07
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
Initial hero content waited for hidden client reveals, and mobile carousel pagination exposed 8px targets. This repair makes initial content readable from server HTML, keeps safe optional entrances below the viewport, and provides stable 44px pagination with specific service/privacy link names and unchanged consent semantics.
Shared rendering now uses exact lazy skyline vector crops, verified compact font faces with complete original-character fallbacks, native illustration observation and native process accordion transitions. Runtime-only Tailwind sources exclude planning examples. Artwork, copy, responsive layout, phase selection, reduced motion and themes are preserved.
Refs #96, #97. Parent #31. These references do not close their production acceptance gates.
Validation
All 24 cold local HTTP/2 runs pass score >90, LCP <2500ms, FCP <1800ms and CLS <0.1. Every desktop score is 100; every final run has accessibility/SEO/best-practices 100 and no warning/error. All final CLS values are zero.
Recorded production requests use HTTP/2; earlier next-start localhost results use HTTP/1.1. The additional controlled baseline/repair comparison uses the same loopback HTTPS/HTTP2 proxy, host and unchanged Lighthouse 13.4.1 default simulated CPU/network settings. All 12 unchanged-main HTTP/2 mobile LCPs fail. Local TLS uses a disposable certificate exception only in owned headless profiles. All 198 formal reports and earlier failures are retained; transport matching is local evidence, not final production acceptance.
Latest PR head c343e6d also passes the complete CI workflow; runtime source is unchanged from the measured commit.
Release verification
Implementation is complete and ready for review. #96/#97/#31 remain open for final production measurements, final target/link audits and their genuine field-data gate. Vercel builds the measured source successfully, but anonymous preview requests reach sign-in. Automatic approval review also blocked using the existing browser session without explicit preview-access authorization; that authorization is requested. No protected-preview result, production deployment or field INP success is claimed.