Skip to content

Commit de99af5

Browse files
mlaetitiaclaude
andcommitted
Trim CSS comments to the ones that prevent silent breakage
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 58a3170 commit de99af5

5 files changed

Lines changed: 7 additions & 15 deletions

File tree

‎themes/osi/assets/scss/_6_components.header.scss‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,6 @@
161161
// open-panel layers: full-bleed white veil (::after escapes the wrapper inset) and
162162
// page dim; z-index -1 keeps both under content, tree order paints veil over dim
163163
@mixin nav-open-layers {
164-
// the border is the header/panel divider: on the veil so it never fades with panel switches
165164
&::after {
166165
background-color: $Nwhite;
167166
border-bottom: 1px solid $Nlight;

‎themes/osi/assets/scss/_6_components.mobile-menu.scss‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
.site-header {
2-
// centered in the sticky header row: follows any logo height or admin-bar shift
32
.open-button-wrapper {
43
position: absolute;
54
right: 16px;
@@ -190,7 +189,7 @@
190189
position: static;
191190
}
192191

193-
// stack while open: menu overlay (2) < white logo-row bar (3) < logo and buttons (4)
192+
// stack while open: overlay (2) < white bar (3) < logo and buttons (4)
194193
.header--blog-name {
195194
position: relative;
196195
z-index: 4;

‎themes/osi/assets/scss/_6_components.navigation--subnav.scss‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,6 @@
154154
}
155155
}
156156

157-
// selected items recolor only, no underline on mobile
158157
.menu-item.current-menu-item > a,
159158
.menu-item.current-menu-parent > a,
160159
.menu-item.current-menu-ancestor > a {
@@ -172,7 +171,6 @@
172171
padding: $smallPadding;
173172
}
174173

175-
// expanded section runs edge-to-edge: cancel the menu's side padding
176174
& > .menu-item > ul.sub-menu {
177175
margin-left: -$smallPadding;
178176
margin-right: -$smallPadding;
@@ -205,7 +203,6 @@
205203
}
206204
}
207205

208-
// filled triangle per the mobile design, replacing the base chevron glyph
209206
&:after {
210207
color: inherit;
211208
content: '\25BC';
@@ -380,10 +377,8 @@
380377
padding-right: .5em;
381378
}
382379

383-
// full-bleed panel: negative offsets escape the wrapper's inset, and the
384-
// content padding re-applies it so panel items align with the header content.
385-
// Closing freezes the panel solid for .2s so panel switches never cross-fade
386-
// (two translucent panels would flash the dimmed page through)
380+
// offsets/padding align the full-bleed panel with the header content; closing
381+
// freezes it solid for .2s so switches never cross-fade (dim would flash through)
387382
& > ul.sub-menu {
388383
box-shadow: 0 14px 18px -10px rgba(#{$Ndarkest_RGB}, 0.2);
389384
display: grid;
@@ -504,8 +499,7 @@
504499
}
505500
}
506501

507-
// keyboard open is JS-driven (.is-open); :focus-within here would defeat Escape.
508-
// The z-index bump paints the entering panel above a still-solid exiting one
502+
// keyboard open is JS-driven (.is-open); :focus-within here would defeat Escape
509503
&:hover > ul.sub-menu,
510504
&.is-open > ul.sub-menu {
511505
opacity: 1;

‎themes/osi/assets/scss/_6_components.navigation.scss‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ nav {
110110
display: none;
111111
}
112112

113-
// nav hover only recolors; the desktop top-level rule re-adds its own underline
113+
// source order matters: the desktop top-level rule re-adds its own underline below
114114
.nav-main--menu .menu-item a:hover,
115115
.nav-main--menu .menu-item a:focus,
116116
.nav-mobile--menu .menu-item a:hover,
@@ -207,7 +207,7 @@ footer {
207207
right: $smallPadding;
208208
text-align: center;
209209
text-decoration: none;
210-
z-index: 5; // local to the .nav-main overlay's own stacking context
210+
z-index: 5;
211211

212212
&:hover, &:focus {
213213
background-color: $brandColor1_Dk;

‎themes/osi/style.css.map‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)