Repository navigation
[DONE] 모든 쪽이 받는 globals.css 에서 아무도 쓰지 않는 규칙 87개와 그것만 쓰던 애니메이션 둘, 변수 넷이 빠진다 - #252
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
…ders The version tag, tooltip, spec and stack tables, roadmap track, placeholders, key points, marquee comments, build steps and tweaks panel left the components long ago. Their 87 rules, the two keyframes and four variables only they read go, and seven grouped selectors lose the dead names. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
patternflow-bot
marked this pull request as ready for review
October 25, 2026 08:41
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 26, 2026
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 26, 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.
ops / https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
모든 쪽이 받는 globals.css 에서 아무 컴포넌트도 그리지 않는 클래스의 규칙 87개와, 그 규칙만 쓰던 애니메이션 둘과 변수 넷을 걷었어요. 화면은 그대로이고, 머지하면 사이트의 모든 쪽이 받는 CSS 덩이가 8,942바이트(gzip 1,693) 가벼워져요.
좋아지는 것: 없음
체크리스트
getComputedStyle, 스물세 쪽에서 그려지는 클래스/contact와/build를 390 · 1440 으로 찍어 앞뒤 픽셀 비교916d933과836fe0b에서 Drift checks, Web CI 둘 다 success6384a41을 merge 했어요(836fe0b). 잠금 파일이 바뀌어npm ci부터 다섯 검사를 다시 돌려 통과했어요. 겹치는 다른 [DONE] PR 은 없었어요한 일
web/src/app/globals.css에서 규칙 87개를 지웠어요. 표가 센 70개에, 표가 다시 보라고 한.spec,.stack,.track,.placeholder넷의 규칙 7개, 다시 재며 더 나온 10개예요.표가 센 70개는 이래요. 옛 버전 꼬리표(
.version-tag)와 말풍선(.has-tip), 사양 표(.spec-row,.spec-total), 층 그림(.stack-layer), 로드맵 점(.track-dot), 자리 표시 칸(.ph-*), 요점 목록(.key-points), 흐르는 댓글 띠(.pf-marquee,.pf-track,.pf-comment들), Tweaks 창의.tw-head,.tw-row들, 연락 쪽의.business-contact-link,.business-intro-grid,.business-decks, 그리고 하나씩인.hero-viewer둘,.pretext-line둘,.tab-bar,.swipe-indicator,.pf-head둘이에요.더 나온 10개는 Build steps 덩어리 여섯(
.steps,.step figure,.step svg,.step figcaption,.step figcaption span, 540px 의.steps), Tweaks 의.tw와.tw.open,.panel-header .sub둘(기본과 900px)이에요.steps,tw,sub는 코드에 낱말로 흔해서 글자 찾기로는 산 것처럼 보였어요.이 규칙만 읽던
@keyframes slideUpFadeIn과pf-slide, 변수--led-hex,--pf-fs-card,--pf-fs-ghost(:root와 760px 둘),--pf-marquee-dur도 지웠어요. 묶인 셀렉터 일곱(.pf-kicker묶음,.business-*다섯, 720px 의.business-hero묶음)에서는 죽은 이름만 뺐어요. 빼고 나니 같은 셀렉터 규칙이 바로 붙어 선 곳이 넷이라 하나로 합쳤어요. 합친 짝마다 겹치는 선언이 없어서 어느 값도 순서가 바뀌지 않아요.증거
다시 잰 것
세 가지로 쟀어요. 레포의 postcss 로 이 파일의 클래스 164개를 뽑아
web/src,web/content,web/public,web/scripts,web/test,docs,tools에서 낱말로 찾으니 33개가 어디에도 없었어요. 저널 셋을 빼면 표의 70개와 같은 규칙이에요. 둘째로 className, class 가 든 줄만 보니track,tw,sub같은 낱말이 더 나왔어요. 셋째로 TypeScript AST 로className과class값 안의 글자, 그 안의 지역 변수가 받는 글자,classList,querySelector,closest,matches에 주는 글자를 다 모으니 53개가 빠졌어요. 그 가운데 손으로 본 낱말 넷(spec은TextPanel.tsx의spec.x같은 변수,placeholder는 입력 칸 속성)도 클래스로 쓰는 곳이 없었어요. 혼자 서는 셀렉터의 이름 여덟(placeholder,spec,stack,track,steps,step,tw,sub)은 쪽에 DOM 을 그리는 라이브러리(monaco-editor 0.55.1, dockview, react-markdown, esp-web-tools)에서도 클래스로 나오지 않아요.CSS 덩이 크기
같은 트리에서 이 PR 의 고침 하나만 빼고 넣어
npm run build를 두 번 했어요..next/static/chunks/*.css15개 가운데 바뀐 것은 모든 쪽이 받는 덩이 하나예요.그려지는 클래스
앞 빌드에서 23쪽(
/,/build,/pattern,/inside,/contact,/business,/editions,/features,/roadmap,/terms,/update,/journal,/journal/v1-30-days,/journal/en,/guide와 그 아래 넷,/pattern-lab,/build/breadboard,/compliance,/inside/seunghun-korea)을 390 과 1440 으로 열어 DOM 의 클래스를 다 모았어요. 서로 다른 클래스 828개 가운데 이 파일에서 빠진 이름 51개는 하나도 없었어요.계산된 스타일
/,/build,/pattern,/inside,/contact,/journal,/journal/v1-30-days를 390×844(2×)와 1440×900 으로 열어load뒤 3초를 기다리고,body아래 모든 요소와::before,::after의getComputedStyle을 받아 맞댔어요(reducedMotion 은 reduce).사진
맨 위 셋이 걷은 뒤예요. 앞 사진과 픽셀로 맞대니
/contact두 폭과/build390 은 0 차이였어요./build1440 은 오른쪽 아래 Case 고르기의 사진 칸(940,859~1388,900) 안에서만 달랐고, 채널 차이가 가장 커도 2/255 예요. 뒤 빌드를 두 번 찍은 것끼리는 0 차이라 빌드마다 사진을 다시 줄이는 데서 온 것으로 봤어요. 계산된 스타일은 그 쪽도 0 차이예요.검사
web/에서 돌렸어요.dev merge 뒤(
836fe0b) 다시 돌렸어요.PR 의 Actions(head
836fe0b): Drift checks, Web CI 둘 다 success.실행법
클래스를 재는 스크립트는 셋이에요. postcss 로 규칙마다 셀렉터의 클래스를 뽑아 낱말로 찾는 것, className 줄만 보는 것, TypeScript 컴파일러 API 로
className값의 글자를 모으는 것이에요. 스타일을 받는 스크립트는 일곱 쪽을 두 폭으로 열어 요소마다 DOM 경로를 열쇠로 계산된 속성을 JSON 에 적고, 비교 스크립트가 앞뒤 JSON 을 열쇠와 속성마다 맞대요.가정
.spec,.stack,.track,.placeholder넷은 이 바퀴가 다시 보라고 했어요. 다시 보니 넷도 클래스로 쓰는 곳이 없어 같이 걷었어요. AST 로 더 나온 10개도 「남은 안 쓰는 규칙」 이라 이 줄에 넣었어요. 그래서 제목의 수가 70 이 아니라 87 이에요.journal-list-number,journal-figure-wide,journal-figure-full)은 표대로 두었어요.web/src/app/globals.css:780~:783에 「Sponsor」 머리 주석과 그 아래 「60px horizontal」 주석이 Milestones 덩어리 위에 있고,.sponsor규칙은:823부터 머리 없이 시작해요. 바퀴 100 이 적은 그대로예요.Changelog
Web
slideUpFadeIn,pf-slideand four variables went with them. The CSS chunk every page loads is 8,942 bytes smaller (1,693 gzipped), and computed styles on seven pages at 390 and 1440 px are unchanged.경험
언제: 바퀴 컨테이너에서 띄운
next start나 Playwright 스크립트를 Bash 로 내릴 때교훈:
pkill -f "<명령 글>"은 같은 글이 든 제 Bash 까지 죽이니ps -eo pid,args로next-server의 PID 를 찾아kill해요근거: #252 에서
pkill -f를 앞에 둔 Bash 두 번이 제 명령 글에 걸려 스스로 죽었고(exit 144), 뒤에 이은 클래스 훑기와 뒤 빌드가 돌지 않아 다시 띄웠어요바퀴
바퀴 102 · 후보 10 「
globals.css에 남은 안 쓰는 규칙 70개와 그것만 쓰던 애니메이션 둘, 변수 넷이 빠진다」남은 것
import type을 바라고, 닿지 않는 코드를 잡는다」. 고칠 곳이 열한 곳이에요. 13~15위도 남았어요.globals.css에 남은 저널의 죽은 규칙 넷과 묶인 셀렉터 하나(journal-list-number,journal-figure-wide,journal-figure-full). 저널 MDX 가 앞으로 쓸 수도 있어 표가 뺐어요. 창작자가 안 쓴다고 하면 같은 셈으로 걷어요.https://claude.ai/code/session_011sntQbg3DAqGu1RXBiPrwW
Generated by Claude Code