[BOM-491] fix: 고정폭 이메일 아티클 본문이 하단 영역을 침범하는 버그 수정#292
Open
JeLee-river wants to merge 1 commit into
Hidden character warning
The head ref may contain hidden characters: "BOM-491-\ub274\uc2a4\ub808\ud130\uac00-\uae68\uc9c0\ub294-\ubb38\uc81c-\uc218\uc815"
Open
Conversation
jaeyoung-kwon
approved these changes
Jul 14, 2026
jaeyoung-kwon
left a comment
Contributor
There was a problem hiding this comment.
🤖 PR Review
✅ 확실하게 수정이 필요한 항목을 찾지 못했습니다.
고정폭 이메일 아티클이 스케일 축소 시 하단 영역을 침범하던 버그를 layoutRef/contentRef 분리와 ResizeObserver 기반 재계산으로 수정한 변경입니다. Container(overflow:hidden, 높이 예약)와 Content(scale 적용) 분리 구조가 이중 축소를 방지하고, scrollWidth 기반 측정이 CSS transform에 영향받지 않는 점도 정확합니다. ArticleBodyWrapper의 calc(100% + 12px)/-6px 마진 계산도 PC_HORIZONTAL_PADDING 상수 값(12px)과 일치해 본문 패딩만 6px로 줄어드는 의도대로 동작합니다. MSW 목 데이터 추가와 tsconfig resolveJsonModule 설정도 테스트/빌드에 필요한 범위로 문제없습니다. production 코드 경로에서 introduced된 critical/major 이슈는 발견하지 못했습니다.
🚨 0 Critical ·
📋 검증 과정
- Claude structured review 결과 중 확신도가 있는 항목만 정리했습니다.
- Critical/Major는 inline comment로 게시하고, Minor는 참고 항목으로 summary에 포함합니다.
- 자동 생성된 OpenAPI 타입 선언 파일과 lock 파일은 리뷰 대상에서 제외합니다.
🤖 Claude PR Review
Generated by Claude Code
JeLee-river
force-pushed
the
BOM-491-뉴스레터가-깨지는-문제-수정
branch
from
July 14, 2026 11:19
6a53537 to
f6ddfcd
Compare
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.
📌 What
[개선 전 - 개선후]


❓ Why
transform: scale()로 본문을 축소하더라도, 축소 전 높이로 레이아웃이 예약돼 하단 콘텐츠를 덮는 문제가 발생함🔧 How
useAutoScaleContent를layoutRef/contentRef2-ref 구조로 재작성clientWidth(이미 viewport에 잘린 폭)로 scale 계산 → 사실상 0.94x만 적용됨scrollWidth(콘텐츠 실제 폭)로 측정해 올바른 scale(예: 600px→390px = 0.65x) 계산layoutRef에만style.height예약,contentRef에만transform: scale()적용해 이중 축소 방지ResizeObserver로 이미지 로드 후 높이 변화 감지 →layoutRef.style.height재계산ArticleContent를Container(overflow:hidden, 높이 예약) /Content(scale 대상) 구조로 분리useAutoScaleContent인자를 순서 기반 파라미터에서 객체({ layoutRef, contentRef })로 변경해 역할 혼동 방지👀 Review Point