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

♻️ refactor: BottomSheet 컴포넌트 코드 최소화 리팩토링 #55

Closed
8 tasks done
mihyunLee opened this issue Jun 16, 2023 · 0 comments · Fixed by #60
Closed
8 tasks done

♻️ refactor: BottomSheet 컴포넌트 코드 최소화 리팩토링 #55

mihyunLee opened this issue Jun 16, 2023 · 0 comments · Fixed by #60
Assignees
Labels
♻️리팩토링 리팩토링 관련 수정

Comments

@mihyunLee
Copy link
Member

mihyunLee commented Jun 16, 2023

🤷 구현할 기능

BottomSheet와 사용되는 ListModal, BasicModal 코드를 최소화하기 위한 리팩토링을 진행한다.

🔨 상세 작업 내용

  • props에 전달한 click 함수로 modal handler와 dim으로도 모달을 닫을 수 있게 한다.
  • BottomSheet가 가지고 있는 animation에 대한 상태값을 제거한다.
  • props에 따른 animation 동작 코드 중복 줄이기
  • BottomSheet 위치를 잡아주기 위해 BasicLayout의 LayoutMain styled component position을 변경한다.
    - 이로 인해 BottomSheet의 Slide 애니메이션의 높이를 조절하기 위한 LEN props를 삭제할 수 있다.
  • BottomSheet의 Handler의 위치를 콘텐츠로 받아올 children의 하단으로 마크업 위치를 옮긴다. (접근성 측면)
  • BottomSheet의 ListModal과 BasicMdoal에 렌더링 될 props를 전달하기 위해 BottomSheet와 분리
  • ListModal, BasicModal 디자인 피그마와 맞추기
  • BottomSheet 뒤에 스크롤 기능 막기

⚙️ 구현방법 및 예상 동작

📄 참고 사항

  • 추후 모달 외의 다른 곳에서는 tab 키가 동작하지 않게 로직을 추가해야함
  • 모달이 내려갈 때 리스트의 텍스트가 흔들리는 현상 발생 -> LayoutWrapper에 overflow: hidden 속성 추가
@mihyunLee mihyunLee added the ♻️리팩토링 리팩토링 관련 수정 label Jun 16, 2023
@mihyunLee mihyunLee self-assigned this Jun 16, 2023
mihyunLee added a commit that referenced this issue Jun 18, 2023
♻️ refactor: BottomSheet 컴포넌트 코드 최소화 리팩토링
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️리팩토링 리팩토링 관련 수정
Projects
None yet
1 participant