Skip to content

[FIX] 캘린더 팝업 하단 날짜 잘림 수정 - #220

Merged
kimminna merged 1 commit into
developfrom
fix/ui/218-calendar-popup-clipping
Jul 15, 2026
Merged

[FIX] 캘린더 팝업 하단 날짜 잘림 수정 #220
kimminna merged 1 commit into
developfrom
fix/ui/218-calendar-popup-clipping

Conversation

@yumin-kim2

@yumin-kim2 yumin-kim2 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

ISSUE 🔗

close #218



What is this PR? 🔍

캘린더 팝업에서 6주차까지 표시되는 달의 하단 날짜가 잘리는 문제를 수정했습니다! (height 고정값과 row 6 고정값 삭제)


To Reviewers

August 2026처럼 6주차가 있는 달에서도 마지막 주 날짜가 팝업 내부에 정상적으로 보이는지 확인 부탁드립니다.



Screenshot 📷

image image



Test Checklist ✔

  • 5주차 달에서 캘린더 UI가 기존처럼 표시되는지 확인
  • 6주차 달에서 하단 날짜가 잘리지 않는지 확인

@yumin-kim2 yumin-kim2 linked an issue Jul 15, 2026 that may be closed by this pull request
@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
timo Ready Ready Preview, Comment Jul 15, 2026 6:33am

Request Review

@github-actions github-actions Bot added ⌚ Timo-Design-system Timo 디자인 시스템 🐛 Bug 기능이 정상적으로 작동하지 않는 문제 수정 ♣️ 유민 유민양 labels Jul 15, 2026
@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

Calendar의 고정 높이와 6행 고정 설정을 제거하고, 날짜 그리드에 grid 클래스를 추가해 6주가 표시되는 달력 레이아웃을 조정했습니다.

Changes

달력 레이아웃 수정

Layer / File(s) Summary
달력 컨테이너 및 그리드 레이아웃
packages/timo-design-system/src/components/calendar/Calendar.tsx
외곽 컨테이너에서 h-66을 제거하고, 날짜 그리드에 grid를 추가하면서 grid-rows-6 설정을 제거했습니다.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • Team-Timo/Timo-client#119: Calendar.tsx의 렌더링 및 그리드 스타일 변경과 직접적으로 관련되어 있습니다.

Suggested reviewers: ehye1, jjangminii, kimminna

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 캘린더 팝업 하단 날짜 잘림 수정이라는 핵심 변경을 정확히 요약합니다.
Description check ✅ Passed 설명은 6주차 달에서 하단 날짜가 잘리는 문제와 높이·행 고정 제거를 직접 설명합니다.
Linked Issues check ✅ Passed 직접 링크된 #218의 요구사항인 5주·6주 구성에서 날짜가 팝업 안에 온전히 보이도록 하는 수정과 일치합니다.
Out of Scope Changes check ✅ Passed 변경은 Calendar 레이아웃 클래스 조정에 한정되어 있어 요구사항 밖의 변경이 보이지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ui/218-calendar-popup-clipping

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Storybook Preview

항목 링크
Storybook 열기
Chromatic 빌드 확인

