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

Feature: bottom sheet 구현 #15

Merged
merged 10 commits into from
Aug 2, 2024
Merged

Feature: bottom sheet 구현 #15

merged 10 commits into from
Aug 2, 2024

Conversation

soulchicken
Copy link
Member

Feature: bottom sheet 구현

PR을 한 이유 🎯

  • �bottom sheet 자체를 어떻게 구현해야하는지 막막했지만 어뜨케 꾸겨넣었습니다! 이 코드 기반에서 조금씩 디벨롭하면 나중에 더 멋진 컴포넌트가 될거에요! ㅠㅠㅠ
  • 드디어 백엔드 API와 연결이 잘 되고 있어요. 행복한 일이에요.

이슈 번호 📎

변경사항 🛠

  • 지도를 클릭하면 navBar가 사라지고 bottom sheet 가 뜬! 뜹니다.
  • bottom sheet 상단의 핸들을 끌고 올리면 올라가고 아래로 쭉 내리면 사라지고 다시 navBar가 나오도록 했어요.
  • 레스토랑 리스트를 받아오는 API가 잘 이어져서 얹어놨어요.
  • axios, react-query 세팅을 어딘가를 보고 잘 훔쳐베껴왔어요. 너무 좋은 코드인데 다 가져오기엔 능력 밖입니다. 공부할게요!

특이사항 📌

  • 드디어 처음으로 API 연결에 성공했어요. 어마어마한 진척입니다.
  • bottom sheet의 크기를 일단 1000px로 맞춰놓고 사용하는데. 조금 더 유연한 사용법, 유명한 사용법이 있을 것 같아요. 깡 구현이 아니라 조금 더 그럴싸한 방법이 있는지 찾아봐야겠어요.
  • 이제 bottom sheet에 detail 데이터를 넣어서 한줄평도 만들 수 있게 디벨롭해야해요.
  • sheet 내부에서 스크롤을 하면 상단 핸들이 사라지는 크나큰 단점이 있어요. 조만간 고치러 올게요.

테스트 결과 📝

  • 테스트 결과는 톡방이나 스크럼에서 자랑질했기 때문에 보여주지 않을거에요.

@soulchicken soulchicken merged commit f35968d into develop Aug 2, 2024
1 check passed
@soulchicken soulchicken deleted the feature/bottom-sheet branch August 4, 2024 05:43
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: 바텀 시트 구현
1 participant