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: 현재 위치와 체험장소 간 거리 안내, 주소 복사 버튼 #55

Merged
merged 18 commits into from
Jul 22, 2024

Conversation

MEGUMMY1
Copy link
Collaborator

@MEGUMMY1 MEGUMMY1 commented Jul 20, 2024

🔎 작업 내용

  • 지도 內 현재 위치와 체험장소 간 거리 안내
  • 주소 복사 버튼(길 찾기 구현은 어려울듯하여.. 주소를 복사할 수 있으면 좋을 것 같다고 생각해서 추가했습니다!)

image

  • 체험 상세 페이지 태블릿 디자인 수정(디자인 시안이 아이패드미니 기준이라서 아이패드프로로 보면 왼쪽으로 쏠려보이는 점 개선하였습니다.)

image

  • popup 버튼 순서 변경

image

  • appKey env 추가
  • 다크모드용 공유 버튼 추가

image

  • 캘린더 예약 가능 일자 표시

image

⚠️ 이슈

  • 캘린더 날짜 글자색으로 구분하고 싶었으나.. 지정이 되지 않아서.. 배경색으로 구분하였습니다.
    또한 예약 가능 시간은 api 상 예약 마감 기준?이 없는 관계로... 표시할 방법이 없어서 기존 PopUp으로 유지하겠습니다.

📢 전달사항

  • npm i --save react-toastify
  • 주소 복사, 공유 링크 복사에서 클립보드에 복사 성공 시 alert창 -> toast 로 변경하였습니다.

@MEGUMMY1 MEGUMMY1 added the 라이브러리 추가 추가한 라이브러리가 있습니다! label Jul 20, 2024
@MEGUMMY1 MEGUMMY1 requested review from bomin0830 and ssumai-kr July 20, 2024 09:15
@MEGUMMY1 MEGUMMY1 self-assigned this Jul 20, 2024
Copy link

netlify bot commented Jul 20, 2024

Deploy Preview for globalnomad-5-8 ready!

Name Link
🔨 Latest commit 2bc81b0
🔍 Latest deploy log https://app.netlify.com/sites/globalnomad-5-8/deploys/669de4d26471420008320ba7
😎 Deploy Preview https://deploy-preview-55--globalnomad-5-8.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@bomin0830 bomin0830 left a comment

Choose a reason for hiding this comment

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

공유 버튼 다크모드용 만들어주셔서 감사합니다!!
수고하셨습니다

Copy link
Collaborator

@ssumai-kr ssumai-kr left a comment

Choose a reason for hiding this comment

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

확인했습니다!!

@MEGUMMY1 MEGUMMY1 merged commit b725a16 into master Jul 22, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
라이브러리 추가 추가한 라이브러리가 있습니다!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants