Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat/server] 모달 컴포넌트 공통화 #158

Merged
merged 5 commits into from
Jun 7, 2024

Conversation

jihyun-j
Copy link
Contributor

@jihyun-j jihyun-j commented Jun 6, 2024

#️⃣연관된 이슈

📝작업 내용

  1. 모달 컴포넌트가 프로젝트 안에서 자주 쓰일것 같아서 공통화 했습니다.
  2. 커스텀 훅을 사용해서 모달 열기/닫기와 상태를 확인할 수 있습니다.
// 현재 제가 만들어 놓은 Modal component 입니다

const ModalComponent = ({ children, isOpen }: ModalComponentProps) => {
  return (
    <Modal
      open={isOpen}
      aria-labelledby="parent-modal-title"
      aria-describedby="parent-modal-description">
      <Box sx={modalStyle}>{children}</Box> // children을 props로 받아서 사용
    </Modal>
  );
};

다른 컴포넌트에서 모달을 사용하고 싶다면 아래와 같이 사용하시면 됩니다.

const { isOpen, openModal, closeModal } = useModal(); // 모달을 열고 닫을 수 있는 커스텀 훅입니다. 

 <ModalComponent isOpen={isOpen}>
       // 이 부분이 children인데 다양하게 넣어서 사용하시면 됩니다. 
       // 저는 사용자 프로필 컴포넌트를 사용했는데 모달을 닫기 위해서는 closeModal를 반드시 props로 내려 주셔야합니다. 
       <Profile closeModal={closeModal} /> 
 </ModalComponent>

###스크린샷 (선택)

💬리뷰 요구사항(선택)

제가 모달 컴포넌트 공통화 시킨다고 예원님 코드를 수정 했습니다. 혹시 충돌 해결 하시다가 문제생기시면 알려주세용~!

@jihyun-j jihyun-j added the UI/UX front 기능 label Jun 6, 2024
@jihyun-j jihyun-j self-assigned this Jun 6, 2024
infra/Chart.lock Outdated
Comment on lines 1 to 6
dependencies:
- name: ingress-nginx
repository: https://kubernetes.github.io/ingress-nginx
version: 4.10.1
digest: sha256:d25ff4f8cab798c10c0cf44b9cac931287596320e9645bd294dc9eaaa5717bd9
generated: "2024-06-05T19:52:15.651144+09:00"
Copy link
Contributor

Choose a reason for hiding this comment

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

이 파일 삭제하고 .gitignore에 등록해주실 수 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

파일 삭제하고 .gitignore에도 추가했습니다~!

Copy link
Collaborator

@yewon830 yewon830 Jun 7, 2024

Choose a reason for hiding this comment

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

타입스크립트 처음 써봐서 잘 모르는데 types에 style관련 코드도 같이 작성하는 이유가 궁금합니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아마 제가 스타일 폴더 만들어서 스타일 파일 분리해 놓는 것을 깜빡한것같습니다. 🤦‍♀️
스타일 폴더 만들어서 넣어 놓겠습니다~!

@yewon830 yewon830 merged commit dc44751 into kSideProject:dev Jun 7, 2024
1 check passed
@jihyun-j jihyun-j deleted the feat/server branch June 7, 2024 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX front 기능
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants