Feat: 개발 QA 1차( 다운로드 url 변경)

[jjangminii]

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

• close #

📄 Tasks

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

• 스타일
  • 기능 북마크/리마인더/리워드 섹션의 레이아웃을 중앙 정렬에서 상단 정렬로 조정해, 두 열에서 텍스트와 이미지가 위에서부터 자연스럽게 배치되도록 개선했습니다. 시각적 일관성과 가독성이 향상됩니다.
  • Lottie 애니메이션 참조명을 정리해 자산 연결을 명확히 했습니다. 애니메이션 내용과 동작은 기존과 동일합니다.

[coderabbitai]

Walkthrough

Landing 페이지의 세 개 섹션 컴포넌트에서 레이아웃 정렬 클래스를 상단 정렬로 변경하고, 두 컴포넌트의 Lottie 애니메이션 import 별칭을 교체했습니다. 한 컴포넌트에서는 불필요한 justify-start 클래스를 제거했습니다. 공개 API 시그니처 변경은 없습니다.

Changes

Cohort / File(s)	Change Summary
Layout alignment updates apps/landing/src/components/FeatureBookmarkSection.tsx, apps/landing/src/components/FeatureReminderSection.tsx, apps/landing/src/components/FeatureRewardSection.tsx	컨테이너 정렬을 items-center에서 items-start로 변경하여 상단 정렬로 통일
Lottie import alias swaps apps/landing/src/components/FeatureReminderSection.tsx, apps/landing/src/components/FeatureRewardSection.tsx	Lottie 애니메이션 import 별칭 교체 및 사용처 업데이트(자산 파일은 동일)
Minor container class cleanup apps/landing/src/components/FeatureBookmarkSection.tsx	텍스트 컬럼에서 justify-start 제거(동일 동작)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Feat(landing): landing page 디테일 수정 #105: 동일 Landing 섹션과 Lottie 사용을 도입/수정한 PR과 직접적으로 연속된 레이아웃/별칭 조정.
• Feat(landing): landing page #104: 동일 컴포넌트 군(FeatureBookmark/Reminder/Reward) 초기 추가와 관련된 후속 정렬/별칭 변경.
• Api(client): 북마크, 리마인드 페이지 article delete 연결 #98: 동일 Lottie 자산과 섹션 컴포넌트 흐름을 다루는 선행 변경과 연관.

Suggested labels

feat, 정민

Suggested reviewers

• jllee000
• constantly-dev

Poem

토끼는 귀를 쫑긋, 레이아웃을 쓱 정렬해요
가운데서 위쪽으로, 줄 맞춤이 또렷해요
딸랑딸랑 벨도 이름표를 새로 달고 🛎️
업! 하고 올라가는 프레임의 춤사위 ↗️
오늘도 코드밭에 당근 같은 픽셀 수확해요 🥕

Pre-merge checks and finishing touches

❌ Failed checks (4 warnings)

