Repository navigation
[DONE] Layers 패널의 규칙 24개가 dock 틀 CSS 에서 Layers 패널의 CSS 로 간다 - #239
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SQS5m2sJXtwJ16cxu5cKsz
…el.module.css LabPanels.module.css is meant to hold the dock chrome; 24 of its rules were read by LayersPanel.tsx alone. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SQS5m2sJXtwJ16cxu5cKsz
patternflow-bot
marked this pull request as ready for review
October 25, 2026 03:23
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
Patternflow-lab#183 Patternflow-lab#184 Patternflow-lab#186 Patternflow-lab#187 Patternflow-lab#189 Patternflow-lab#191 Patternflow-lab#192 Patternflow-lab#193 Patternflow-lab#194 Patternflow-lab#196 Patternflow-lab#197 Patternflow-lab#198 Patternflow-lab#199 Patternflow-lab#200 Patternflow-lab#202 Patternflow-lab#203 Patternflow-lab#204 Patternflow-lab#205 Patternflow-lab#206 Patternflow-lab#208 Patternflow-lab#209 Patternflow-lab#210 Patternflow-lab#214 Patternflow-lab#217 Patternflow-lab#218 Patternflow-lab#221 Patternflow-lab#222 Patternflow-lab#223 Patternflow-lab#227 Patternflow-lab#237 Patternflow-lab#238 Patternflow-lab#239 Patternflow-lab#240
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_01SQS5m2sJXtwJ16cxu5cKsz
Layers 패널만 쓰던 규칙 24개를 dock 틀 CSS(
LabPanels.module.css)에서 새panels/LayersPanel.module.css로 옮겼어요. 화면은 그대로이고, 머지하면 dock 틀 CSS 가 652줄에서 465줄로 줄어 다른 패널 규칙을 옮기는 4위와 5위가 덜 겹쳐요.좋아지는 것: 없음
체크리스트
LabPanels.module.css에서 Layers 만 읽는 클래스 다시 세기 — 15개, 규칙 24개, 다LayersPanel.tsx만 읽어요panels/LayersPanel.module.css로 옮기고@media (hover: none)을 Layers 와 Pixel 둘로 나누기.layerControlRow select:focus-visible도 옮기기labCss.test.ts를 새 파일로 맞추고, 두 파일 사이 순서를 지키는 시험 하나getComputedStyle비교fe9a161그대로라 merge 할 것이 없었고,8c1426f에서 Drift checks · Web CI 초록한 일
web/src/app/pattern-lab/LabPanels.module.css의 「Layers panel」 절(:314:472, 규칙 19개)과@media (hover: none)안의 Layers 규칙 5개(:478:501)를 새panels/LayersPanel.module.css로 옮겼어요. 값과 순서는 그대로예요. 같은 블록에 있던.pixelMouseHint는 제@media (hover: none)하나를 갖고LabPanels.module.css에 남아요(5위 몫이에요).:642~:644의 포커스 규칙은 셀렉터 셋을 묶어서.layerControlRow select:focus-visible를 안고 있었어요. 그 클래스가 다른 파일의 해시 이름이 되면 Blend 칸에 닿지 않으니, 그 하나를 떼어 새 파일 끝에 같은 값으로 두었어요.LayersPanel.tsx는dock.layer*열아홉 곳을local.layer*로 읽어요.dock.panel,dock.panelBar,dock.panelBody는 그대로예요..layerBar button(터치에서 44px)은LabPanels.module.css의.panelBar button(28px)과 특이도가 같아서, 뒤에 오는 쪽이 이겨요. 지금까진 한 파일 안의 순서였고 이제는 번들 안의 순서예요.LayersPanel.tsx가LabPanels.module.css를 먼저 불러오게 두고, 두 파일 주석에 까닭을 적었어요.labCss.test.ts의 「Layers and Knobs on a touch screen」 은 Layers 규칙을 새 파일에서 읽어요. 「come after the rules they override」 엔LayersPanel.tsx의 두 import 순서를 더했어요.:1~:3)은 「knob/ramp/gallery/modal styles are shared from PatternLab.module.css」 라고 했어요. Gallery 는 10월 23일에 걷었고 knob 과 ramp 규칙도 9월에 제 파일로 갔으니, 이 파일이 지금 담는 것과 아직 남은 패널 규칙을 적었어요.증거
옮긴 클래스(
git grep으로dock.layer*를 읽는 파일을 셌어요):화면이 그대로인지: 바퀴 89(#223)처럼
NEXT_PUBLIC_BUILD_ENABLED=1로 dev 와 이 브랜치를 빌드해next start로 열고, Chromium 으로/pattern-lab을 390×844(터치,hover: none)와 1440×900 에서 29가지 상태로 다뤘어요. 요소마다getComputedStyle의 모든 속성과::before,::after를 정렬해 모아 DOM 경로로 맞춰 봤어요. 레이어 넷을 미리 넣은 프로젝트(오류가 나는 Code,screen 50%Pixel, 숨긴 Pixel, 뒤집은 마스크 Text)로 시작해서 Layers 클래스 15개가 다 화면에 나와요. 상태는 마스크 레이어 선택, Pixel 레이어 선택(Opacity, Blend 줄), 줄 hover(1440), 키보드로 Blend 칸에 포커스(:focus-visible확인), 끌어 놓기 표시(data-dragover), 이름 고치기 칸, 패널 탭 여덟을 하나씩 앞으로, Panels 메뉴예요.다른 곳은 dockview 탭 줄뿐이에요. 390 에서 Graphic Export, Director 를 앞으로 꺼내면 탭 줄 끝의 넘침 수가 실행마다 3 이나 4 로 달라지고, 그 글자 폭(38.125px 와 38.766px)이 탭 줄 폭을 바꿔요. dev 여섯 번과 이 브랜치 세 번에서 두 값이 양쪽에 다 나왔어요:
터치 규칙이 파일을 건너서도 이기는지:
빌드된 CSS(
.next/static/chunks/*.css)에서 두 규칙은 같은 덩이에 있고,.layerBar button이 뒤예요:시험이 순서를 잡는지:
LayersPanel.tsx에서 두 import 를 뒤바꾸고 돌리면 빨개져요.다섯 검사(
web/, 끝줄):8c1426f의 GitHub 검사:화면을 바꾸지 않아서 PR 에 사진은 싣지 않았어요. 29가지 상태의 사진은 스크래치 폴더에서 상태가 맞는지 보는 데만 썼어요.
실행법
cd web npx vitest run src/app/pattern-lab/labCss.test.ts가정
.layerControlRow select:focus-visible하나를 떼어 옮겼어요. 남겨 두면 Layers 의 Blend 칸이 키보드 포커스 테두리를 잃어요..layerBar button은 특이도를 올리지 않고 import 순서로 이기게 두었어요. 클래스를 두 번 쓰는 셀렉터 같은 꼼수보다 읽기 쉽고, 순서는 시험이 지켜요. Knobs, Ramp, Text 패널은 제 CSS 를dock보다 먼저 불러오는데, 그 파일들엔dock과 다투는 규칙이 없어요.Changelog
Web
LayersPanel.tsxread moved from the dock'sLabPanels.module.csstopanels/LayersPanel.module.css, and computed styles in 29 Lab states at 390px and 1440px matchdev.경험
언제: 바퀴에서 Pattern Lab 을 390px 로 열어 dev 와 브랜치의
getComputedStyle을 비교할 때교훈: Graphic Export 나 Director 처럼 늦게 불러오는 탭을 앞으로 꺼낸 상태는 dockview 탭 줄의 넘침 수가 실행마다 달라 dev 끼리도 2~24 요소가 다르니,
.dv-tabs-and-actions-container안은 따로 세고 그 밖이 0 인지로 판정해요근거: #239 에서 dev 를 여섯 번 찍으니 넘침 칸 폭이 38.125px 와 38.766px 로 갈렸고, 다른 곳은 브랜치와 비교해도 0 이었어요
바퀴
바퀴 93 · 후보 3 「Layers 패널의 규칙 24개가 dock 틀 CSS 에서 Layers 패널의 CSS 로 간다」
다음 바퀴
PatternLab.module.css에서 셸의 CSS 로 간다」: 이번 바퀴의 비교를 그대로 쓰되, 390 의 탭 줄 차이는 따로 세요.LabPanels.module.css줄 번호가 187줄 앞당겨졌고,.pixelMouseHint는 이미 제@media (hover: none)블록에 혼자 있어요.Generated by Claude Code