Skip to content

improvement(landing): re-encode embedded brand logos to cut comparison page weight - #9631

Merged
waleedlatif1 merged 1 commit into
stagingfrom
improvement/compress-brand-logos
Nov 3, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
improvement/compress-brand-logos

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • 13 brand logos in icons.tsx (11 competitors used on comparison pages, plus LiteLLM and Dust) are raster images inlined as base64. A comparison page renders the same logo up to 16 times, and each copy is serialized twice (HTML plus RSC payload). Some carried several KB of Photoshop XMP padding.
  • Re-encoded each one with sharp: metadata stripped, PNGs palette-quantized, JPEGs re-encoded with mozjpeg, and rasters over 128px downscaled to 128px. Tiles render at 32–36px, so 128px stays sharp at 3x. Total base64 drops from 95 KB to 29 KB. The markup, the viewBox, and the <image> width and height are unchanged. Only the href data changes.
  • Four logos were JPEG bytes labelled data:image/png. They're now labelled image/jpeg.
  • Kept them as data URIs on purpose. apps/docs renders a byte-identical copy of icons.tsx (synced by generate-docs), so a /public path would break on the docs origin. The docs copy is updated to match.
  • Estimated compressed size (current live HTML with the new strings swapped in): /comparisons/n8n 313 → 209 KB, /comparisons 179 → 93 KB, /library/best-zapier-alternatives 132 → 104 KB.

Type of Change

  • Improvement

Testing

  • Before/after contact sheet of all 13 logos at 108px (3x the tile size). No visible difference, and transparency is kept.
  • biome check on both icon files and check-icon-path-precision pass. apps/docs/components/icons.tsx is byte-identical to apps/sim/components/icons.tsx, which is the invariant check:docs enforces. CI runs the full suite.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Nov 3, 2026 8:30pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issues were found.

Summary

Re-encodes 13 embedded brand logos in both icon files to reduce page weight.

  • Embedded brand logos carry less image data at the same display size.

Reviews (1) · Last reviewed commit: "improvement(landing): re-encode embedded..." · Reviewed by Greptile

@waleedlatif1
waleedlatif1 merged commit 1e0a887 into staging Nov 3, 2026
81 checks passed
@waleedlatif1
waleedlatif1 deleted the improvement/compress-brand-logos branch November 3, 2026 20:39

This branch was successfully deployed

1 active deployment
Preview — 5e236ba3 Deployed Nov 3, 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