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

[ISSUE-74] 바텀시트 리펙토링 #75

Merged
merged 2 commits into from
Jul 6, 2022

Conversation

KwonDae
Copy link
Member

@KwonDae KwonDae commented Jun 30, 2022

ISSUE


작업 내용

  • 컴포넌트별로 컴포저블을 나눴습니다
  • BottomSheetDialogProperties 를 통해 dismissOnBackPress, dismissOnClickOutside, navigationBarColor 를 컨트롤 할 수 있도록 생성'만' 해두었습니다.
  • 바텀시트가 활성상태일 때, onBackPressed 처리에 대한 이슈가 남아있습니다.

TODO

  • 바텀시트 활성상태 - onBackPressed 처리

Check List

  • PR 제목은 [ISSUE-{N}] PR 제목으로 작성
  • CI/CD 통과 여부
  • 1명 이상의 Approve 확인 후 Merge
  • 관련 이슈 연결

@KwonDae KwonDae added this to the Sprint #5 milestone Jun 30, 2022
@KwonDae KwonDae changed the title refactor - 바텀시트 생성 부분 리펙토링 [ISSUE-74] 바텀시트 생성 부분 리펙토링 Jun 30, 2022
@KwonDae KwonDae changed the title [ISSUE-74] 바텀시트 생성 부분 리펙토링 [ISSUE-74] 바텀시트 리펙토링 Jun 30, 2022
@KwonDae KwonDae self-assigned this Jun 30, 2022
- BottomSheetScaffold -> ModalBottomSheetLayout
@jihee-dev jihee-dev self-assigned this Jul 5, 2022
Copy link
Member

@jihee-dev jihee-dev left a comment

Choose a reason for hiding this comment

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

대원 님과 상의 후에 BottomSheetScaffold에서 ModalBottomSheetLayout으로 교체하였습니다!
사유는 기존 바텀 시트 스캐폴드는 뒷배경을 어둡게 하는 처리가 되어 있지 않아서,
배경 클릭을 통해 시트를 닫는 기능과 애니메이션 등이 조금 자연스럽지 못해서 해당 부분을 수정하는 중에
더 적합한 기본 컴포넌트를 찾았습니당(ModalBottomSheetLayout)
배경 클릭을 통한 닫기, 애니메이션 등 대부분 미리 구현되어 있기 때문에, 플랜즈 바텀 시트의 디자인만 적용하여 플랜즈 컴포넌트로 구현해 두었습니다!

@KwonDae
Copy link
Member Author

KwonDae commented Jul 6, 2022

@jihee-dev
코드 확인했습니다 수정할 내용은 안보였어요!
고생하셨습니다 👏

cc. @hoyahozz

@KwonDae KwonDae merged commit 9cc7582 into develop Jul 6, 2022
@KwonDae KwonDae deleted the feature/issue-074-refactor-bottomSheet branch July 6, 2022 13:21
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.

[Sprint #5] 메인홈 - 홈 바텀시트 리펙토링
2 participants