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

[FE][Feat] #257 : Search 기능 구현 및 검색창 기능들 구현 #270

Merged
merged 4 commits into from
Nov 26, 2024

Conversation

leedongyull
Copy link
Collaborator

📝 PR 개요

  • 검색을 통한 마커 찍기 구현
  • 검색창에 마커 위치 정보 구현
  • 검색창 지도 안으로 이동
  • 출발지 / 도착지 선택시에만 검색창 나오도록 구현

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#257


📷 스크린샷 및 동영상 (선택 사항)

-.DDara.-.Chrome.2024-11-26.15-31-05.mp4
  • 검색을 통한 마커 찍기 구현
  • 마커 위치 검색창에 뜨도록 구현
  • 출발지 / 도착지 선택시에만 검색창 나오도록 구현

📚 이야기 거리

  • 아까 스크럼 때 이야기 했던 컨텍스트 문제는 맵 안에 searchBox가 들어있는 형태로 바뀌게 되어 어느정도 해결 됐습니다!
  • 글꼴의 경우 제 크롬문제인지 바꿔도 똑같이 보이네요... 일단 클래스에 넣어놓긴했습니다!

@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 26, 2024
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

오 너무 좋은데요.....? 아침에 말씀드린 거 다 반영되었군요

폰트 한번 더 확인 부탁드립니다!
채널 추가 화면에서 폰트 왼쪽 정렬해준 텍스트 세로 가운데 정렬 부탁드립니다!

@juwon5272
Copy link
Collaborator

고생하셨습니다
궁금한게, 영상에서 플로팅버튼 왼쪽 텍스트 시작부분이 가끔 깜빡이는데 거기는 왜 그런가요?

Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

그리고 subtitle이랑 지도 사이의 간격 24px이고, 지도와 footer 버튼 사이 간격 20px인데 안 맞는 것 같습니다! 확인 부탁드릴게요!

Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

그리고 콘솔은 제거하고 올려주세요!

@leedongyull leedongyull reopened this Nov 26, 2024
@leedongyull
Copy link
Collaborator Author

description p태그여서 select-none 속성으로 깜빡거림 제거했습니다
콘솔 다 지웠고 스타일 변경했습니다!

Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

@leedongyull leedongyull merged commit 20f96e6 into frontend Nov 26, 2024
4 checks passed
@leedongyull leedongyull changed the title Feature/fe/search [FE][Feat] #257 : Search 기능 구현 및 검색창 기능들 구현 Nov 26, 2024
@leedongyull leedongyull deleted the feature/fe/search branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants