Skip to content

Commit d312795

Browse files
committed
fix(hub): sidebar — grey background, uppercase section labels, section borders, 260px width to match GitHub docs style
1 parent 7e6b622 commit d312795

1 file changed

Lines changed: 23 additions & 14 deletions

File tree

public/styles/global.css

Lines changed: 23 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -65,15 +65,15 @@ svg {
6565
position: fixed;
6666
top: 0;
6767
left: 0;
68-
width: 240px;
68+
width: 260px;
6969
height: 100vh;
7070
padding-top: 52px;
7171
/* Primer Header height */
7272
padding-bottom: 40px;
7373
border-right: 1px solid var(--borderColor-default, var(--color-border-default, #d1d9e0));
7474
overflow-y: auto;
7575
overflow-x: hidden;
76-
background-color: var(--bgColor-default, var(--color-canvas-default));
76+
background-color: var(--bgColor-muted, var(--color-canvas-subtle, #f6f8fa));
7777
z-index: 50;
7878
}
7979

@@ -89,15 +89,23 @@ svg {
8989

9090
/* -- Nav section wrappers ---------------------------------- */
9191
.nav-section {
92-
padding: 0 0 4px;
92+
padding: 0 0 8px;
93+
border-top: 1px solid var(--borderColor-muted, var(--color-border-muted, #d8dee4));
94+
}
95+
96+
.nav-section:first-child {
97+
border-top: none;
98+
padding-top: 8px;
9399
}
94100

95101
/* Static section heading (e.g. "Understand") */
96102
.nav-section-label {
97103
display: block;
98104
padding: 12px 16px 4px;
99-
font-size: var(--font-size-small, 0.75rem);
105+
font-size: 0.6875rem;
100106
font-weight: 600;
107+
text-transform: uppercase;
108+
letter-spacing: 0.04em;
101109
color: var(--fgColor-muted, var(--color-fg-muted));
102110
}
103111

@@ -107,12 +115,14 @@ svg {
107115
align-items: center;
108116
justify-content: space-between;
109117
width: 100%;
110-
padding: 8px 14px 4px 16px;
118+
padding: 10px 14px 10px 16px;
111119
background: none;
112120
border: none;
113121
cursor: pointer;
114-
font-size: var(--font-size-small, 0.75rem);
122+
font-size: 0.6875rem;
115123
font-weight: 600;
124+
text-transform: uppercase;
125+
letter-spacing: 0.04em;
116126
color: var(--fgColor-muted, var(--color-fg-muted));
117127
line-height: 1.5;
118128
}
@@ -151,7 +161,7 @@ svg {
151161
/* .SideNav-item (Primer) + .nav-link (ours) + optional .active */
152162
.SideNav-item.nav-link {
153163
display: block;
154-
padding: 6px 16px;
164+
padding: 5px 16px 5px 14px;
155165
font-size: 0.875rem;
156166
font-weight: 400;
157167
color: var(--fgColor-default, var(--color-fg-default));
@@ -165,7 +175,8 @@ svg {
165175
}
166176

167177
.SideNav-item.nav-link:hover {
168-
background-color: var(--bgColor-muted, var(--color-canvas-subtle));
178+
background-color: var(--bgColor-neutral-hover, rgba(0,0,0,0.05));
179+
border-left-color: var(--borderColor-default, var(--color-border-default));
169180
color: var(--fgColor-default, var(--color-fg-default));
170181
text-decoration: none;
171182
}
@@ -174,12 +185,12 @@ svg {
174185
border-left-color: var(--fgColor-accent, var(--color-accent-fg));
175186
color: var(--fgColor-default, var(--color-fg-default));
176187
font-weight: 600;
177-
background-color: var(--bgColor-muted, var(--color-canvas-subtle));
188+
background-color: var(--bgColor-neutral-hover, rgba(0,0,0,0.05));
178189
}
179190

180191
/* -- Content offset (right of fixed sidebar) --------------- */
181192
.page-offset {
182-
margin-left: 240px;
193+
margin-left: 260px;
183194
display: flex;
184195
flex-direction: column;
185196
min-height: calc(100vh - 52px);
@@ -283,14 +294,12 @@ svg {
283294
}
284295

285296
.sidebar {
286-
left: -240px;
297+
left: -260px;
287298
top: 0;
288299
padding-top: 56px;
289300
z-index: 300;
290301
transition: left 0.2s ease;
291-
/* Solid fallback + CSS variable for light/dark */
292-
background-color: #ffffff;
293-
background-color: var(--bgColor-default, var(--color-canvas-default, #ffffff));
302+
background-color: var(--bgColor-muted, var(--color-canvas-subtle, #f6f8fa));
294303
}
295304

296305
.sidebar.open {

0 commit comments

Comments
 (0)