Repository navigation
[DONE] Pattern Lab 의 Del 이 그림이 든 레이어를 지우기 전에 한 번 묻는다 - #98
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
Layer deletion has no undo and autosave overwrites the saved project within 3 s, so one misclick lost a whole pixel drawing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
patternflow-bot
marked this pull request as ready for review
October 16, 2026 16:09
This was referenced Oct 16, 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.
dev / https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
Pattern Lab 의 Del 은 누르자마자 레이어를 지웠고 되돌릴 길이 없었어요. 자동 저장이 3초 안에 지운 판을 덮어써서 잘못 한 번 누르면 픽셀 그림 하나가 사라졌어요. 머지하면 그림이 든 픽셀 레이어나 코드를 고친 코드 레이어는 지우기 전에 이름을 대고 한 번 물어요. 빈 레이어는 지금처럼 바로 지워요.
좋아지는 것: 레이어를 지우기 전에 한 번 묻는다 — 잘못 눌러 그림을 잃지 않는다
할 일
dev에선 아직 고쳐지지 않았어요devmerge한 일
app/pattern-lab/panels/LayersPanel.tsx의 Del 이window.confirm으로Delete the layer "Pixel 1"? It can't be brought back.을 물어요. Director 가 쇼를 갈아 끼우기 전에 묻는 꼴과 같아요(DirectorPanel.tsx:97). 취소하면 아무것도 바뀌지 않아요.묻지 않고 지우는 레이어가 둘 있어요. 알파가 0 이 아닌 픽셀이 하나도 없는 픽셀 레이어, 그리고 코드가
STARTER_CODE와 글자까지 같은 코드 레이어예요.holdsWork가 이 둘을 가려요.새 시험
app/pattern-lab/panels/LayersPanel.test.tsx는 여섯 개예요. 한 레이어일 때 Del 이 꺼져 있다. 빈 픽셀 레이어와 시작 코드 그대로인 코드 레이어는 묻지 않고 지운다. 그린 픽셀 레이어는 묻는다. 취소하면 픽셀까지 남는다. 확인하면 지운다. 코드를 바꾼 레이어도 묻는다.증거
고치기 전
LayersPanel.tsx로 새 시험을 돌리면 묻는 시험 둘이 빨개져요.고친 판에서
web/의 다섯 줄 끝:roadmap은gh로그인이 없는 컨테이너라skipped예요.npm ci가 알리는 audit 25건은 잠금 파일 그대로의 것이에요.빌드한 사이트(
next start)를 Chromium 으로 열어 직접 눌렀어요. 두 폭 다 같은 줄이 나왔어요.사진은 주황 선을 그은 Pixel 1 에서 Del 을 누르고 취소한 뒤예요. 레이어와 선이 그대로 남았어요.
실행법
가정
window.confirm을 썼어요. Lab 의 Director 와 Capture 가 이미 그렇게 물어요..test.tsx예요. ROUND.md 는*.test.ts라고 적었지만 패널을 그리려면 JSX 가 들고,vitest.config.ts는src/**/*.test.{ts,tsx}를 다 찾아요. 같은 폴더의DirectorPanel.test.tsx,PixelPanel.test.tsx도 그래요.dialog이벤트로 받아 위에 적었어요.Changelog
Web
바퀴
바퀴 3 · 후보 3 「Pattern Lab 의 Del 이 레이어를 지우기 전에 한 번 묻는다」
다음 바퀴
preventDefault()가 먹지 않아요. 같은 Lab 의useTimelineZoom.ts에 고친 꼴이 있어요.prefers-reduced-motion을 보지 않아요.🤖 Generated with Claude Code
https://claude.ai/code/session_01BL8iimZp7KVXdh7mzkP7ex
Generated by Claude Code