Skip to content

[NDGL-65] NDGLNavigationBar의 Trailing 아이콘이 없을 때 타이틀이 중앙에 정렬되지 않는 이슈 수정#14

Merged
jihee-dev merged 3 commits intodevelopfrom
fix/NDGL-65
Feb 4, 2026
Merged

[NDGL-65] NDGLNavigationBar의 Trailing 아이콘이 없을 때 타이틀이 중앙에 정렬되지 않는 이슈 수정#14
jihee-dev merged 3 commits intodevelopfrom
fix/NDGL-65

Conversation

@jihee-dev
Copy link
Copy Markdown
Member

@jihee-dev jihee-dev commented Feb 3, 2026

NDGLNavigationBar의 Trailing 아이콘이 없을 때 타이틀이 중앙에 정렬되지 않는 이슈 수정
클로드 로컬 설정 파일 제거 및 .gitignore 추가


연관 문서

디자인

변경사항

  • TrailingIcon이 없을 경우 Box로 여백을 두어서 trailing icon 유무에 관계 없이 타이틀이 동일한 곳에 위치하도록 수정
  • 터치 범위를 고려하여 간격 수정
  • 아이콘 색상이 잘못 적용된 이슈 수정(700 -> 600)
  • 클로드 로컬 설정 파일 제거 및 .gitignore 추가

Summary by CodeRabbit

  • 스타일
    • 네비게이션 바 세로 패딩을 12dp → 4dp, 가로 간격을 8dp → 4dp로 줄여 화면 공간 활용을 개선했습니다.
    • 네비게이션 아이콘의 고정 크기 규격을 변경하고 6dp 패딩을 추가했으며 색상을 진한 검정에서 조금 더 연한 톤으로 조정했습니다.
    • 트레일링 콘텐츠가 없을 때 40dp 자리표시자를 표시해 레이아웃 균형을 유지합니다.
  • 문서
    • 트레일링 콘텐츠 유무를 보여주는 미리보기를 추가했습니다.

@jihee-dev jihee-dev requested a review from mj010504 February 3, 2026 15:42
@jihee-dev jihee-dev self-assigned this Feb 3, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 3, 2026

Walkthrough

NDGLNavigationBar의 레이아웃 패딩, 아이콘 모디파이어와 틴트, 트레일링 콘텐츠 처리 로직이 변경되었고, 관련 프리뷰가 추가되었습니다. .claude 설정 파일에서 permissions 블록이 제거되었습니다.

Changes

Cohort / File(s) Summary
Design System - Navigation Bar
core/ui/src/main/java/com/yapp/ndgl/core/ui/designsystem/NDGLNavigationBar.kt
세로 패딩 12dp→4dp, 가로 간격 8dp→4dp로 축소; 클릭 가능한 아이콘에서 size(28.dp) 제거 후 padding(6.dp) 추가(클릭 가능 모디파이어 순서 변경); 아이콘 틴트 black700black600; 트레일링 콘텐츠가 없을 때 40.dp 크기 Box 플레이스홀더 렌더링(필요해 Box import 추가); NDGLNavigationBarPreviewNDGLNavigationBarNoTrailingPreview 프리뷰 추가
Local Claude Settings
.claude/settings.local.json
permissions 블록(예: Bash git 명령 허용 목록)이 제거됨 (해당 권한 설정 전체 삭제)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 NDGLNavigationBar의 Trailing 아이콘이 없을 때 타이틀 중앙 정렬 문제를 명확하게 설명하며, 변경 사항의 주요 내용과 일치합니다.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/NDGL-65

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
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In
`@core/ui/src/main/java/com/yapp/ndgl/core/ui/designsystem/NDGLNavigationBar.kt`:
- Around line 89-94: The icon's Modifier chain in NDGLNavigationBar.kt currently
uses .size(40.dp).padding(6.dp) which results in a 52dp visual size and
mismatches the 40dp placeholder and 48dp Row height; update the modifier on the
icon (the chain that includes clip(CircleShape), clickable(onClick = onClick),
size(40.dp), padding(6.dp)) so the padding comes before size (i.e.,
.padding(6.dp).size(40.dp)) to keep the visible element 40dp while preserving a
larger touch target, or alternatively adjust the placeholder/Row dimensions to
52dp to match the current ordering—pick one approach and apply it consistently
to the icon modifier and related placeholder sizing in NDGLNavigationBar.

@jihee-dev
Copy link
Copy Markdown
Member Author

클로드 로컬 설정 파일이 깃에 올라가 있어서 제거하고 gitignore 파일에 추가했습니다

@mj010504
Copy link
Copy Markdown
Contributor

mj010504 commented Feb 4, 2026

클로드 로컬 설정 파일이 깃에 올라가 있어서 제거하고 gitignore 파일에 추가했습니다

감사합니다!!

@mj010504
Copy link
Copy Markdown
Contributor

mj010504 commented Feb 4, 2026

수고하셨습니다!

@jihee-dev jihee-dev merged commit f5b0d68 into develop Feb 4, 2026
2 checks passed
@jihee-dev jihee-dev deleted the fix/NDGL-65 branch February 4, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants