chore(match-css): remove two blocks of dead cascade-overridden CSS
Two provably-unreachable blocks in resources/views/videos/types/match.blade.php
whose cascade targets are overridden further down with !important:
1. The pre-override pair at ~line 320:
.events-sidebar { display: none; }
.events-sidebar.show { display: flex; }
is overridden by the primary .events-sidebar definition further down
that sets `display: none !important` and `.show { display: flex !important }`.
2. The first `@media (max-width: 1300px)` block that sized
.yt-sidebar-container + .events-sidebar to 300 px is overridden by the
SECOND `@media (max-width: 1300px)` block immediately after, which sets
both to `width: 100% !important` as part of the stacking layout. Same
breakpoint, same selectors, !important wins.
Both removals leave Blade comments in place so the next reader can see why
the gap is there. Smoke-tested: /, /videos still 200; full test suite
80/80 green. Full consolidation of the four responsive .events-sidebar
variants (desktop, 1300 px stacking, 991 px bottom sheet, fullscreen
drawer) is left for a future turn with a browser visual-regression
harness — each variant has a distinct behavior that would be risky to
merge without one.
Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
This commit is contained in:
1 parent
8fef257c86
commit
c59b36ebf7
1 file changed
+12
-17
@@ -317,14 +317,12 @@
|
|||||||
|
|
||||||
/* No rotation animation */
|
/* No rotation animation */
|
||||||
|
|
||||||
/* Hide sidebar by default */
|
{{-- Dead-code removal (2026-10-21 audit): the previous `.events-sidebar
|
||||||
.events-sidebar {
|
{ display: none }` + `.show { display: flex }` pair here was
|
||||||
display: none;
|
overridden further down (search for the `display: none !important`
|
||||||
}
|
definition under /* ===== Match Events Sidebar ===== */). The
|
||||||
|
!important forms silently won every cascade; this pair never
|
||||||
.events-sidebar.show {
|
rendered. --}}
|
||||||
display: flex;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* ===== Playlist Navigation Controls ===== */
|
/* ===== Playlist Navigation Controls ===== */
|
||||||
.playlist-controls {
|
.playlist-controls {
|
||||||
@@ -2009,15 +2007,12 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* ===== Responsive ===== */
|
/* ===== Responsive ===== */
|
||||||
@media (max-width: 1300px) {
|
{{-- Dead-code removal (2026-10-21 audit): a prior
|
||||||
.yt-sidebar-container {
|
@media (max-width: 1300px) block sized .yt-sidebar-container +
|
||||||
width: 300px;
|
.events-sidebar to 300 px. The immediately-following
|
||||||
}
|
@media (max-width: 1300px) rule overrides both to 100% with
|
||||||
|
!important as part of the stacking layout, so the 300 px pair
|
||||||
.events-sidebar {
|
never applied. --}}
|
||||||
width: 300px;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/* ── Narrow desktop: too little room for a side rail ──────────────
|
/* ── Narrow desktop: too little room for a side rail ──────────────
|
||||||
Between 992px and 1300px the left nav (240px) and the 300px rail
|
Between 992px and 1300px the left nav (240px) and the 300px rail
|
||||||
|
|||||||
Reference in new issue
Block a user