Skip to content

fix/#188 컬렉션 생성 시 빈 값 전송 및 문구 수정#192

Merged
nahy-512 merged 3 commits intodevelopfrom
fix/#188-collection-create-QA
Jan 23, 2026
Merged

fix/#188 컬렉션 생성 시 빈 값 전송 및 문구 수정#192
nahy-512 merged 3 commits intodevelopfrom
fix/#188-collection-create-QA

Conversation

@chanmi1125
Copy link
Contributor

@chanmi1125 chanmi1125 commented Jan 23, 2026

📮 관련 이슈

📌 작업 내용

  • 컬렉션 생성 시 빈 값 전송 및 문구 수정

📸 스크린샷

스크린샷

😅 미구현

  • [ ]

🫛 To. 리뷰어

Summary by CodeRabbit

  • UI 개선

    • 컬렉션 생성 안내 문구가 "작품을 2개 이상 추가해주세요"로 업데이트되었습니다.
    • 버튼·레이아웃의 간격 조정으로 화면 표현이 더 일관되게 개선되었습니다.
    • 화면 외부 터치 시 입력 포커스가 해제되어 전체 화면에서 포커스 제어가 향상되었습니다.
  • 버그 수정

    • 컬렉션 생성 시 제목·설명과 추천 사유의 빈 값 처리 방식이 안정화되었습니다.
  • 사용성

    • 검색/입력 완료 시 소프트 키보드가 자동으로 숨겨지도록 개선되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@chanmi1125 chanmi1125 self-assigned this Jan 23, 2026
@chanmi1125 chanmi1125 requested a review from a team as a code owner January 23, 2026 08:12
@chanmi1125 chanmi1125 added the Fix 🐛 버그, UI 오류, 오타, 명세와의 불일치 등을 수정 label Jan 23, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

컬렉션 생성/추가 화면의 안내 문구와 레이아웃 스페이싱이 수정되고, ViewModel에서 전송 payload에 UI 상태값을 사용하도록 정리했으며, 입력 키보드 제어와 전체 화면 탭으로 포커스 해제 동작이 추가되었습니다.

Changes

Cohort / File(s) 변경 요약
컬렉션 생성 화면 UI
app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateScreen.kt
헤더 안내문 변경: "최대 10개까지 추가할 수 있어요" → "작품을 2개 이상 추가해주세요."; 버튼 위 빈 줄 추가(레이아웃 간격 조정); 포맷/공백 소소 변경
ViewModel — 전송 payload 정리
app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateViewModel.kt
postCollectionCreate에서 title/description을 하드코드 더미 대신 uiState.value.titleuiState.value.description ?: "" 사용; CollectionCreateContentModel.reason 기본값으로 빈 문자열 적용
추가 콘텐츠 화면 — 키보드 제어 도입
app/src/main/java/com/flint/presentation/collectioncreate/AddContentScreen.kt
LocalSoftwareKeyboardController 도입 및 keyboardController?.hide() 호출을 통한 IME 숨김 처리(검색 액션에서)
앱 전체 포커스 해제 동작
app/src/main/java/com/flint/presentation/main/MainScreen.kt
전체 화면을 Box로 감싸고 pointerInput + detectTapGestures로 외부 탭 시 LocalFocusManager를 이용해 포커스 해제 동작 추가(기존 Scaffold 구조 유지)

Sequence Diagram(s)

(조건 미충족 — 생략)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

📱 UI

Suggested reviewers

  • nahy-512
  • giovannijunseokim

Poem

🐰 깡충, 깡충 코드 밭을 달려
텍스트 맞추고 키보드 숨겼지요
탭 한 번에 포커스 훌쩍 사라져
기본값은 깔끔히 비워두고
컬렉션에 두 작품, 출발해요 ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.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
Title check ✅ Passed 제목은 빈 값 전송 방지 및 문구 수정이라는 PR의 주요 변경사항을 명확하게 요약하고 있습니다.
Description check ✅ Passed 설명이 템플릿 구조를 따르고 있으나, 작업 내용과 스크린샷 정보가 불완전합니다.
Linked Issues check ✅ Passed 코드 변경사항은 컬렉션 생성 화면의 빈 값 전송 방지 및 문구 수정이라는 목표를 달성하고 있습니다.
Out of Scope Changes check ✅ Passed MainScreen.kt의 포커스 관리 및 키보드 숨김 처리는 컬렉션 생성 화면 UX 개선과 관련이 있으며 범위 내의 변경입니다.

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

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link
Contributor

@giovannijunseokim giovannijunseokim left a comment

Choose a reason for hiding this comment

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

LGTM 💯

리뷰 하나만 확인해주세요 !

Comment on lines 308 to 309
Copy link
Contributor

Choose a reason for hiding this comment

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

p1: 여기 변경사항 없어져도 될 것 같아요. 작품 추가하기 버튼의 레이아웃은 제 PR에서 수정했습니다!

feat: 작품 추가하기 버튼 위 아래 넓이 확인 한번 해주세요 살짝 좁은 느낌입니다.

Copy link
Contributor

@nahy-512 nahy-512 left a comment

Choose a reason for hiding this comment

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

리뷰 확인 부탁해요

Comment on lines 58 to 59
Copy link
Contributor

Choose a reason for hiding this comment

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

p1
이거 title, description이 없을 때 빈 문자열로 초기화되는 거 , ifBlank 설정을 따로 해주는 이유가 있을까요?
title은 안 들어가면 막힐거고, description은 빈 문자열 그대로 넣으면 되는 거 아닌가요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네!! 반영하겠습니다앙

…n-create-QA

# Conflicts:
#	app/src/main/java/com/flint/presentation/collectioncreate/CollectionCreateScreen.kt
@nahy-512 nahy-512 merged commit b23248a into develop Jan 23, 2026
2 checks passed
@nahy-512 nahy-512 deleted the fix/#188-collection-create-QA branch January 23, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix 🐛 버그, UI 오류, 오타, 명세와의 불일치 등을 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 컬렉션 생성 화면 QA

3 participants

Comments