🔗 :: (#382) 알림 없음 화면 추가

[parkuiery]

개요

알림 및 공지 없는 화면을 추가하였습니다.

알림	공지	모집의뢰서	회사

작업 내용

• 알림 없는 화면 추가
• 공지 없는 화면 추가

할 말

없음

Summary by CodeRabbit

Summary by CodeRabbit

• 새로운 기능

  • JobisIcon에 새로운 아이콘 리소스 Empty 추가.
  • 여러 화면에서 비어 있는 상태를 표시하기 위한 새로운 컴포저블 함수 EmptyContent 추가.

• 버그 수정

  • 비어 있는 목록에 대한 적절한 피드백 제공을 위한 조건부 체크 추가.

• 문서화

  • 새로운 문자열 리소스 추가: "아직 공지사항이 없어요", "아직 알림이 없어요".

[coderabbitai]

Walkthrough

이 변경 사항은 JobisIcon 객체에 새로운 아이콘 리소스인 Empty를 추가하여 디자인 시스템의 아이콘 세트를 확장합니다. 또한, 여러 화면에서 빈 상태를 처리하기 위한 새로운 컴포저블 함수들이 추가되고, 기존의 이미지 리소스가 새로운 아이콘 리소스로 업데이트됩니다. 추가적으로, 공지사항 및 알림이 없을 때 표시할 문자열 리소스가 새로 추가되었습니다.

Changes

파일 경로	변경 요약
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt	새로운 아이콘 리소스 Empty 추가: val Empty = R.drawable.ic_empty
core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt	새로운 컴포저블 함수 EmptyContent 추가: 빈 콘텐츠를 표시하는 UI 요소 생성
feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt	EmptySearchContent 제거 및 EmptyContent로 대체, 빈 상태 처리 로직 업데이트
feature/notice/src/main/java/team/retum/notice/ui/NoticesScreen.kt	빈 공지사항 처리 로직 추가, EmptyContent 사용
feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt	빈 알림 처리 로직 추가, EmptyContent 사용
feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt	EmptyRecruitmentContent 제거 및 EmptyContent로 대체, 빈 상태 처리 로직 업데이트

Suggested labels

리팩토링

Suggested reviewers

• rladmsdh
• Tmdhoon2

Poem

🐰 빈 공간에 아이콘 하나,
디자인에 생기를 주네,
공지와 알림도 함께,
새로운 메시지 전해주네.
기쁜 마음으로 변화를 맞이해,
우리 시스템이 더욱 빛나리! 🌟

---

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 93d21ac and c341342.

📒 Files selected for processing (2)

• feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (2 hunks)
• feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

• feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt
• feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt

---

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (6)

feature/notice/src/main/res/values/string.xml (1)

9-9: 새로운 문자열 리소스가 잘 추가되었습니다.

새로운 "no_notice" 문자열 리소스가 PR의 목적에 맞게 잘 추가되었습니다. 이는 공지사항이 없을 때 사용자에게 적절한 피드백을 제공할 것입니다.

일관성을 위해 다음과 같이 문장 끝에 마침표를 추가하는 것이 어떨까요?

-    <string name="no_notice">아직 공지사항이 없어요</string>
+    <string name="no_notice">아직 공지사항이 없어요.</string>

이렇게 하면 다른 문자열 리소스들과 일관된 스타일을 유지할 수 있습니다.

feature/notification/src/main/res/values/strings.xml (1)

17-17: 새로운 문자열 리소스가 적절히 추가되었습니다.

새로운 "no_notification" 문자열 리소스가 PR의 목적에 맞게 잘 추가되었습니다. 이는 알림이 없는 경우를 처리하는 데 유용할 것입니다.

일관성을 위해 다른 문자열 리소스들과 마찬가지로 문장 끝에 마침표를 추가하는 것이 어떨까요? 예를 들면:

-    <string name="no_notification">아직 알림이 없어요</string>
+    <string name="no_notification">아직 알림이 없어요.</string>

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt (1)

38-38: 새로운 Empty 아이콘이 추가되었습니다.

새로운 Empty 아이콘의 추가는 적절해 보입니다. 이는 PR의 목적인 '알림 없음' 및 '공지사항 없음' 화면 구현에 사용될 것으로 보입니다.

가능하다면, 이 새로운 아이콘의 용도나 사용 사례에 대한 간단한 주석을 추가하는 것이 좋을 것 같습니다. 예를 들어:

// 빈 상태를 나타내는 아이콘 (예: 알림이나 공지사항이 없는 경우)
val Empty = R.drawable.ic_empty

feature/notice/src/main/java/team/retum/jobis/notice/ui/NoticesScreen.kt (2)

100-102: 개선사항: 빈 공지사항 처리가 잘 추가되었습니다.

공지사항이 없을 때 EmptyNoticeContent를 표시하는 로직이 잘 구현되었습니다. 사용자 경험을 향상시키는 좋은 추가사항입니다.

개선을 위한 제안:

• if 문 앞뒤로 공백을 추가하여 가독성을 높이는 것이 좋습니다.

다음과 같이 수정하는 것을 고려해보세요:

-        if(notices.isEmpty()) {
+        if (notices.isEmpty()) {
             EmptyNoticeContent()
         }

---

146-164: LGTM: EmptyNoticeContent 함수가 잘 구현되었습니다.

새로 추가된 EmptyNoticeContent 함수는 공지사항이 없을 때 적절한 UI를 표시합니다. Jetpack Compose 컴포넌트들을 잘 활용하고 있으며, 디자인 시스템을 따르고 있습니다.

개선을 위한 제안:

• 일관성을 위해 JobisTheme.colors를 사용하여 텍스트 색상을 지정하는 것이 좋습니다.

다음과 같이 수정하는 것을 고려해보세요:

         JobisText(
             text = stringResource(id = R.string.no_notice),
             style = JobisTypography.SubHeadLine,
+            color = JobisTheme.colors.onSurface,
         )

feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt (1)

208-226: EmptyNotificationContent 함수가 잘 구현되었습니다.

새로운 EmptyNotificationContent 함수는 다음과 같은 장점이 있습니다:

1. Compose 모범 사례를 따르는 잘 구조화된 함수입니다.
2. 중앙 정렬 레이아웃을 위해 적절한 간격과 정렬을 사용합니다.
3. 텍스트 내용에 stringResource를 사용하여 쉬운 현지화가 가능합니다.
4. 이 파일 내에서만 사용되므로 private으로 적절하게 선언되었습니다.

접근성 개선을 위해 Image 컴포저블의 contentDescription을 좀 더 구체적으로 변경하는 것이 좋습니다. 예를 들어:

-            contentDescription = "empty notification",
+            contentDescription = stringResource(id = R.string.empty_notification_description),

그리고 strings.xml 파일에 다음과 같은 리소스를 추가하세요:

<string name="empty_notification_description">알림이 없음을 나타내는 이미지</string>

이렇게 하면 화면 읽기 프로그램 사용자에게 더 명확한 정보를 제공할 수 있습니다.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 39c10a4 and 823ce55.

⛔ Files ignored due to path filters (1)

• core/design-system/src/main/res/drawable/ic_empty.png is excluded by !**/*.png

📒 Files selected for processing (7)

• core/design-system/src/main/java/team/retum/jobisdesignsystemv2/foundation/JobisIcon.kt (1 hunks)
• feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (2 hunks)
• feature/notice/src/main/java/team/retum/jobis/notice/ui/NoticesScreen.kt (3 hunks)
• feature/notice/src/main/res/values/string.xml (1 hunks)
• feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt (6 hunks)
• feature/notification/src/main/res/values/strings.xml (1 hunks)
• feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt (1 hunks)

🧰 Additional context used

🔇 Additional comments (8)

feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (3)

28-28: JobisIcon 임포트 추가 승인

JobisIcon 클래스를 임포트하는 것은 중앙 집중식 아이콘 관리 시스템으로의 전환을 나타내며, 이는 코드의 일관성과 유지보수성을 향상시킵니다.

---

98-98: JobisIcon.Empty 리소스 사용 승인 및 시각적 확인 제안

JobisIcon.Empty 리소스를 사용하는 것은 중앙 집중식 아이콘 시스템과 일치하며 승인됩니다. 그러나 새 아이콘이 빈 상태를 적절히 나타내는지 시각적으로 확인하는 것이 좋습니다.

디자이너 또는 UI/UX 팀과 협력하여 새로운 JobisIcon.Empty 아이콘이 이 컨텍스트에 적합한지 확인해주세요.

---

Line range hint 1-114: 전체 변경 사항 요약 및 영향 평가

이 PR의 변경 사항은 중앙 집중식 아이콘 관리 시스템으로의 전환을 나타냅니다. 이는 코드베이스의 일관성을 향상시키고 향후 아이콘 관련 변경을 더 쉽게 만들 것입니다. 파일의 전반적인 구조와 기능은 그대로 유지되었으며, 변경 사항은 최소화되어 잘 관리되었습니다.

추가 조치 사항:

1. 새로운 JobisIcon.Empty 아이콘의 시각적 적합성 확인
2. 다른 파일들에서도 유사한 아이콘 변경이 필요한지 검토

feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt (2)

102-102: 아이콘 리소스 업데이트가 잘 이루어졌습니다.

JobisIcon.Empty를 사용하도록 변경한 것은 좋은 개선입니다. 이는 다음과 같은 이점을 제공합니다:

1. 아이콘 관리의 일관성 향상
2. 향후 아이콘 변경 시 유지보수 용이성 증가
3. 중앙 집중식 아이콘 시스템 활용으로 인한 코드 가독성 개선

---

Line range hint 1-124: 전반적인 코드 품질이 우수합니다.

이 파일의 변경사항은 작지만 중요한 개선을 이루었습니다. EmptyRecruitmentContent 컴포저블에서 아이콘 리소스를 업데이트한 것은 PR의 목적인 "알림 없음 화면 추가"와 잘 부합합니다. 또한, 이 변경으로 인해 아이콘 관리의 일관성이 향상되었습니다.

코드의 나머지 부분은 잘 구조화되어 있으며, Jetpack Compose의 모범 사례를 따르고 있습니다. 함수들이 명확하게 분리되어 있고, 상태 관리가 적절히 이루어지고 있습니다.

feature/notice/src/main/java/team/retum/jobis/notice/ui/NoticesScreen.kt (1)

3-14: LGTM: 새로운 import 문이 적절히 추가되었습니다.

새로 추가된 import 문들은 EmptyNoticeContent 함수 구현에 필요한 컴포넌트들을 가져오고 있습니다. 이는 코드의 새로운 기능을 지원하기 위해 적절하게 추가되었습니다.

feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt (2)

Line range hint 3-33: import 문 변경이 적절합니다.

새로운 EmptyNotificationContent 함수 구현에 필요한 Image, painterResource, JobisIcon을 import 하였습니다. 이 변경은 적절하며 코드의 기능을 향상시킵니다.

---

Line range hint 114-129: NotificationsScreen 함수의 개선사항이 훌륭합니다.

1. LazyColumn의 modifier를 Modifier.fillMaxWidth()로 변경하여 레이아웃 일관성을 개선했습니다.
2. notificationList가 비어있을 때 EmptyNotificationContent()를 호출하는 조건문을 추가하여 사용자 경험을 향상시켰습니다.

이러한 변경으로 알림이 없을 때 적절한 피드백을 제공하여 사용자 인터페이스가 더욱 직관적이고 반응적으로 개선되었습니다.

[coderabbitai]

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (4)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (3)

21-26: LGTM: 함수 시그니처가 적절합니다. 문서화를 위한 제안이 있습니다.

함수 시그니처가 Compose 모범 사례를 따르고 있으며, 매개변수의 기본값과 이름이 적절합니다.

함수에 KDoc 주석을 추가하여 각 매개변수의 목적과 함수의 전반적인 동작을 설명하는 것이 좋습니다. 예를 들면:

/**
 * 빈 콘텐츠 상태를 표시하는 컴포저블 함수입니다.
 *
 * @param modifier 이 컴포저블에 적용할 수정자
 * @param title 표시할 제목 텍스트
 * @param description 표시할 설명 텍스트 (선택사항)
 */
@Composable
fun EmptyContent(
    modifier: Modifier = Modifier,
    title: String,
    description: String? = null,
) {
    // ...
}

---

27-37: LGTM: 레이아웃 구조가 적절합니다. 접근성 개선을 위한 제안이 있습니다.

메인 Column과 Image 컴포저블의 구조와 사용이 빈 상태 콘텐츠를 표시하기에 적합합니다. JobisIcon.Empty의 사용은 디자인 시스템과의 일관성을 보여줍니다.

접근성을 개선하기 위해 Image 컴포저블의 contentDescription을 더 구체적으로 만드는 것이 좋습니다. 현재의 "empty content"보다는 화면의 컨텍스트에 맞는 설명을 제공하는 것이 좋습니다. 예를 들어:

Image(
    modifier = Modifier.size(128.dp),
    painter = painterResource(JobisIcon.Empty),
    contentDescription = "빈 상태를 나타내는 아이콘",
)

또는 contentDescription을 함수의 매개변수로 받아 호출하는 쪽에서 컨텍스트에 맞는 설명을 제공할 수 있게 하는 것도 좋은 방법입니다.

---

38-55: LGTM: 텍스트 콘텐츠 구조가 잘 구현되었습니다. 텍스트 스타일에 대한 제안이 있습니다.

텍스트 콘텐츠를 위한 중첩된 Column의 사용과 JobisText 컴포저블의 활용이 적절합니다. 설명 텍스트의 조건부 렌더링도 올바르게 구현되었습니다.

텍스트 스타일의 일관성과 유지보수성을 높이기 위해, 하드코딩된 타이포그래피 스타일 대신 의미 있는 이름의 상수를 사용하는 것이 좋습니다. 예를 들어:

object EmptyContentTypography {
    val Title = JobisTypography.SubHeadLine
    val Description = JobisTypography.Description
}

// 사용 시
JobisText(
    text = title,
    style = EmptyContentTypography.Title,
)

JobisText(
    text = it,
    style = EmptyContentTypography.Description,
    color = JobisTheme.colors.onSurfaceVariant,
)

이렇게 하면 나중에 스타일을 변경해야 할 때 한 곳에서만 수정하면 되므로 유지보수가 더 쉬워집니다.

feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (1)

83-86: LGTM: EmptyContent 컴포넌트로 교체

EmptySearchContent 함수를 제거하고 재사용 가능한 EmptyContent 컴포넌트로 대체한 것은 좋은 변경입니다. 문자열 리소스를 사용하여 지역화를 용이하게 한 점도 훌륭합니다.

한 가지 제안사항:

가독성을 높이기 위해 문자열 리소스 참조를 별도의 변수로 추출하는 것을 고려해 보세요. 예를 들면:

val notFoundTitle = stringResource(id = R.string.not_found_company)
val doubleCheckDescription = stringResource(id = R.string.double_check)

EmptyContent(
    title = notFoundTitle,
    description = doubleCheckDescription,
)

이렇게 하면 EmptyContent 호출이 더 간결해지고 각 문자열의 의미가 더 명확해집니다.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between b2c4497 and 93d21ac.

📒 Files selected for processing (5)

• core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (1 hunks)
• feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (2 hunks)
• feature/notice/src/main/java/team/retum/jobis/notice/ui/NoticesScreen.kt (2 hunks)
• feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt (3 hunks)
• feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (3)

• feature/notice/src/main/java/team/retum/jobis/notice/ui/NoticesScreen.kt
• feature/notification/src/main/java/team/retum/notification/ui/NotificationsScreen.kt
• feature/recruitment/src/main/java/team/retum/jobis/recruitment/ui/SearchRecruitmentScreen.kt

🧰 Additional context used

🔇 Additional comments (2)

core/design-system/src/main/java/team/retum/jobisdesignsystemv2/empty/EmptyContent.kt (1)

1-20: LGTM: 패키지 선언 및 임포트가 적절합니다.

패키지 선언이 정확하고 필요한 모든 Jetpack Compose 및 사용자 정의 디자인 시스템 컴포넌트에 대한 임포트가 존재합니다. 사용되지 않는 임포트도 없습니다.

feature/company/src/main/java/team/retum/company/ui/SearchCompaniesScreen.kt (1)

28-28: LGTM: EmptyContent 컴포넌트 import 추가

새로운 EmptyContent 컴포넌트를 사용하기 위한 import 문이 적절히 추가되었습니다.