[FIX/#305] 2차 QA 수정

[ShinHyeongcheol]

Related issue 🛠

• closed [FIX] 2차 QA 반영 및 세부 수정 #305

Work Description ✏️

• 2차 qa 반영

Screenshot 📸

Uncompleted Tasks 😅

• Task1

To Reviewers 📢

• 프로필 카드 홈에서 사용해서 수정을 했는데, 해당 부분 다른 곳에서도 같은 이슈 작성해서 수정 중으로 해두었습니다!!
• 차단하기 관련해서는 서버에서 수정 후에 확인해보도록 할게요.
• 그 외 자잘한 부분 조금 수정했습니다...

Summary by CodeRabbit

릴리스 노트

• Bug Fixes

  • 다이얼로그의 닫기 버튼이 공백 대신 올바른 텍스트를 표시합니다.

• UI & 텍스트 개선

  • 지역 변경 화면의 버튼 스타일과 텍스트를 개선했습니다.
  • 사용자 인터페이스 텍스트를 개선했습니다.

• Chores

  • 코드 정리 작업을 완료했습니다.

[coderabbitai]

Walkthrough

이 PR은 2차 QA 피드백을 반영하여 다이얼로그의 기본 닫기 버튼 텍스트를 로컬라이제이션된 문자열로 변경하고, 지역 변경 화면의 미사용 콜백 매개변수를 제거하며, 버튼 스타일과 확인 텍스트를 업데이트하고, UI 텍스트 리소스를 수정합니다.

Changes

Cohort / File(s)	Summary
Dialog 컴포넌트 app/src/main/java/com/smashing/app/core/designsystem/component/dialog/SmashingDialog.kt	ALERT 및 DESTRUCTIVE 스타일의 다이얼로그 닫기 버튼 기본 텍스트를 빈 문자열에서 stringResource(blank_label)로 변경하고 필요한 import 업데이트
UI 컴포넌트 app/src/main/java/com/smashing/app/presentation/home/ranking/SmashingRankingItem.kt	티어 뱃지 이미지 전 TODO 주석 제거
지역 변경 기능 app/src/main/java/com/smashing/app/presentation/home/regionchange/RegionChangeScreen.kt, app/src/main/res/values/strings.xml	RegionChangeScreen에서 navigateToHome 매개변수 제거, 버튼 스타일을 DISABLED_ACTIVE에서 PRIMARY_WITH_DISABLED로 변경, 확인 버튼 텍스트를 yes에서 region_change_confirm으로 업데이트, 문자열 리소스 추가 및 수정 (record: "기록" → "전적", region_change_title: "지역 변경" → "내 지역 변경", 새로운 region_change_confirm: "변경하기")

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• [REFACTOR/#282] api 검토 및 확인할 수 없는 프로필 로직 추가  #288: SmashingDialog.kt의 매개변수 순서 변경 및 preview 추가로 동일 파일의 다이얼로그 구현 수정
• [UI/#76] Region 선택 구현 #79: RegionChangeScreen/RegionChangeRoute 도입으로 동일 컴포넌트 수정
• [REFACTOR/#268] string resources 수정 #272: 지역 변경 관련 문자열 리소스(region_change_*) 추가 및 업데이트

Suggested labels

🔨 FIX, 📱 UI

Suggested reviewers

• seungjunGong
• vahkjsdf
• oilbeaneda

🚥 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 제목이 '[FIX/#305] 2차 QA 수정'으로 이슈 #305와 관련되어 있으며, 주요 변경 사항(QA 수정)을 명확히 나타내고 있습니다.
Description check	✅ Passed	PR 설명이 리포지토리 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 내용, 검토자 안내가 포함되어 있습니다.
Linked Issues check	✅ Passed	PR의 코드 변경 사항들(다이얼로그 텍스트 수정, 버튼 스타일 변경, 문자열 리소스 업데이트, TODO 제거 등)이 이슈 #305의 '2차 QA 반영' 목적과 일치합니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 이슈 #305의 2차 QA 반영 범위 내에 있으며, 다이얼로그, 버튼, 문자열 등의 UI 관련 수정으로 범위를 벗어나지 않습니다.

_{✏️ 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 fix/#305-qa-fix-2

---

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

[coderabbitai]

🧹 Nitpick comments (1)

app/src/main/java/com/smashing/app/core/designsystem/component/dialog/SmashingDialog.kt (1)

129-145: dismissText null 시 빈 라벨 버튼 fallback은 피하는 것을 권장합니다.

Line 129, Line 144에서 빈 문자열 fallback이 유지되어, 텍스트 없는 취소 버튼이 렌더링될 수 있습니다. 최소한 기본 취소 문구를 두는 편이 UX/접근성에 안전합니다.

제안 diff

-import com.smashing.app.R.string.blank_label
+import com.smashing.app.R.string.cancel_short
@@
-                        text = dismissText ?: stringResource(blank_label),
+                        text = dismissText ?: stringResource(cancel_short),
@@
-                        text = dismissText ?: stringResource(blank_label),
+                        text = dismissText ?: stringResource(cancel_short),

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@app/src/main/java/com/smashing/app/core/designsystem/component/dialog/SmashingDialog.kt`
around lines 129 - 145, 현재 SmashingAlertButton 호출에서 dismissText가 null일 때
stringResource(blank_label)로 빈 레이블을 렌더링하는데, 취소 버튼은 빈 텍스트 대신 명시적 기본 문구를 보여주도록
수정하세요: SmashingAlertButton의 text 인자에 dismissText ?:
stringResource(R.string.default_cancel)처럼 명확한 취소 리소스(또는 프로젝트의 기본 취소 문자열 리소스 명)를
사용하거나 dismissText를 non-null로 강제하는 방식으로 변경해 접근성/UX 문제를 해결하세요; 참조 심볼: dismissText,
SmashingAlertButton, onDismissClick, stringResource(blank_label).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In
`@app/src/main/java/com/smashing/app/core/designsystem/component/dialog/SmashingDialog.kt`:
- Around line 129-145: 현재 SmashingAlertButton 호출에서 dismissText가 null일 때
stringResource(blank_label)로 빈 레이블을 렌더링하는데, 취소 버튼은 빈 텍스트 대신 명시적 기본 문구를 보여주도록
수정하세요: SmashingAlertButton의 text 인자에 dismissText ?:
stringResource(R.string.default_cancel)처럼 명확한 취소 리소스(또는 프로젝트의 기본 취소 문자열 리소스 명)를
사용하거나 dismissText를 non-null로 강제하는 방식으로 변경해 접근성/UX 문제를 해결하세요; 참조 심볼: dismissText,
SmashingAlertButton, onDismissClick, stringResource(blank_label).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: cc0a568f-57b0-4c63-9b29-5b57bd1f81db

📥 Commits

Reviewing files that changed from the base of the PR and between d982241 and ee4e7e7.

📒 Files selected for processing (4)

• app/src/main/java/com/smashing/app/core/designsystem/component/dialog/SmashingDialog.kt
• app/src/main/java/com/smashing/app/core/designsystem/component/ranking/SmashingRankingItem.kt
• app/src/main/java/com/smashing/app/presentation/home/regionchange/RegionChangeScreen.kt
• app/src/main/res/values/strings.xml

💤 Files with no reviewable changes (1)

• app/src/main/java/com/smashing/app/core/designsystem/component/ranking/SmashingRankingItem.kt

[seungjunGong]

텍스트 자잘하게 수정된 부분들 QA 시트에서 못봤던거 같은데
어떻게 찾아서 수정해주셨네요 ㅎ.. 고생하셨습니다

[oilbeaneda]

와우띵 수고하셨습니다!

[vahkjsdf]

수고하셨습니당