Skip to content

feat(home,setting): 알림함·알림 시간 설정을 최종 시안에 맞춘다 (#2323) - #2324

Merged
Sadturtleman merged 3 commits into
developfrom
feat/2323-notification-figma-align
Oct 27, 2026
Merged

Sadturtleman merged 3 commits into
developfrom
feat/2323-notification-figma-align

Conversation

@Sadturtleman

@Sadturtleman Sadturtleman commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

#2321 로 붙인 알림 화면을 최종 시안(Figma 「정리 Screen Design」)에 맞춥니다.

알림함 — Figma 「알람창」(5156:73406)

  • 제목을 「알람」 으로, 항목은 아이콘 24dp + 제목·본문(12sp) + 하단 구분선(gray3)입니다.
  • 목적지별 아이콘: 기록·질문 → BookOpen, 편지·수신인·초대 → Mail, 대리인 설정 → edit, 보관 확인·대리인 확인 → alert-circle.
  • 읽은 알림(alarm_check)은 아이콘·제목·본문을 모두 gray5 로 낮춥니다. 시안에 없는 미읽음 점은 뺐습니다.
  • 본문을 상태 없는 NotificationInboxContent 로 분리했습니다.

알림 시간 설정 — Figma 「설정 - 푸시알림 설정」(5156:18451)

  • 별도 「기록 알림」 화면(라우트·메뉴)을 걷고, 푸시 알림 화면 토글 아래에 구분선 · 「알림 시간 설정」 · 시/분 드롭다운(각 160dp) · 안내 한 줄을 둡니다.
  • 시안에 저장 버튼이 없어 고르는 즉시 PUT …/notification-preferences 로 저장합니다. 요일·목표·켜짐은 불러온 값을 그대로 싣고, 실패하면 이전 시각으로 되돌리고 토스트로 알립니다.

홈 탑바

  • 종 아이콘을 시안의 「bell 1」 에셋으로 바꿨습니다.

모든 아이콘은 시안 SVG 를 그대로 VectorDrawable 로 옮겼습니다(경로·선 굵기·색 그대로).

시안과 다르게 둔 점

  • 안내 문구의 기본 시각: 시안은 「오전 9시」 지만 서버 기본값이 18:00 이라 「오후 6시」 로 적었습니다.
  • 알림함 날짜 줄: 서버 알림함 응답에 시각 필드가 없어 아직 그리지 않습니다.
  • 분 드롭다운은 10분 단위입니다(시안에는 「00」 만 보임).

검증

  • ktlintCheck, :konsist:test, :feature:setting:presentation·:feature:home:presentation 단위 테스트·lintDebug·compileDebugScreenshotTestKotlin, :app:compileDebugKotlin, :app:hiltJavaCompileDebugAndroidTest 로컬 통과
  • 알림 시간 ViewModel 테스트를 즉시 저장·실패 시 되돌림·불러오기 전 미표시로 바꿨습니다.

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "selected",
    "reason": "설정 로컬 스택에서 알림 시간 라우트·이동 명령을 지웠다. 남은 설정 host 의 진입·back 계약과 루트 인자 전달이 그대로인지 API 34 에서 확인한다.",
    "tests": [
      {
        "path": "app/src/androidTest/java/com/afternote/afternote_fe/SettingImplementedCoverageAndroidTest.kt",
        "selector": "com.afternote.afternote_fe.SettingImplementedCoverageAndroidTest#settingNavHost_passwordChangeEntryAndBackKeepSettingsHost",
        "device": "api34"
      },
      {
        "path": "app/src/androidTest/java/com/afternote/afternote_fe/SettingImplementedCoverageAndroidTest.kt",
        "selector": "com.afternote.afternote_fe.SettingImplementedCoverageAndroidTest#appNavigation_settingRouteArgumentSelectsRegisterAsFirstScreen",
        "device": "api34"
      }
    ]
  }
}

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

새 스크린샷 테스트 notificationInboxScreenshot·notificationInboxEmptyScreenshot·pushNotificationWithTimeScreenshot 의 기준선을 screenshot-baseline 라벨로 생성합니다. 기존 기준선은 기본 인자가 그대로라 바뀌지 않습니다.

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

🤖 Generated with Claude Code

- 알림함(Figma 「알람창」): 제목 「알람」, 목적지별 아이콘(기록·편지·설정·주의), 12sp 제목·본문,
  읽은 알림은 아이콘·글자를 gray5 로 낮춘다. 미읽음 점은 뺐다.
- 알림 시간(Figma 「설정 - 푸시알림 설정」): 별도 「기록 알림」 화면을 걷고, 푸시 알림 화면 토글 아래에
  시·분 드롭다운으로 둔다. 고르는 즉시 저장하고 실패하면 되돌린다.
- 홈 탑바 종 아이콘을 시안의 「bell 1」 에셋으로 바꾼다.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Sadturtleman Sadturtleman added enhancement New feature or request screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 area:home 홈 기능 영역 issue-assignee-exempt 담당자 대조 가드 우회 — 대표 Issue 담당자가 아니어도 이 PR 은 통과한다 labels Oct 27, 2026
@github-actions github-actions Bot added the android-test PR head에서 Android Managed Device 계측 테스트 실행 label Oct 27, 2026
@Sadturtleman Sadturtleman added screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 and removed screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 labels Oct 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📸 CI 표준 환경에서 screenshot baseline 3개를 생성·검증해 e36dfc7 로 커밋했습니다.
실행 로그·생성 artifact: https://github.com/Afternote/Afternote-FE/actions/runs/40489467337
이미지 내용은 이 PR의 PNG diff에서 눈으로 최종 확인해 주세요. 필수 검사도 새 SHA에 다시 요청했습니다.

@github-actions github-actions Bot removed the screenshot-baseline CI 표준 환경에서 PR screenshot baseline 생성·적용 요청 label Oct 27, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Sadturtleman
Sadturtleman added this pull request to the merge queue Oct 27, 2026
Merged via the queue into develop with commit 49220ad Oct 27, 2026
21 checks passed
@Sadturtleman
Sadturtleman deleted the feat/2323-notification-figma-align branch October 27, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행 area:home 홈 기능 영역 enhancement New feature or request issue-assignee-exempt 담당자 대조 가드 우회 — 대표 Issue 담당자가 아니어도 이 PR 은 통과한다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(home): 알림함·알림 시간 설정을 최종 시안에 맞춘다

1 participant