Skip to content

Redesign the Highlights landing page - #1155

Open
mdo wants to merge 3 commits into
mainfrom
feat/highlights-landing-page
Open

mdo wants to merge 3 commits into
mainfrom
feat/highlights-landing-page

Conversation

@mdo

@mdo mdo commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • refresh the Highlights hero and interactive TSX playground
  • add a responsive language explorer with featured language icons and the full lexer catalog
  • present published throughput, memory, tokenization, and live-edit benchmark evidence with methodology notes
  • improve responsive controls, editor scrolling, theme selection, and light/dark previews

Verification

  • moon run root:format root:lint
  • moonx docs:typecheck
  • moonx docs:test
  • git diff --check
  • IDE diagnostics for changed docs files

mdo and others added 2 commits September 24, 2026 16:31
Visitors now see focused product copy, an editable TSX example, and a
second playground for exploring the full language catalog.

Split the playground into hero and language variants, add responsive
controls and language icons, and keep theme previews editable in place.

Co-authored-by: Cursor <cursoragent@cursor.com>
Visitors can compare large-file throughput, process memory, themed token
performance, and live-edit latency directly on the Highlights page.

Present the published measurements with accessible figures and link the
methodology beside concise notes about scales and test conditions.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
pierre-diffshub Ready Ready Preview Sep 24, 2026 11:36pm UTC
pierre-docs-diffs Ready Ready Preview Sep 24, 2026 11:36pm UTC
pierre-docs-trees Ready Ready Preview Sep 24, 2026 11:36pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
pierrejs-diff-demo Skipped Skipped Sep 24, 2026 11:36pm UTC

Request Review

Keep the hero's supporting copy focused on the package source by
removing the language count and benchmark methodology link.

Co-authored-by: Cursor <cursoragent@cursor.com>

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: be3e5e40da

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

{FEATURED_LANGUAGES.map((lang) => {
const { icon, color } = FEATURED_LANGUAGE_ICONS[lang];
const selected = language === lang;
const maskImage = `url("${icon}")`;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use the SVG import URL for CSS masks

When this client component is bundled by Next, a static SVG import resolves to image metadata rather than a URL string. Interpolating icon therefore produces url("[object Object]"), so all five featured-language masks are invalid and their icons disappear. Construct the mask from the imported asset's src value, or import the SVGs through a loader that explicitly returns URLs.

Useful? React with 👍 / 👎.

const largeFileResults = [
{
language: 'TypeScript',
size: '517 KB',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve binary units in benchmark labels

These values are copied numerically from packages/highlights/benchmark/README.md, where fixture sizes are reported in KiB, memory in MiB, and throughput in MiB/s, but the new page labels them as KB, MB, and MB/s without conversion. For example, the source result is 517 KiB rather than 517 KB; the same mismatch affects the memory chart and both token-throughput cards. Use the original binary units or convert the numeric values so the published benchmark evidence is not misrepresented.

Useful? React with 👍 / 👎.

This branch was successfully deployed

3 active and 1 inactive deployments
Preview – pierre-docs-diffs — 17f762cb Deployed Sep 24, 2026 by vercel[bot]
Preview – pierre-docs-trees — 17f762cb Deployed Sep 24, 2026 by vercel[bot]
Preview – pierre-diffshub — 17f762cb Deployed Sep 24, 2026 by vercel[bot]
Preview – pierrejs-diff-demo — 17f762cb Deployed Sep 24, 2026 by vercel[bot]
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.

1 participant