Check name	Status	Explanation	Resolution
Title Check	⚠️ Warning	PR 제목 "Feat: 개발 QA 1차(정민	다운로드 url 변경)"은 제공된 변경 요약과 불일치합니다; 변경사항 요약에서는 landing 관련 컴포넌트들의 정렬 클래스 조정 및 Lottie 애셋의 로컬 이름(alias) 변경만 확인되며 "다운로드 URL 변경"과 같은 변경은 보이지 않습니다. 제목이 실제 변경 내용을 정확히 요약하지 못해 히스토리 스캔 시 오해를 유발할 가능성이 큽니다. 따라서 제목은 변경의 주된 의도를 반영하도록 수정이 필요합니다.
Linked Issues Check	⚠️ Warning	연결된 이슈(#25)는 디자인 시스템의 Progress 컴포넌트 구현(값 클램프, variant 지원, 접근성, 스토리북·테스트, @radix-ui/react-progress 의존성 추가 등)을 요구합니다. 그러나 본 PR의 변경 요약은 landing의 세 컴포넌트에서 정렬 클래스와 Lottie JSON import alias 변경만 포함하고 있어 이슈 요구사항(Progress 컴포넌트 코드, 의존성, 문서/스토리북 등)을 충족하지 않습니다. 따라서 현재 PR은 이슈 #25의 목표를 달성하지 못합니다.	이 PR이 실제로 #25를 해결하려는 의도라면 Progress 컴포넌트 구현, 런타임 의존성 추가, 스토리북·테스트 등을 포함하도록 변경을 보완하거나, 관련이 없다면 링크를 제거하고 올바른 이슈를 연결해 주세요.
Out of Scope Changes Check	⚠️ Warning	변경된 내용(세 개의 landing 컴포넌트에서 items-center → items-start로 정렬 변경 및 Lottie import alias 변경)은 linked issue의 범위(Progress 컴포넌트 구현)와 무관합니다. 일부 변경은 기능적 영향이 없는 alias명 변경처럼 보이며 PR 설명에 목적이 명확히 적혀 있지 않아 의도된 범위인지 확인이 필요합니다. 따라서 현재 PR에는 이슈 범위를 벗어난 수정이 포함되어 있습니다.	작업 범위를 명확히 하고 변경을 분리하세요; UI 정렬 및 Lottie 관련 수정은 별도 스타일/수정 PR로 만들고, 이슈 #25 관련 작업은 Progress 구현 전용 PR로 분리하여 각 PR에 적절한 설명과 테스트를 포함하도록 권장합니다.
Description Check	⚠️ Warning	PR 본문은 저장소의 템플릿을 그대로 두고 실제 변경 사항, 작업 항목, 검토 포인트 또는 스크린샷 등이 기입되어 있지 않아 리뷰어가 변경 의도와 검증 방법을 파악하기 어렵습니다. 연결된 이슈 번호도 구체적으로 지정되어 있지 않습니다. 이 상태로는 리뷰가 지연되거나 잘못된 가정 하에 검토될 위험이 있습니다.	PR 본문에 변경된 파일 목록과 요약(무엇을 왜 바꿨는지), 관련 이슈 번호, 테스트/검증 방법 및 리뷰어에게 원하는 포인트를 간단히 추가해 주세요; 필요하면 변경 전/후 스크린샷이나 동작 영상도 첨부하세요.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#102/landing-page

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[vercel]

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

Project	Deployment	Preview	Comments	Updated (UTC)
pinback-client-client	Ready	Preview	Comment	Sep 19, 2025 9:37am

[github-actions]

✅ Storybook chromatic 배포 확인:
🐿️ storybook

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

apps/landing/src/components/FeatureReminderSection.tsx (1)

2-2: 일관성을 위해 import 별칭을 더 명확하게 변경하는 것을 고려해보세요.

Bell은 더 나은 별칭이지만, 다른 파일들과의 일관성을 위해 더 구체적인 이름(예: BellAnimation)을 사용하거나 파일명과 더 일치하는 이름을 고려해볼 수 있습니다.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 72ed6bc and 2c04892.

📒 Files selected for processing (3)

• apps/landing/src/components/FeatureBookmarkSection.tsx (1 hunks)
• apps/landing/src/components/FeatureReminderSection.tsx (1 hunks)
• apps/landing/src/components/FeatureRewardSection.tsx (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: storybook

🔇 Additional comments (7)

apps/landing/src/components/FeatureReminderSection.tsx (2)

7-7: 변경사항이 올바르게 적용되었습니다.

items-center에서 items-start로 변경하여 상단 정렬을 적용한 것이 일관된 레이아웃 정렬 패턴과 잘 맞습니다.

---

10-10: 애니메이션 데이터 사용이 올바르게 업데이트되었습니다.

새로운 Bell 별칭을 사용하여 Lottie 컴포넌트가 정확히 업데이트되었습니다.

apps/landing/src/components/FeatureRewardSection.tsx (3)

2-2: import 별칭이 개선되었습니다.

Progress에서 Up으로 변경한 것이 4_up.json 파일명과 더 잘 일치합니다. 좋은 개선입니다.

---

7-7: 레이아웃 정렬이 다른 컴포넌트들과 일관되게 변경되었습니다.

items-center에서 items-start로 변경하여 전체 랜딩 페이지 섹션들의 정렬을 통일한 것이 좋습니다.

---

20-20: 애니메이션 데이터 사용이 정확히 업데이트되었습니다.

새로운 Up 별칭이 올바르게 적용되었습니다.

apps/landing/src/components/FeatureBookmarkSection.tsx (2)

7-7: 레이아웃 정렬 변경이 적절합니다.

items-center에서 items-start justify-start로 변경하여 다른 섹션들과 일관된 상단 정렬을 적용했습니다. 다만 justify-start는 flex row에서는 기본값이므로 명시적으로 추가할 필요는 없지만, 명확성을 위해 추가한 것으로 보입니다.

---

9-9: 불필요한 justify-start 제거가 올바릅니다.

flex-col에서 justify-start는 주축(세로) 방향의 정렬이므로 실제로 필요하지 않습니다. 이를 제거한 것이 코드를 더 깔끔하게 만듭니다.