Repository navigation
[DONE] Preview, Pixel, Color Ramp, Knobs 패널만 쓰는 규칙 20개가 dock 틀 CSS 에서 그 패널의 CSS 로 간다 - #241
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FgcWUCHedy5R788oTvxvcv
Twenty rules in LabPanels.module.css were read by one panel file each; they now sit in that panel's stylesheet, and a test fails when the dock file holds a class only one panel file reads. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FgcWUCHedy5R788oTvxvcv
patternflow-bot
marked this pull request as ready for review
October 25, 2026 04:44
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_01FgcWUCHedy5R788oTvxvcv
Pattern Lab 의 dock 틀 CSS 에 남아 있던 Preview, Pixel, Color Ramp, Knobs 패널만 쓰는 규칙 20개를 그 패널의 CSS 로 옮겼어요. 화면은 그대로이고, 머지하면 dock 틀 CSS 에 패널 파일 하나만 읽는 클래스가 다시 들어올 때 시험이 빨개져요.
좋아지는 것: 없음
체크리스트
4위는 바퀴 9194([DONE] Pattern Lab 코드에서 아무도 읽지 않는 이름 둘이 빠지고, 다른 파일이 읽지 않는 export 62개가 파일 안으로 들어간다 #237~[DONE] Lab 머리줄 규칙 22개가 여럿이 쓰는 CSS 에서 dock 틀 CSS 로 간다 #240)가 했고, 남은 줄 11개 가운데 맨 위가 5위LabPanels.module.css의 클래스마다 읽는 파일 다시 세기PixelPanel.module.css로labCss.test.ts에 「패널 파일 하나만 읽는 클래스가 없다」 시험, dev CSS 에서 빨개지는지getComputedStyle비교3c27caf,90e4b81,a0fdb46) merge 했고 바뀐 건incoming/의 ROUND.md 둘뿐이에요.80e6ae5에서 Drift checks · Web CI 초록한 일
web/src/app/pattern-lab/LabPanels.module.css에서 패널 파일 하나만 읽는 규칙 20개를 그 패널의 CSS 로 옮겼어요. 값은 그대로예요.PreviewPanel.tsx만 읽는previewBody,previewCanvasBox,previewFrame,cameraCanvas의 규칙 다섯은panels/PreviewPanel.module.css끝으로 갔어요.PixelPanel.tsx만 읽는pixelViewport,pixelStage,pixelStatus,pixelMouseHint의 규칙 다섯과pixel/PixelToolbars.tsx만 읽는swatchRow,swatch,toolSep의 규칙 넷은 새panels/PixelPanel.module.css로 갔어요. 두 파일이 같이 읽어요.RampPanel.tsx만 읽는rampDock,rampAlphaRow,rampAlphaValue의 규칙 다섯은panels/RampPanel.module.css로 갔어요. 그 파일 끝의@container ramp-controls가 이제 그 컨테이너를 정하는.rampDock과 한 파일에 있어요.KnobsPanel.tsx만 읽는.knobsDock은panels/KnobsPanel.module.css의.controls뒤로 갔어요..panelBar .swatch는.swatchRow .swatch가 됐어요. 다른 파일의panelBar는 셀렉터에 쓸 수 없어요(모듈마다 해시 이름이 달라요). 견본은 모두swatchRow안에 있고 그 줄은 패널 바 안에 있어요. 클래스 둘이라.panelBar button(클래스 하나, 요소 하나)을 번들 순서와 상관없이 이겨요.colorWell은 Pixel, Text, Graphic Export 셋이 읽어서LabPanels.module.css에 남았어요.dock.*에서local.*로 바꿨어요.PixelPanel.tsx와PixelToolbars.tsx는 새 CSS 를dock다음에 불러와요.labCss.test.ts에 「LabPanels.module.css holds no class that only one panel file reads」 를 더했어요. 셸만 읽는 머리줄 클래스는 dock 틀이라 세지 않아요.readers()는registry.tsx의def.dock.ref도 읽는 이름으로 세서, 이 시험은 그 파일이 정의한 클래스만 봐요. 「Pixel status line on a touch screen」 은 새 CSS 를 읽어요.증거
LabPanels.module.css의 클래스마다 그 모듈을 불러오는 파일 가운데 그 이름을 읽는 파일을 셌어요(git grep 'dock\.<이름>'):새 시험이 dev 의 CSS 를 잡는지: CSS 넷과 tsx 다섯을 dev 판으로 되돌리고 돌리면 빨개져요. 나머지 둘은 새
PixelPanel.module.css가 디스크에 남아 있어서 빨개진 것이에요.화면이 그대로인지: dev 와 이 브랜치를 빌드해
next start로 열고, Chromium 으로/pattern-lab을 1440×900 와 390×844(터치,hover: none)에서 13가지 상태로 다뤘어요. 요소마다getComputedStyle의 모든 속성과 내용이 있는::before,::after, 입력 칸의::placeholder를 모아 DOM 경로로 맞춰 봤어요. 1440 은 처음 연 화면, Graphic Export 의 Viewfinder 로 Preview 에 카메라 캔버스를 띄운 화면, + Pixel 로 Pixel 패널을 앞에 둔 화면, 견본 hover, 툴바 단추 hover, 색 칸 포커스예요. 390 은 처음 연 화면, Knobs 앞, Color Ramp 앞, 카메라 캔버스, Pixel 앞(마우스 안내가 숨어요), 색 칸 포커스, Pixel 뒤의 Color Ramp 예요.옮긴 규칙이 그 비교에 실제로 잡혔는지, 클래스마다 그 규칙의 값을 모두 가진 요소를 셌어요. dev 와 브랜치가 열세 상태 모두 같았어요.
빌드된 CSS(
.next/static/chunks/*.css). 두 덩이 다/pattern-labHTML 이 처음부터 불러요:다섯 검사(
web/, 끝줄):GitHub 검사(그 사이 merge 는
incoming/만 바꿨어요):화면을 바꾸지 않아서 PR 에 사진은 싣지 않았어요. 상태마다 찍은 사진은 스크래치 폴더에서 상태가 맞는지 보는 데만 썼어요.
실행법
cd web npx vitest run src/app/pattern-lab/labCss.test.ts가정
@media (hover: none)안의.pixelMouseHint와 파일 끝의.rampDock > *가 있었고, 둘 다 한 패널 파일만 읽어서 같이 옮겼어요. 그래서 20개예요.PixelPanel.module.css에 뒀어요. 표의 고칠 곳에 새 CSS 는 그 하나이고, 툴바는 Pixel 패널을 이루는 파일이라pixel/안에 CSS 를 따로 두지 않았어요..panelBody와 한 요소인데 둘이 쓰는 속성이 겹치지 않고,.rampDock > *의flex-shrink를 쓰는 다른 규칙도 없어요. Knobs 의.controls는padding을 쓰지 않아요. 그래서 파일 안의 자리는 읽기 좋은 곳으로 정했어요.incoming/ROUND.md는 지시 1~5 를 코드 블록 안에 두었고 남은 지시가 없었어요. 지시 6 과 7 은 04:27 에 dev(90e4b81)에 들어왔고, 그때 후보 5 는 이미 끝나 검사가 초록이었어요. 그 커밋이 표를 쓰는 바퀴를 102 까지 늘리고 두 바퀴를 지시 6 과 7 에 두어서, 이 바퀴는 후보 5 그대로 두고 다음 바퀴가 지시 6 부터 해요.web/ARCHITECTURE.md:78의 「Each panel owns its stylesheet …LabPanels.module.cssthe dock chrome」 은 이제 맞아서 고치지 않았어요.elementFromPoint가 그 줄의 왼쪽, 가운데, 오른쪽 모두dv-sash를 돌려줘서, 그 높이를 누르면 메뉴 대신 sash 를 잡아요.dockview.css:2942의.dv-sash가z-index: 99이고.menuPop(LabPanels.module.css:254)이 60 이에요. 고치지 않았어요.Changelog
Web
LabPanels.module.cssto that panel's stylesheet, Pixel's in a newPixelPanel.module.css, and a test fails when the dock file holds a class only one panel file reads. Computed styles in 13 Lab states at 390px and 1440px matchdev.경험
언제: 바퀴에서 390px 터치 화면의 Pattern Lab Panels 메뉴를 Playwright 로 누를 때
교훈: 메뉴 줄 하나를 dockview sash(
z-index: 99)가 덮어click()이 30초 기다리다 실패하니, 메뉴 단추는locator.evaluate((el) => el.click())로 눌러요근거: #241 에서 390 흐름이 Panels 메뉴 클릭에서
dv-sash … intercepts pointer events로 두 번 멈췄고,elementFromPoint로 보니 「Color Ramp」 줄을 sash 가 덮었어요. 바꾼 뒤엔 13가지 상태가 다 떴어요바퀴
바퀴 95 · 후보 5 「Preview, Pixel, Color Ramp, Knobs 패널만 쓰는 규칙 18개가 dock 틀 CSS 에서 그 패널의 CSS 로 간다」
다음 바퀴
90e4b81)에 들어왔어요.PatternLabClient.tsx:6~:7머리 주석은 이번 바퀴가 건드리지 않았어요.https://claude.ai/code/session_01FgcWUCHedy5R788oTvxvcv
Generated by Claude Code