마지막 업데이트: 2026-07-15 06:34 UTC

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/timo-design-system/src/components/calendar/Calendar.tsx`:
- Line 91: Calendar.stories.tsx에 5주 월과 6주 월을 각각 고정하는 Storybook 또는 테스트 케이스를
추가하세요. 기존 2026년 7월 케이스에 더해 2026년 8월을 포함하고, DateSelector에서 Dropdown.Panel을 연 상태로
각 달의 마지막 행이 잘리지 않고 표시되는지 검증하세요.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 5258964f-69f3-48a1-a792-a642d96981be

📥 Commits

Reviewing files that changed from the base of the PR and between fef3dcb and 155f618.

📒 Files selected for processing (1)
  • packages/timo-design-system/src/components/calendar/Calendar.tsx

<div
className={cn(
"shadow-timo h-66 w-67 rounded-[4px] bg-white px-6 py-4",
"shadow-timo w-67 rounded-[4px] bg-white px-6 py-4",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win

5주·6주 달력 회귀 케이스를 추가해 주세요.

Line 91과 Line 118의 변경으로 높이와 행 제약은 완화되었지만, 현재 Calendar.stories.tsxnew Date(2026, 6, 1)(2026년 7월)만 렌더링해 6주 달인 2026년 8월을 검증하지 않습니다. 5주·6주 월을 각각 Storybook 또는 테스트로 고정하고, DateSelector → Dropdown.Panel 내부에서도 마지막 행이 잘리지 않는지 확인해 주세요.

암시적 Grid 행 동작은 Tailwind CSS 공식 문서를 참고할 수 있습니다. (tailwindcss.com)

Also applies to: 118-118

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/timo-design-system/src/components/calendar/Calendar.tsx` at line 91,
Calendar.stories.tsx에 5주 월과 6주 월을 각각 고정하는 Storybook 또는 테스트 케이스를 추가하세요. 기존 2026년
7월 케이스에 더해 2026년 8월을 포함하고, DateSelector에서 Dropdown.Panel을 연 상태로 각 달의 마지막 행이 잘리지
않고 표시되는지 검증하세요.

Source: MCP tools

@github-actions

Copy link
Copy Markdown

Timo Performance Report

Bundle Size — timo-web
라우트 크기 First Load JS
/[locale]/home 209.85 kB 🔴 415.71 kB
/[locale]/today 193.96 kB 🔴 399.82 kB
/[locale]/focus 160.04 kB 🔴 365.90 kB
/[locale]/settings 166.58 kB 🔴 372.44 kB
/[locale]/statistics 232.91 kB 🔴 438.77 kB
/[locale]/[...rest] 0 B 🟡 205.86 kB
/[locale]/login 212.94 kB 🔴 418.79 kB
/[locale]/oauth/callback 119.65 kB 🟡 325.51 kB
/[locale]/onboarding 232.88 kB 🔴 438.74 kB
/[locale] 118.96 kB 🟡 324.82 kB
/[locale]/policy 125.09 kB 🟡 330.94 kB

공유 번들: 205.86 kB
🟢 < 200kB  |  🟡 < 350kB  |  🔴 ≥ 350kB (First Load JS · gzip)

Lighthouse — timo-web
URL Perf A11y LCP CLS TBT
/en/home 🔴 60 🟢 95 🔴 15.8s 🟢 0.000 🟡 547ms
/en/today 🔴 59 🟢 95 🔴 15.5s 🟢 0.000 🟡 585ms
/en/focus 🔴 61 🟢 95 🔴 15.3s 🟢 0.000 🟡 542ms
/en/statistics 🔴 63 🟢 95 🔴 15.3s 🟢 0.000 🟡 487ms

Perf ≥ 70 / A11y ≥ 85 목표
LCP 🟢 < 2.5s 🟡 < 4s 🔴 ≥ 4s  |  CLS 🟢 < 0.1 🟡 < 0.25 🔴 ≥ 0.25  |  TBT 🟢 < 200ms 🟡 < 600ms 🔴 ≥ 600ms

Image Optimization — timo-web
파일 크기 포맷 상태
favicon.png 27.84 kB PNG ⚠️ 🟢
images/google-calendar.png 36.20 kB PNG ⚠️ 🟢
images/google-logo.png 26.79 kB PNG ⚠️ 🟢

총 3개 · 90.84 kB  |  🟢 < 200KB  |  🟡 < 500KB  |  🔴 ≥ 500KB
⚠️ 3개 파일 WebP/AVIF 변환 권장

측정 커밋: c28a2b4

@kimminna
kimminna merged commit 8285644 into develop Jul 15, 2026
27 checks passed
@kimminna
kimminna deleted the fix/ui/218-calendar-popup-clipping branch July 15, 2026 06:41
@kimminna kimminna mentioned this pull request Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⌚ Timo-Design-system Timo 디자인 시스템 ♣️ 유민 유민양 🐛 Bug 기능이 정상적으로 작동하지 않는 문제 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 공통 컴포넌트 캘린더 오류 수정

2 participants