Skip to content

Commit 48364f8

Browse files
committed
refactor(ui): drop duplicate spin keyframe + trim repeated comment
Post-review cleanup of 98e8fa2: - Skills-toggle spinner now references the existing global `@keyframes spin` (global.css:1027) instead of declaring a local `skills-toggle-spin` with the identical body. - Trim the .library-tabs--top opacity-fade comment to a one-line cross-reference; the full backdrop-filter / stacking-context rationale already lives next to the JSX hoist in CenterPanel.tsx. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 98e8fa2 commit 48364f8

1 file changed

Lines changed: 2 additions & 11 deletions

File tree

src-ui/src/components/center/CenterPanel.css

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -799,12 +799,7 @@
799799
top: 24px;
800800
left: 50%;
801801
transform: translateX(-50%);
802-
/* Now a sibling of launchpad-chrome--library (not a child), so it
803-
owns its own show/hide transition. Self-fading opacity is fine
804-
here — only an ANCESTOR's opacity<1 isolates a stacking context
805-
that breaks backdrop-filter; the element's own opacity scales
806-
the post-filter result, which renders as the expected gradual
807-
glass reveal in sync with the back-chevron's fade. */
802+
/* Self-fading opacity (see hoist rationale in CenterPanel.tsx). */
808803
opacity: 0;
809804
pointer-events: none;
810805
transition: opacity 0.3s ease;
@@ -1012,11 +1007,7 @@
10121007
border: 2px solid rgba(255, 255, 255, 0.35);
10131008
border-top-color: #fff;
10141009
border-radius: 50%;
1015-
animation: skills-toggle-spin 0.7s linear infinite;
1016-
}
1017-
1018-
@keyframes skills-toggle-spin {
1019-
to { transform: rotate(360deg); }
1010+
animation: spin 0.7s linear infinite;
10201011
}
10211012

10221013
.skills-toggle-track {

0 commit comments

Comments
 (0)