Skip to content

fix(home): 수강신청 기간 패널 SNU와 시각 매칭#111

Merged
user983740 merged 2 commits into
mainfrom
fix/period-panel-snu-match
Apr 19, 2026
Merged

fix(home): 수강신청 기간 패널 SNU와 시각 매칭#111
user983740 merged 2 commits into
mainfrom
fix/period-panel-snu-match

Conversation

@user983740

Copy link
Copy Markdown
Collaborator

Summary

Playwright로 SNU 패널의 computed style과 DOM 치수를 전부 비교한 뒤 주요 차이 반영.

  • panel: bg transparent, border #c7d1de, margin-bottom 15px, color #333, font-family에 Arial/Tahoma fallback 추가
  • panel 레벨: letter-spacing -1px, word-break keep-all 상속
  • panelHead/periodBody padding을 SNU 내부 레이아웃(32px 40px 컨테이너 기준)과 동일하게
  • 타이틀 19px → 20px, line-height 26px
  • th padding 14px 18px → 10px 5px, line-height 24px, vertical-align baseline
  • 시간 컬럼 18% → 19%
  • td:first-child padding-left 18px → 15px
  • td에 `overflow-wrap: break-word` 추가 — 긴 한글 단어(예: "정규학기수강취소)")가 셀 경계를 넘어 파란 배경 밖으로 튀어나오던 문제 수정

제외: 패널 전체 폭 (768 vs SNU 790)은 home 그리드 레이아웃 제약 (`containerX` max 1180 + 우측 380px). 내부 치수·스타일은 거의 일치 (td4 높이 261px 동일).

Test plan

  • 배포 후 https://snuclear.wafflestudio.com/ 메인 기간 패널 육안 확인
  • 대상 칸 텍스트가 cell 경계(파란 배경) 안에 머무는지
  • 타이포그래피(제목 20px, 표 line-height 24px) SNU와 일치

🤖 Generated with Claude Code

user983740 and others added 2 commits April 19, 2026 18:39
Playwright로 SNU 패널과 computed style diff 수행 후 주요 차이 반영:

- panel: bg transparent, border-color #c7d1de, margin-bottom 15px, color #333
- panel 레벨: letter-spacing -1px, word-break keep-all 상속
- font-family에 Arial, Tahoma fallback 추가 (SNU와 동일)
- panelHead padding 32px 40px 0 / periodBody padding 0 40px 32px + margin-top 23px
- 타이틀 font-size 19px → 20px, line-height 26px
- th padding 14px 18px → 10px 5px, line-height 24px, vertical-align baseline
- 시간 컬럼 width 18% → 19% (SNU 비율)
- td:first-child padding-left 18px → 15px
- currentPeriod 행 하이라이트 제거 (SNU엔 없음)

결과: 셀 높이 260.5px 동일, 전체 패널 높이 422 vs SNU 426 (4px 차).
남은 폭 차 22px은 home 그리드 레이아웃 제약.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- td에 overflow-wrap: break-word 추가 (데스크탑/모바일)
  → "정규학기수강취소)" 같은 긴 CJK 단어가 셀을 넘쳐 파란 배경 밖으로
    튀어나오던 문제 수정. SNU도 동일 속성 사용 중.
- currentPeriod 행 하이라이트 #eef5ff 복원 (SNU엔 없지만 UX상 유지)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@user983740
user983740 merged commit 2ea1cee into main Apr 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant