Feat/#103 디자인 QA

[DongChyeon]

🛠 Related issue

closed #103

어떤 변경사항이 있었나요?

• 🐞 BugFix Something isn't working
• 🎨 Design Markup & styling

✅ CheckPoint

• PR 컨벤션에 맞게 작성했습니다. (필수)
• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
• BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

✏️ Work Description

• 홈 화면 카테고리 리스트 패딩 및 간격 조정
• 홈 화면 EmptyView 상단 간격 조정
• ActionSheet 디자인 수정 (contentPadding 및 아이템 고정 높이 제거 → vertical padding으로 교체)
• ActionPopup 내부 여백 조정 (vertical padding 추가)
• NotificationScreen 필터 간격 수정 (8dp → 6dp)
• 업로드 화면 사진 선택 시 stroke 추가
• HomeHeader에서 미사용 selectedFilter 파라미터 제거
• 링크 툴팁 dismiss 상태를 ViewModel로 이관하여 세션 내 재노출 버그 수정
  • 버그 원인: FeedCard 내 remember(showProductLinkTooltip) 키가 true인 채로 유지되어 LazyColumn 스크롤 후 재진입 시 툴팁 재초기화
  • 수정: dismiss 시 HomeIntent.DismissTooltip 전달 → ViewModel isTooltipDismissed = true 유지 (앱 재시작 시 초기화)

😅 Uncompleted Tasks

N/A

📢 To Reviewers

• 툴팁 버그 수정은 HomeContract, HomeViewModel, HomeScreen, FeedCard 4개 파일에 걸쳐 있습니다.
• 세션(앱 프로세스 생존 기간) 동안 dismiss 상태가 유지되며, 앱 재시작 시 ViewModel 재생성으로 자동 초기화됩니다.

📃 RCA 룰

• R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
• C: 웬만하면 반영해 주세요. (Comment)
• A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Summary by CodeRabbit

릴리스 노트

• 새 기능

  • 제품 링크 툴팁 해제 기능 추가

• 스타일

  • 액션 팝업 및 액션 시트 레이아웃 패딩 조정
  • 피드 필터 칩 간격 6dp로 통일
  • 알림 화면 필터 행 간격 조정
  • 업로드 화면 이미지 미리보기에 테두리 추가

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: f9f77ee2-91f3-4849-9101-4243653a4a82

📥 Commits

Reviewing files that changed from the base of the PR and between 94ac557 and dc6c4a8.

📒 Files selected for processing (8)

• core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/ActionPopup.kt
• core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/ActionSheet.kt
• core/designsystem/src/main/java/com/sseotdabwa/buyornot/core/designsystem/components/FeedCard.kt
• feature/home/src/main/java/com/sseotdabwa/buyornot/feature/home/ui/HomeContract.kt
• feature/home/src/main/java/com/sseotdabwa/buyornot/feature/home/ui/HomeScreen.kt
• feature/home/src/main/java/com/sseotdabwa/buyornot/feature/home/ui/HomeViewModel.kt
• feature/notification/src/main/java/com/sseotdabwa/buyornot/feature/notification/ui/NotificationScreen.kt
• feature/upload/src/main/java/com/sseotdabwa/buyornot/feature/upload/ui/UploadScreen.kt

---

개요

이 PR은 디자인 QA를 통해 컴포넌트 간격, 여백, 경계선 등을 조정하고, 피드 카드의 툴팁 해제 콜백을 추가하여 상태 추적 기능을 구현합니다.

변경 내용

툴팁 해제 기능

계층 / 파일	요약
상태 및 인텐트 정의 feature/home/.../HomeContract.kt	HomeUiState에 isTooltipDismissed: Boolean 추가, HomeIntent에 DismissTooltip 인텐트 추가
디자인 컴포넌트 업데이트 core/designsystem/.../FeedCard.kt	onTooltipDismiss: () -> Unit = {} 파라미터 추가, 해제 시 콜백 호출
화면 상태 관리 feature/home/.../HomeScreen.kt	로컬 showLinkTooltip 상태를 uiState.isTooltipDismissed 기반 계산으로 변경, FeedItemCard를 통해 콜백 전달
ViewModel 인텐트 처리 feature/home/.../HomeViewModel.kt	DismissTooltip 인텐트 처리 로직 추가, 상태를 isTooltipDismissed = true로 업데이트

디자인 시스템 간격 조정

계층 / 파일	요약
팝업 컴포넌트 core/designsystem/.../ActionPopup.kt	메뉴 컨테이너 패딩을 균일 6.dp에서 horizontal = 6.dp, vertical = 10.dp로 변경
액션 시트 컴포넌트 core/designsystem/.../ActionSheet.kt	LazyColumn contentPadding을 top = 26.dp, bottom = 30.dp로 변경, ActionItemRow의 고정 높이 제거 및 패딩을 vertical = 6.dp 추가
화면 필터 칩 레이아웃 feature/home/.../HomeScreen.kt	필터 칩 행의 contentPadding과 horizontalArrangement 간격을 8.dp에서 6.dp로 조정, 빈 상태 화면 수직 패딩 조정
알림 화면 필터 feature/notification/.../NotificationScreen.kt	NotificationFilterRow의 칩 간 간격을 8.dp에서 6.dp로 변경

업로드 화면 경계선 스타일

계층 / 파일	요약
이미지 미리보기 스타일 feature/upload/.../UploadScreen.kt	SelectedImagePreview에 1dp 둥근 경계선(gray300, 12dp 모서리) 추가

예상 코드 리뷰 난이도

🎯 2 (Simple) | ⏱️ ~12분

변경 사항이 주로 UI 간격 조정, 스타일 개선, 그리고 명확한 상태 추적 콜백 추가로 구성되어 있습니다. 각 변경이 독립적이며 로직 복잡도가 낮아 검토 시간이 적게 소요됩니다.

관련 PR

• Nexters/BuyOrNot-Android#34: 알림 화면의 필터 행 간격 조정이 겹쳐 있습니다.
• Nexters/BuyOrNot-Android#97: ActionSheet 컴포넌트의 레이아웃/패딩 조정이 직접 관련됩니다.
• Nexters/BuyOrNot-Android#57: FeedCard 공개 API 변경이 겹쳐 있습니다.

제안 레이블

✨ FEAT, 🎨 DESIGN

제안 리뷰어

• Imagine-Choi

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 40.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 내용의 주요 부분과 관련이 있으나, 연결된 이슈 #103의 '디자인 QA' 개선 작업을 다루고 있습니다. 제목은 구체적이지만, 여러 파일의 다양한 변경사항(padding, spacing, tooltip 상태 관리 등)을 포괄하므로 변경의 전체 범위를 명확히 반영합니다.
Linked Issues check	✅ Passed	PR의 모든 코드 변경사항이 이슈 #103의 목표와 일치합니다. 컴포넌트 간격(ActionSheet, ActionPopup, HomeScreen, NotificationScreen), padding/margin(ActionSheet, FeedCard, HomeScreen), stroke 추가(UploadScreen) 등이 디자인 QA 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	tooltip 상태 관리를 로컬에서 ViewModel로 이동한 변경사항은 이슈 #103의 기술적 개선(버그 수정)으로 볼 수 있으나, 디자인 QA의 시각적 메트릭 조정 범위를 약간 벗어납니다. 그러나 PR 목표에서 명시되었으므로 의도된 범위 내입니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/#103-design-qa

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

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.}