Skip to content

[Fix] 탐색 QA 반영#186

Merged
nahy-512 merged 3 commits intodevelopfrom
fix#179-explore-qa
Jan 23, 2026
Merged

[Fix] 탐색 QA 반영#186
nahy-512 merged 3 commits intodevelopfrom
fix#179-explore-qa

Conversation

@giovannijunseokim
Copy link
Contributor

@giovannijunseokim giovannijunseokim commented Jan 23, 2026

📮 관련 이슈

📌 작업 내용

  • Flint 버튼 텍스트 weight 계산 로직 수정

Summary by CodeRabbit

  • 리팩토링

    • 버튼 텍스트 스타일이 버튼 상태 기반으로 결정되도록 개선했습니다.
    • 컬렉션 필드명(title/description)을 contentTitle/contentDescription로 변경하고 관련 데이터와 화면 호출부를 일관되게 업데이트했습니다.
  • 새 기능

    • 탐색 화면 카드에 전체 크기 수직 그라데이션 배경을 적용해 시각적 표현을 개선했습니다.

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

- `FlintButtonState`에서 `enabled` 파라미터 제거
- `FlintBasicButton`에서 텍스트 스타일 결정 조건을 `enabled`에서 `state == FlintButtonState.Able`로 변경
@giovannijunseokim giovannijunseokim self-assigned this Jan 23, 2026
@giovannijunseokim giovannijunseokim requested a review from a team as a code owner January 23, 2026 06:24
@giovannijunseokim giovannijunseokim added the Fix 🐛 버그, UI 오류, 오타, 명세와의 불일치 등을 수정 label Jan 23, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

Flint 버튼 상태에서 enabled 생성자 파라미터를 제거하고 버튼 텍스트 스타일 선택을 상태 기반으로 변경했습니다. 컬렉션 관련 DTO·도메인 모델 필드 title/descriptioncontentTitle/contentDescription로 변경하고 매퍼 및 UI 호출부(ExploreScreen) 전반을 갱신했습니다.

Changes

Cohort / File(s) 변경 요약
버튼 컴포넌트
app/src/main/java/com/flint/core/designsystem/component/button/FlintButtonState.kt, app/src/main/java/com/flint/core/designsystem/component/button/FlintBasicButton.kt
FlintButtonState에서 enabled: Boolean 생성자 파라미터 제거(모든 enum 상수를 매개변수 없는 형태로 변경). FlintBasicButton의 텍스트 스타일 선택 로직을 enabled 대신 state == FlintButtonState.Able 조건으로 변경( Able → body1Sb16, 그 외 → body1M16).
컬렉션 DTO / 도메인 / 매퍼
app/src/main/java/com/flint/data/dto/collection/response/CollectionsResponseDto.kt, app/src/main/java/com/flint/domain/mapper/collection/CollectionMapper.kt, app/src/main/java/com/flint/domain/model/collection/CollectionsModel.kt
DTO 직렬화 필드 title/descriptioncontentTitle/contentDescription으로 변경. 도메인 모델과 매퍼에서 해당 필드명으로 매핑 및 데이터 클래스 프로퍼티 이름 갱신.
탐색 화면 UI 업데이트
app/src/main/java/com/flint/presentation/explore/ExploreScreen.kt
ExplorePageItem 시그니처 및 호출부(프리뷰 포함)에서 title/descriptioncontentTitle/contentDescription로 변경. 그에 따른 텍스트 바인딩 및 샘플 데이터 업데이트. 그라디언트 배경 관련 레이아웃 조정 포함.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

🧩 Component, 📱 UI

Suggested reviewers

  • nahy-512

Poem

🐰 깡총, 필드 이름 바꿨다네,
contentTitle로 또르르 부르네.
버튼도 상태로 폰트를 고르네,
당근 한 토막 축하해요 🥕✨

🚥 Pre-merge checks | ❌ 5
❌ Failed checks (2 warnings, 3 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 매우 불완전합니다. '작업 내용'에는 버튼 텍스트 weight 계산 로직만 언급되어 있으나, 실제 변경사항(FlintButtonState 구조 변경, 필드명 변경 등)이 누락되어 있습니다. PR 설명을 보완하세요. 모든 주요 변경사항(FlintButtonState 리팩토링, CollectionsResponseDto/Model 필드명 변경, ExploreScreen 업데이트, 그라데이션 추가)을 작업 내용에 포함시키세요.
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.
Title check ❓ Inconclusive PR 제목이 '[Fix] 탐색 QA 반영'으로 일반적이며, 실제 변경 사항(버튼 텍스트 스타일, 필드명 변경 등)을 구체적으로 설명하지 못하고 있습니다. PR 제목을 더 구체적으로 수정하세요. 예: '[Fix] 버튼 상태별 텍스트 스타일 및 컬렉션 필드명 변경' 또는 주요 변경 내용을 반영하는 제목으로 개선하세요.
Linked Issues check ❓ Inconclusive 연결된 이슈 #179는 '탐색 화면 QA 반영'이지만 구체적인 요구사항이 제공되지 않아, PR 변경사항이 이슈의 목표를 충족하는지 검증하기 어렵습니다. 이슈 #179의 구체적인 요구사항을 확인하거나, PR 설명에서 각 변경사항이 어떤 QA 항목을 해결하는지 명시하세요.
Out of Scope Changes check ❓ Inconclusive 수정: ExploreScreen에서 그라데이션 배경 추가 및 레이아웃 변경(fillMaxSize 사용)이 버튼 텍스트 weight 계산과 직접 관련이 없어 보이며, FlintButtonState 구조 변경(enabled 제거)이 확인됩니다. ExploreScreen 그라데이션 추가 변경과 FlintButtonState 구조 변경이 이슈 #179의 범위에 포함되는지 확인하세요.

✏️ 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.

- `CollectionsModel` 및 `CollectionsResponseDto` 내 `title`, `description` 필드명을 각각 `contentTitle`, `contentDescription`으로 변경
- `ExploreScreen` 및 `CollectionMapper`에서 변경된 필드명을 사용하도록 수정
- `imgBlur` 브러쉬를 커스텀 `verticalGradient`로 변경
- 배경 영역을 `aspectRatio` 기반에서 `fillMaxSize`로 확장
- 불필요한 `aspectRatio` import 제거
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.

LGTM 👍🏻

@nahy-512 nahy-512 merged commit 9918402 into develop Jan 23, 2026
2 checks passed
@nahy-512 nahy-512 deleted the fix#179-explore-qa branch January 23, 2026 07:58
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 반영

2 participants

Comments