Skip to content

Commit 693ccab

Browse files
docs: remove logo from docs pages header, fix code block bg
Header: - Removed logo from Starlight config entirely (no longer shows on any page by default) - Homepage Header renders the logo SVG manually via an img tag - Docs pages show plain text 'Sentry CLI' title with search — no logo Code blocks: - PackageManagerCode code area bg changed from hardcoded #0a0a0f to var(--ve-code-bg) to match the theme's code block background
1 parent 6a64022 commit 693ccab

4 files changed

Lines changed: 20 additions & 13 deletions

File tree

docs/astro.config.mjs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,6 @@ export default defineConfig({
4747
starlight({
4848
title: "Sentry CLI",
4949
favicon: "/favicon.png",
50-
logo: {
51-
src: "./src/assets/logo.svg",
52-
},
5350
social: [
5451
{
5552
icon: "github",

docs/src/components/Header.astro

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
/**
33
* CLI-specific Header — conditionally renders:
44
* - Homepage: branded header with logo, "Docs" link, social icons, no search
5-
* - Docs pages: theme's standard header with search (no logo)
5+
* - Docs pages: theme's standard header with text title + search
66
*/
77
import config from "virtual:starlight/user-config";
88
@@ -23,10 +23,13 @@ const base = baseUrl.endsWith("/") ? baseUrl : baseUrl + "/";
2323
---
2424

2525
{isHomepage ? (
26-
<!-- Homepage: branded header -->
26+
<!-- Homepage: branded header with logo -->
2727
<div class="header sentry-header header-homepage">
2828
<div class="title-wrapper sl-flex">
29-
<SiteTitle />
29+
<a href={base} class="site-title sl-flex">
30+
<img src={`${base}glyph.svg`} alt="" class="homepage-logo" />
31+
<span class="homepage-title">Sentry CLI</span>
32+
</a>
3033
</div>
3134
<div class="sentry-header-actions sl-flex print:hidden">
3235
<a href={`${base}getting-started/`} class="header-nav-link">Docs</a>
@@ -36,7 +39,7 @@ const base = baseUrl.endsWith("/") ? baseUrl : baseUrl + "/";
3639
</div>
3740
</div>
3841
) : (
39-
<!-- Docs pages: theme's standard header with search -->
42+
<!-- Docs pages: theme's standard header with text title + search -->
4043
<div class="header sentry-header">
4144
<div class="title-wrapper sl-flex">
4245
<SiteTitle />
@@ -57,6 +60,18 @@ const base = baseUrl.endsWith("/") ? baseUrl : baseUrl + "/";
5760
margin: 0 auto;
5861
}
5962

63+
.homepage-logo {
64+
height: 2.3rem;
65+
width: auto;
66+
}
67+
68+
.homepage-title {
69+
font-family: var(--sl-font);
70+
font-size: 1.25rem;
71+
font-weight: 700;
72+
color: var(--ve-text, #fafafa);
73+
}
74+
6075
.header-nav-link {
6176
color: #fff !important;
6277
text-decoration: none;

docs/src/components/PackageManagerCode.astro

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ const id = `pm-${Math.random().toString(36).substr(2, 9)}`;
125125

126126
.pm-content {
127127
padding: 0.75rem 1rem;
128-
background: #0a0a0f;
128+
background: var(--ve-code-bg, #141419);
129129
border-radius: 8px;
130130
}
131131

docs/src/styles/cli.css

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,6 @@
5050
fill: rgba(255, 255, 255, 0.8) !important;
5151
}
5252

53-
/* Logo sizing */
54-
.site-title img {
55-
height: 2.3rem;
56-
width: auto;
57-
}
5853

5954
/* ==========================================================================
6055
Splash / Landing Page Styles

0 commit comments

Comments
 (0)