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/#296] 회의 생성 전 확인 바텀시트 구현 #313

Merged
merged 7 commits into from
Aug 20, 2024

Conversation

simeunseo
Copy link
Member

🌀 해당 이슈 번호

🔹 어떤 것을 변경했나요?

  • BottomSheet 공통컴포넌트로 분리 (light한 분리...)
  • 회의 생성 전 확인 바텀시트 구현

🔹 어떻게 구현했나요?

function BottomSheet({ children, isOpen }: BottomSheetProps) {
  return (
    <>
      <BottomSheetModal $isModalOpen={isOpen}>{children}</BottomSheetModal>
      <ModalOverlay $isModalOpen={isOpen} />
    </>
  );
}

이렇게 BottomSheet의 기본적인 틀과 애니메이션을 구성하는 컴포넌트를 따로 분리했습니다.
BottomSheet가 사용되는 곳이 두 부분이기 때문에, 각각에 부분에서 BottomSheet안에 들어가야할 내부 컴포넌트를 children으로 넣어주어 사용했습니다.

🔹 PR 포인트를 알려주세요!

전에 말씀드린대로, ui 라이브러리 기반으로 리팩토링예정입니다.

🔹 스크린샷을 남겨주세요!

2024-08-18.1.36.10.mov

@simeunseo simeunseo added feat new feature 은서 은서의 개발 라벨 labels Aug 17, 2024
@simeunseo simeunseo requested a review from ljh0608 August 17, 2024 16:40
@simeunseo simeunseo self-assigned this Aug 17, 2024
Copy link
Member

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다~

Comment on lines 49 to 50
console.log(meetingInfo.duration);

Copy link
Member

Choose a reason for hiding this comment

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

p4) 저도 콘솔 지우는걸 자주 깜빡하는데 eslint 콘솔 설정을 error로 변경할까요??

Copy link
Member Author

Choose a reason for hiding this comment

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

어머나 좋습니다 ㅎ...ㅎ 추가로 배포 환경에서는 console이 보이지 않게하는 방법이 있다고 들었는데 알아보겠습니다

@simeunseo simeunseo merged commit 89cdcd7 into develop Aug 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat new feature size/L 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 회의 생성 전 바텀 시트 구현
2 participants