Repository navigation
[DONE] 모든 쪽이 받는 globals.css 에서 아무도 쓰지 않는 옛 Inside 패널 규칙 377줄이 빠진다 - #249
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
InsidePanel moved to its own CSS module, and no file reads the 27 inside-* and reddit-* classes left behind. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
patternflow-bot
marked this pull request as ready for review
October 25, 2026 06:49
This was referenced Oct 25, 2026
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 25, 2026
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
lsh / https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
모든 쪽이 받는 globals.css 에서 아무도 쓰지 않던 옛 Inside 패널 규칙 377줄을 걷었어요. Inside 패널은 이미 제 CSS 모듈을 써서 화면은 그대로이고, 머지하면 사이트의 모든 쪽이 받는 CSS 덩이가 6,107바이트(gzip 1,104) 가벼워져요.
좋아지는 것: 없음
체크리스트
!단언 가운데 타입으로 좁힐 수 있는 일곱이 빠진다 #248 에서 꼬리말 없는 세 직함 글은 [DONE] Lab 갤러리 패널·Gemini 걷어 내기(#183)가 원래 레포 dev 위에 겹침 없이 다시 얹힌다 #215 와 [DONE] 매니페스토 · AGENTS.md · /pattern 카드가 걷은 배치 생성 대신 지금 Pattern Lab 을 말한다 #243 의 닫는 알림 둘뿐이고 일을 시키지 않아요getComputedStyle비교/와/inside를 390 · 1440 으로 찍어 앞뒤 픽셀 비교09a4e42와2a66592에서 Drift checks, Web CI 둘 다 successa21d4aa를 merge 했어요(2a66592). 바뀐 곳이incoming/pattern-study/뿐이라 lint, typecheck,markers만 다시 돌려 통과했어요한 일
web/src/app/globals.css의:1117~:1494를 지웠어요. 「"Inside" panel list」 주석부터 그@media (max-width: 900px)의 닫는 괄호와 뒤 빈 줄까지예요. 「Sponsor」 주석 앞 빈 줄 하나는 남겨서 덩어리 사이 간격이 그대로예요. 파일은 2,852줄 75,019바이트에서 2,474줄 66,630바이트가 됐어요.지우기 전에 다시 쟀어요. 레포의 postcss 로 그 줄들을 읽으니 규칙 62개, 선언 239개, at-rule 은
@media (max-width: 900px)하나였고, 셀렉터마다.inside-*나.reddit-*클래스가 하나씩 들어 있었어요. 그 클래스 24개(inside-*20개,reddit-comment-band,reddit-comment-card,reddit-comment-track,reddit-upvotes)는web/src,web/content,web/public,docs,tools어디에도 없어요. 같이 걸린is-active,is-next,is-open,is-prev,panel-header,sub는 다 이 클래스들 밑에서만 쓰여서 다른 쪽 규칙에 닿지 않아요. 덩어리 안에서 정의한 변수 다섯(--inside-bg,--inside-text,--inside-heading,--inside-accent,--inside-wire)도 덩어리 밖에서var()로 읽는 곳이 없어요.web/src에서inside-나reddit-를 글자로 이어 클래스 이름을 만드는 곳도 없고,reddit-comment는 저널 MDX 의 사진 파일 이름(/journal/v1-30-days/reddit-comment-1.png)으로만 나와요.증거
CSS 덩이 크기
같은 트리에서 이 커밋만 빼고 넣어
npm run build를 두 번 했어요..next/static/chunks/*.css15개 가운데 바뀐 것은 모든 쪽이 받는 덩이(/의 HTML 이 부르는 둘 가운데 하나) 하나예요.뒤 빌드의 CSS 에서
inside-panel을 찾으면 0개예요.계산된 스타일
/,/inside,/build,/pattern,/journal/v1-30-days를 390×844(2×)와 1440×900 으로 열어load뒤 3초를 기다리고,body아래 모든 요소와::before,::after의getComputedStyle을 다 받아 비교했어요(reducedMotion 은 reduce). 앞 빌드를 두 번 받아 비교한 것이 0 차이라 이 비교에 흔들림은 없어요.사진
맨 위 셋과 아래 하나가 걷은 뒤예요. 앞 사진과 픽셀로 비교하니
/390 은 0 차이였어요. 나머지 셋은 3D 캔버스(/inside390 에서0,0 390×371, 1440 두 쪽에서0,0 691×900CSS px) 안에서만 달랐는데, 걷은 뒤 빌드를 두 번 찍어도 1440 두 쪽은 같은 칸이 달라요. 지구본과 제품 모델이 도는 그림이라서예요.검사
web/에서 돌렸어요.globals.css를 읽는 시험 셋(fonts.test.ts,hoverUnderline.test.ts,panelNotes.test.ts)이 다 지났어요.PR 의 Actions(head
2a66592): Drift checks, Web CI 둘 다 success.실행법
스타일을 받는 스크립트는 다섯 쪽을 두 폭으로 열어 요소마다 DOM 경로(태그와 형제 순번)를 열쇠로 계산된 속성을 JSON 에 적고, 비교 스크립트가 앞뒤 JSON 을 열쇠와 속성마다 맞대요. 앞 빌드로 한 번, 이 커밋으로 다시 빌드해 한 번 돌려요.
가정
inside-*와reddit-*이름은 24개예요. 그 밑에서만 쓰는 이름 여섯까지 넣으면 30개라, 27 이 어디서 왔는지는 모르겠어요. 지운 줄은 표가 적은 범위(:1117~:1493)와 같아요. 첫 커밋 메시지에 표의 27 을 그대로 옮겼는데, squash 머지라 dev 에는 PR 제목이 남아요.web/src/app/globals.css:1117~:1119의 「Sponsor」 머리 주석과 「60px horizontal, matching .hero」 주석이 Milestones 덩어리 위에 있고,.sponsor규칙은:1160부터 머리 없이 시작해요.Changelog
Web
InsidePanelmoved to its own CSS module, and nothing read the 24inside-*andreddit-*classes left inglobals.css. The CSS chunk every page loads is 6,107 bytes smaller (1,104 gzipped), and computed styles on five pages at 390 and 1440 px are unchanged.바퀴
바퀴 100 · 후보 8 「모든 쪽이 받는
globals.css에서 아무도 쓰지 않는 옛 Inside 패널 규칙 377줄이 빠진다」경험
언제: 바퀴에서 CSS 를 걷거나 옮기고 화면이 그대로인지 사진으로 보일 때
교훈: 홈 탭과
/inside는 3D 캔버스가 같은 빌드에서도 찍을 때마다 달라서 사진 비교는 캔버스 칸을 빼고 읽고, 판정은 reducedMotion 으로 연 쪽의 모든 요소getComputedStyle을 앞뒤로 맞대어 해요근거: #249 에서 같은 뒤 빌드를 두 번 찍어도 1440 의
/와/inside가 캔버스0,0 691×900안에서 달랐고, 계산된 스타일은 같은 빌드 두 번과 앞뒤 빌드 모두 10쌍 0 차이였어요다음 바퀴
globals.css에 남은 안 쓰는 규칙 70개와 그것만 쓰던 애니메이션 둘, 변수 넷이 빠진다」. 이 PR 뒤로 표가 적은:1117아래 줄 번호는 378씩 올라가요(:1770은:1392,:2597은:2219).https://claude.ai/code/session_014FvJjjS6wAXXHdmeMcBk47
Generated by Claude Code