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 : MapModal 컴포넌트 기능 구현 #112

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jymaeng1234
Copy link
Collaborator

@jymaeng1234 jymaeng1234 commented Oct 1, 2024

📝 Summary

공연장 정보(장소명,주소,전화번호,지도)를 표시하는 컴포넌트 입니다.

✨ Changes

  • MapModal 컴포넌트를 추가했습니다.
  • 카카오맵 라이브러리를 추가했습니다.

🖼️ Screenshots

image

✅ PR Checklist

  • 하나의 목적을 가진 PR입니다.
  • 코딩 컨벤션과 스타일 가이드를 준수합니다. 📌Conventions
  • 불필요한 코드 중복이 없습니다.
  • 컴포넌트의 책임이 단일합니다.
  • 리액트 훅을 올바르게 사용했습니다.
  • 민감한 정보를 포함하지 않았습니다.
  • 불필요한 콘솔 로그나 주석을 포함하지 않았습니다.
  • 컴포넌트 key값에 고유한 값을 할당했습니다.

🔗 References

💬 Comments

  • 카카오 개발자 사이트에서 API KEY를 발급받고 실행해야 합니다. (https://apis.map.kakao.com/web/guide/)
  • 환경 변수에 VITE_KAKAO_MAP_API_KEY 키를 등록해야합니다.
  • 라이브러리가 추가되었습니다. (npm install 실행 필요)
  • 사용예
    image

@jymaeng1234 jymaeng1234 linked an issue Oct 1, 2024 that may be closed by this pull request
bluedog129
bluedog129 previously approved these changes Oct 1, 2024

&::after {
content: '';
border: 1px solid ${props => props.theme.colors.gray};
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 컨벤션에 맞게 바꾸어주세요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

코멘트 주신 부분 반영했습니다.

background-color: transparent;
font-size: 20px;
&:hover {
color: ${props => props.theme.colors.gray};
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분도요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이부분도 반영했습니다!

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] MapModal 컴포넌트 구현
3 participants