Skip to content

[UI/#188] 제보 완료화면 오류사항을 수정합니다. #189

Merged
wjdrjs00 merged 2 commits intodevelopfrom
ui/#188-report-complete
Mar 1, 2026
Merged

[UI/#188] 제보 완료화면 오류사항을 수정합니다. #189
wjdrjs00 merged 2 commits intodevelopfrom
ui/#188-report-complete

Conversation

@wjdrjs00
Copy link
Member

@wjdrjs00 wjdrjs00 commented Feb 28, 2026

[ PR Content ]

스크럼 시간에 발견한 제보 완료 화면관련 문제를 피그마 명세에 맞게 수정했습니다.

Related issue

Screenshot 📸

  • N/A

Work Description

  • 제보 완료 화면 텍스트 정렬 변경 및 콘텐츠 겹침 문제 해결

To Reviewers 📢

  • 간단한 작업이라 없슴다!

Summary by CodeRabbit

릴리스 노트

  • Style
    • 리포트 카드의 텍스트 정렬을 조정했습니다.
    • 리포트 카드의 레이아웃 배치 방식을 개선했습니다.

- 버튼과 제보완료 content 영역 겹침 문제 해결을 위해 padding으로 변경
@wjdrjs00 wjdrjs00 requested a review from l5x5l February 28, 2026 00:15
@wjdrjs00 wjdrjs00 self-assigned this Feb 28, 2026
@wjdrjs00 wjdrjs00 added 📱 UI UI 추가 및 수정 (비지니스 로직을 포함하지 않는 작업) 🧤 대현 labels Feb 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dc20169 and a76b1dd.

📒 Files selected for processing (2)
  • presentation/src/main/java/com/threegap/bitnagil/presentation/screen/reportwrite/component/CompleteReportCard.kt
  • presentation/src/main/java/com/threegap/bitnagil/presentation/screen/reportwrite/component/template/CompleteReportContent.kt

요약

제보 완료 화면의 UI 레이아웃 및 텍스트 정렬을 수정합니다. LocalContext 사용을 제거하고, 텍스트 정렬을 우측 정렬로 변경하며, 카드의 위치 조정 방식을 offset에서 padding으로 변경하여 레이아웃 겹침 문제를 해결합니다.

변경 사항

Cohort / File(s) 요약
제보 완료 카드 컴포넌트
presentation/src/main/java/com/threegap/bitnagil/presentation/screen/reportwrite/component/CompleteReportCard.kt
LocalContext 제거, 제목/카테고리/주소 필드에 우측 정렬(TextAlign.End) 추가, 행의 수직 정렬을 CenterVertically에서 Top으로 변경
제보 완료 콘텐츠 레이아웃
presentation/src/main/java/com/threegap/bitnagil/presentation/screen/reportwrite/component/template/CompleteReportContent.kt
CompleteReportCard의 위치 조정 방식을 offset(y = 115.dp) 기반에서 padding(top = 115.dp, start = 18.dp, end = 18.dp) 기반으로 변경, offset 임포트 제거

예상 코드 리뷰 시간

🎯 2 (단순) | ⏱️ ~8분

🐰 완료 화면이 이제 반듯하게,
텍스트는 우측에 자리 잡고,
겹침 없이 떨어져 있으니,
사용자가 행복해할 거야,
깔끔한 UI의 춤! ✨

🚥 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 제목이 주요 변경사항인 제보 완료 화면의 UI 오류 수정을 명확하게 설명하고 있으며, 이슈 번호 #188을 포함하여 구체적입니다.
Description check ✅ Passed PR 설명이 관련 이슈, 작업 내용, 리뷰어 대상을 모두 포함하고 있으며 템플릿 구조를 충분히 따르고 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 이슈 #188의 모든 요구사항을 충족합니다: 텍스트 정렬 수정(TextAlign.End 추가)과 겹침 문제 해결(offset에서 padding으로 변경).
Out of Scope Changes check ✅ Passed 모든 코드 변경사항이 제보 완료 화면 UI 수정이라는 정의된 범위 내에 있으며, 관련 없는 변경사항이 없습니다.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ui/#188-report-complete

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

@l5x5l l5x5l left a comment

Choose a reason for hiding this comment

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

🚀🚀

@wjdrjs00 wjdrjs00 merged commit 77d64c0 into develop Mar 1, 2026
2 checks passed
@wjdrjs00 wjdrjs00 deleted the ui/#188-report-complete branch March 1, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📱 UI UI 추가 및 수정 (비지니스 로직을 포함하지 않는 작업) 🧤 대현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UI] 제보 완료 UI 오류사항을 수정합니다.

2 participants