Skip to content

docs(site): adopt new Sentry CLI brand lockup + app-icon favicon#1181

Merged
BYK merged 4 commits into
mainfrom
docs/site-brand-assets
Jul 2, 2026
Merged

docs(site): adopt new Sentry CLI brand lockup + app-icon favicon#1181
BYK merged 4 commits into
mainfrom
docs/site-brand-assets

Conversation

@BYK

@BYK BYK commented Jul 2, 2026

Copy link
Copy Markdown
Member

Updates the documentation site (cli.sentry.dev) with the new Sentry CLI brand assets from the brand portal, matching the CLI banner (#1169) and README (#1178).

Changes

  • Header logo → the official "SENTRY CLI" brand lockup (arch mark + retro wordmark) as a single white logo (docs/src/assets/logo.svg, logo.src + replacesTitle). The site theme (@sentry/starlight-theme) maps [data-theme="light"] and [data-theme="dark"] to the same dark palette (effectively dark-only), so a theme-split logo would render dark marks invisibly on the always-dark header — a single white logo is correct for every mode.
  • Favicon → new app icon (dark tile + purple/green gradient + arch) as favicon.svg, with an explicit favicon.png (256×256) <link rel="icon"> fallback for browsers without SVG-favicon support.
  • OG images regenerated: og-image.png (1200×630) and og-image-twitter.png (1920×1080) — white lockup + tagline on the brand-dark background with subtle purple/green gradient blobs.
  • Accent color → brand purple #7553FF (--sl-color-accent-rgb: 117 83 255), the mid-anchor of the banner gradient. Hardcoded accent-text purples (#a78bfa) aligned to the light anchor #9e86ff.
  • wordmark.svg (white) / wordmark-light.svg (dark #181225) / glyph.svg (new arch) refreshed; old logo.svg (arch + lowercase "sentry") replaced.

Verification

  • astro build succeeds (39 pages). Built HTML renders a single /_astro/logo.*.svg header image (no light:sl-hidden/dark:sl-hidden), <link rel="icon" href="/favicon.svg"> + PNG fallback, and og:image/twitter:image → the new cards.
  • An independent adversarial review confirmed: logo not clipped (ink fits the 1135×195 viewBox), OG dimensions + rendered text correct, SVGs hygienic (no scripts/external refs/ID collisions), no stale old-accent refs. Verdict: mergeable.

Hold

Do not merge yet — pending designer sign-off on the brand treatment.

Assets exported from the Sentry brand portal.

Swap the site header logo to the official 'SENTRY CLI' brand lockup (arch mark +
retro wordmark) with theme-aware light/dark variants, and use the new app-icon
(purple/green blobs + arch) as an SVG favicon.

- docs/src/assets/logo-{light,dark}.svg: brand lockup, background stripped and
  clip-paths removed (transparent; unique per theme)
- docs/public/favicon.svg (+ favicon.png fallback): new app icon
- astro.config.mjs: logo.{light,dark} + favicon -> /favicon.svg
- remove old logo.svg (arch + lowercase 'sentry')

Assets exported from the Sentry brand portal. Verified via astro build (39 pages).
@github-actions github-actions Bot added the risk: low PR risk score: low label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://cli.sentry.dev/_preview/pr-1181/

Built to branch gh-pages at 2026-07-02 20:38 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@cursor cursor 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6791eb2. Configure here.

Comment thread docs/astro.config.mjs Outdated
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ Patch coverage is 100.00%. Project has 5197 uncovered lines.
❌ Project coverage is 81.66%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    81.72%    81.66%    -0.06%
==========================================
  Files          410       410         —
  Lines        28334     28334         —
  Branches     18437     18437         —
==========================================
+ Hits         23153     23137       -16
- Misses        5181      5197       +16
- Partials      1898      1892        -6

Generated by Codecov Action

…glyph

The docs theme maps light and dark to the same dark palette (dark-only), so the
theme-split logo showed the dark-marks variant on a dark header (invisible).
Revert to a single white logo that works in every mode.

Also apply the rest of the brand refresh:
- Accent color -> brand purple #7553FF (mid-anchor of the banner gradient)
- og-image.png (1200x630) + og-image-twitter.png (1920x1080) regenerated with
  the SENTRY CLI lockup + gradient blobs on the brand-dark background
- wordmark.svg (white) / wordmark-light.svg (dark) / glyph.svg -> new brand marks

Verified via astro build: header renders a single /_astro/logo.svg (no
light/dark hiding), favicon = /favicon.svg.
@github-actions github-actions Bot added risk: medium PR risk score: medium and removed risk: low PR risk score: low labels Jul 2, 2026
BYK added 2 commits July 2, 2026 20:36
…t text

- Add explicit <link rel=icon> PNG fallback so favicon.png is actually used
  (Starlight only emits the SVG link) for browsers without SVG-favicon support
- Align hardcoded accent-text purple #a78bfa -> brand light anchor #9e86ff in
  cli.css and Terminal.astro (matches the new #7553FF accent)

Left the PackageManagerCode .pm-tab fallback as-is: it mirrors the theme's own
--ve-accent-soft (unchanged upstream); only the active theme value drives it.
@github-actions github-actions Bot added risk: high PR risk score: high and removed risk: medium PR risk score: medium labels Jul 2, 2026
@BYK BYK merged commit 4110b3a into main Jul 2, 2026
32 checks passed
@BYK BYK deleted the docs/site-brand-assets branch July 2, 2026 20:47
BYK added a commit that referenced this pull request Jul 3, 2026
Headings use the self-hosted Dammit Sans display font (base-aware @font-face, Vite-hashed); body uses the theme's self-hosted Rubik Variable; code stays JetBrains Mono. Follow-up to #1181.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant