Skip to content

Conversation

elecharu
Copy link
Collaborator

📖 개요

  • 디자인 시스템 Empty 재사용 컴포넌트 추가

✅ 관련 이슈

🛠️ 상세 작업 내용

  • Empty 상태 표시를 위한 컴포넌트 개발
  • 라이트/다크 모드별 스타일링
  • hover, Active 상태 스타일링

📸 스크린샷 (옵션)

image image

👥 리뷰 확인 사항 (옵션)

라이트모드, 다크모드별 렌더링 확인까지는 확인을 했습니다.
추가적으로 구현되면 좋겠다라는 부분이 있으면 말씀해주세요!

@elecharu elecharu self-assigned this Oct 10, 2025
@sageherb sageherb requested a review from Copilot October 10, 2025 08:09
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a reusable EmptyState component for displaying empty or no-data states with customizable icons, text, and action buttons. The component supports both light and dark modes with hover and active state styling.

  • Added EmptyState component with TypeScript definitions
  • Implemented responsive design with Tailwind CSS for light/dark mode support
  • Included comprehensive documentation and usage examples

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/features/Empty/type.ts Defines TypeScript interfaces for EmptyState component props
src/features/Empty/index.tsx Main EmptyState component implementation with styling and documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

};

// Empty State 관리
export interface EmptyStateProps {
Copy link
Collaborator

Choose a reason for hiding this comment

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

컨벤션 변경으로 컴포넌트와 같은 파일로 있어야할 것 같습니당

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

확인했습니다!

<p className="text-wh/75 dark:text-wh/70 label-text-s text-[15px]">{description}</p>
</div>

{action && (
Copy link
Collaborator

Choose a reason for hiding this comment

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

actionIcon이 action에 종속적이면 action type에 포함되는건 어떨지 제안드립니당

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

생각해보니 action 관련 영역에서 이루어지는 활동들이기에 icon을 따로 빼는 것이 아닌 action type에 포함되는게 맞겠네요!
위에 컨벤션 수정하면서 추가적으로 작업하도록 하겠습니다. 감사합니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

해당 작업 완료했습니다!

sehyun0518
sehyun0518 previously approved these changes Oct 13, 2025
@sehyun0518
Copy link
Collaborator

확인했습니다 수고하셨습니당

@sageherb sageherb requested a review from Copilot October 14, 2025 02:35
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@sageherb sageherb left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 😀

Copy link
Collaborator

@sehyun0518 sehyun0518 left a comment

Choose a reason for hiding this comment

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

copilot 내용 반영 확인했습니다

@elecharu elecharu merged commit f5f23ac into dev Oct 14, 2025
@elecharu elecharu deleted the feature/empty-state-component branch October 14, 2025 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Empty 컴포넌트

3 participants