Repository navigation
[DONE] 저널 CSS 에서 아무것도 그리지 않던 규칙 61줄이 빠지고, 저널 화면은 그대로다 - #315
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gdqi6PP5SVyTK6YeNSu1Pn
.journal-list-number and .journal-figure, -wide, -full name classes no component or post writes; the span rules in the two cover slots and the slot's ::before switch-offs never had anything to style. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gdqi6PP5SVyTK6YeNSu1Pn
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gdqi6PP5SVyTK6YeNSu1Pn
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gdqi6PP5SVyTK6YeNSu1Pn
patternflow-bot
marked this pull request as ready for review
October 27, 2026 19:03
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 28, 2026
engmung
pushed a commit
to engmung/Patternflow
that referenced
this pull request
Oct 28, 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_01Gdqi6PP5SVyTK6YeNSu1Pn
저널 CSS 에서 아무것도 그리지 않던 규칙 61줄을 걷었어요. 어느 컴포넌트도 글도 쓰지 않는 클래스 넷, 표지 칸에 생기지 않는 span 규칙 둘, 아무도 켜지 않은 ::before 를 끄던 규칙 둘이에요. 저널 76쪽을 390 과 1440 에서 다 견주니 모든 요소의 계산된 스타일과 상자가 dev 와 같았고, 화면은 그대로예요.
좋아지는 것: 없음
체크리스트
141바퀴([DONE] 데스크톱 저널 목록에서 날짜가 두 줄로 꺾이지 않는다 #306[DONE] 저널 그림을 크게 보고 닫아도 읽던 자리가 그대로고, Tab 이 창 뒤로 새지 않는다 #314)와 겹치지 않음: 그 바퀴들은 이 규칙들을 건드리지 않았고, 135바퀴부터 일곱 바퀴가 「다음 바퀴」에 이 일을 적었어요web/src와 글 74편에서 세기, 빌드한 CSS 비교)한 일
web/src/app/globals.css에서 저널 규칙 61줄을 걷었어요..journal-list-number: mono 묶음의 한 줄과 색 규칙 하나. 목록의 번호는.pf-ghost예요..journal-figure,.journal-figure-wide,.journal-figure-full과 그img,video,figcaption, 폰 규칙 하나. 글 74편의 그림은journal-md-figure와journal-figure-row만 써요..journal-index-thumb span,.journal-cover-slot span. 두 칸엔JournalImage하나만 들고,next/image는 맨<img>하나만 그려요..journal-cover-slot::before { content: none }과.journal-hero-cover .journal-cover-slot::before { content: none; display: none }. 이::before에 content 를 주는 규칙은 없어요.새 시험 셋은
web/src/components/journal/journalCss.test.ts끝에 더했어요. globals.css 가 부르는journal-클래스가 다web/src의 코드나 글에 있는지, 표지 칸 안 요소 규칙이JournalImage가 그리는 태그만 부르는지,content: none으로 끄는 저널::before마다 켜는 규칙이 있는지 봐요. 화면 글과 마크업은 그대로예요.증거
dev 의 globals.css 에 새 시험을 돌리면
고친 뒤엔
Tests 14 passed (14)이고,src/components/journal/네 파일이25 passed (25)예요.빌드한 CSS 는 56,781 바이트에서 55,756 바이트가 됐어요. 두 빌드의 CSS 를 규칙마다 줄을 나눠
diff하면 걷은 열한 줄과 mono 묶음의 한 줄 말고는 같아요.화면이 그대로인지는 dev 빌드(3200)와 고친 빌드(3100)를 같이 띄워 쟀어요. 저널 76쪽(목록 둘, 글 74)을 390 과 1440 에서 열고, 그림을 다 받은 뒤
<body>아래 모든 요소의 상자와 계산된 스타일 전부(요소,::before,::after)를 적었어요. 한 쪽을 열어 둔 채로 고친 판 CSS, dev CSS, 고친 판 CSS 를 차례로 같은 origin 의<style>로 끼워 세 번 쟀어요..journal-shell에letter-spacing: .001em을 더하면 달라짐content: none인::before는 상자가 없어서 「상자 없음」으로 적었어요. 그 자리에서 달라지는 값은 하나예요./journal/refocus1440 의.journal-cover-slot::before는 dev 가display: none, 고친 판이display: inline이고, 둘 다content: none이라 그려지지 않아요.사진 여덟 장(목록과
/journal/refocus, 390 · 1440, 전후)은 전후가 바이트까지 같았어요. 위 셋은 고친 뒤, 아래는 나머지예요.다섯 검사(
web/에서):links-absolute 와 roadmap 은 이 컨테이너의 gh 로 원래 레포를 읽지 못해 건너뛰었어요. PR 에서 drift.yml 이 다시 돌려요.
실행법
비교 스크립트는 레포 밖 스크래치 폴더에서 컨테이너의 Playwright 와 Chromium 으로 돌렸고 레포에 두지 않았어요.
가정
::before를 끄기만 하는 두 규칙을 더했어요. 같은 표지 칸 규칙이고 아무것도 바꾸지 않아서예요..journal-cover-slot과.journal-hero-cover .journal-cover-slot처럼 둘 다 그려지지만 겹치는 규칙은 그대로 뒀어요. 지우면 값이 옮겨 가는 일이라 이번 정리와 따로예요.<script>자리가 열 때마다 바뀌고,/journal/wins-and-losses-next-step/en1440 은 글<article>의 계산된 margin 이 열 때마다 198px 이거나 0px 였어요. 상자는 둘 다 x 246, 폭 948 로 같아서 화면엔 보이지 않아요. 그래서 한 쪽 안에서 CSS 만 갈아 끼우는 비교를 썼어요.incoming/과 저널 글, 그림, 화면 글은 그대로예요.Changelog
Web
.journal-list-numberor.journal-figurewith its-wideand-full, the cover slots never held thespantwo rules styled, and two rules switched off a::beforenothing switched on. Every journal page renders as before.경험
언제: 저널이나 사이트의 CSS 만 고친 뒤 화면이 그대로인지 두 빌드로 견줄 때
교훈: 두 빌드를 따로 열어 계산된 스타일을 견주지 말고, 한 쪽을 연 채 두 빌드의 CSS 를
url(../media/를/_next/static/media/로 바꿔 같은 origin 의<style>로 갈아 끼워 견줘요. 다른 origin 의<link>로 바꾸면 글꼴을 다시 받아 글자 폭이 0.04px 달라져요근거: #315 에서 따로 연 dev 와 dev 가 152번 가운데 16번 달랐고(흘려 넣는 script 자리, article 의 auto margin), CSS 를 갈아 끼운 비교는 대조가 152번 모두 같았어요
바퀴
바퀴 142 · 후보: 저널 — 고친 곳: web/src/app/globals.css(.journal-list-number, .journal-figure 와 -wide · -full, .journal-index-thumb span, .journal-cover-slot span 과 ::before) · 까닭: 저널 76쪽 어디서도 맞는 요소가 없거나 아무것도 바꾸지 않는 규칙이 61줄이었다
남은 것
globals.css:1646의.journal-lightbox-close예요.ko·en이 17.7×18.8px, 글 쪽 맨 위 「All writing」 이 114.1×17.0px 이에요. 1440 에선.journal-lockup이 판 폭 1180×88px 를 다 차지해서 「Patternflow」 오른쪽 빈 곳을 눌러도 첫 화면으로 가요(390 은 342×50px)..journal-cover-slot(globals.css:1249)은.journal-hero-cover안에서만 그려지는데,:1425가 그 margin-top, display, overflow, border, background 를 다 덮어써서 듣는 값은position: relative하나예요. align-items 와 justify-content 는 block 에선 듣지 않아요..journal-cover-slot img(:1259)와:1435도 display, width, height, max-width 를 둘 다 적어요. 한 쌍으로 합치면 돼요.https://claude.ai/code/session_01Gdqi6PP5SVyTK6YeNSu1Pn
Generated by Claude Code