[Feat] 수리점 페이지 제작#76
Hidden character warning
Conversation
PR 검증 결과✅ TypeScript: 통과 |
KyeongJooni
left a comment
There was a problem hiding this comment.
고생하셨습니다! 현재 피그마에서는 헤더사이즈와 미니맵 사이즈가 동일한 가로를 차지하는 것으로 보이는데 구현해주신 부분에서는 사이즈가 살짝 다른 것 같습니다! 확인해보시면 좋을 것 같아요!! 추가로 전체적으로 Text를 작성하실 때와 컴포넌트 속성 값들을 인라인에서 속성 설정해서 작성하고 있는데 디자인 시스템 토큰 값하고 type 스타일 확인하시고 변경하면 좋을 것 같습니다!! 👍
| CustomOverlay: new (options: { | ||
| yAnchor: number; | ||
| zIndex: number; | ||
| content?: string; | ||
| position?: unknown; | ||
| }) => KakaoOverlay; |
There was a problem hiding this comment.
clickable 타입 정의해야 할 것 같습니다 TypeScript 에러 발생할 수 도 있을 것 같습니다...!
| Promise.all(SEARCH_KEYWORDS.map((keyword) => searchByKeyword(keyword))).then((results) => { | ||
| clearMarkers(); |
There was a problem hiding this comment.
현재 검생중의 에러 발생 시 처리가 없기 때문에 에러처리를 추가하면 좋을 것 같습니다!
| const [isMapReady, setIsMapReady] = useState(false); | ||
|
|
||
| useEffect(() => { | ||
| const kakao = (window as Window & { kakao?: { maps?: KakaoMaps } }).kakao; |
There was a problem hiding this comment.
동일한 선언이 중복되고 있는 것 같습니다 상수로 추출하거나 헬퍼 함수로 분리하면 좋을 것 같아요! 😄
There was a problem hiding this comment.
getKakaoMaps() 헬퍼 함수로 분리했습니다!
| <Button | ||
| variant="fill" | ||
| size="full" | ||
| className="h-[44px] rounded-[var(--radius-l)] bg-[var(--color-white)] text-[var(--color-gray-900)] hover:bg-[var(--color-gray-100)]" | ||
| onClick={handleContactClick} | ||
| > |
There was a problem hiding this comment.
Button 컴포넌트의 variant 시스템을 사용하지 않고 직접 스타일을 덮어쓰고 있는 것 같습니다! 필요하다면 Button에 새로운 variant를 추가해서 사용하면 좋을 것 같습니다!
There was a problem hiding this comment.
Button에 새로운 variant를 추가해서 사용했습니다!
| const kakaoMapKey = import.meta.env.VITE_KAKAO_JS_KEY; | ||
| const kakaoMapSrc = kakaoMapKey | ||
| ? `https://dapi.kakao.com/v2/maps/sdk.js?appkey=${kakaoMapKey}&libraries=services&autoload=false` | ||
| : null; |
There was a problem hiding this comment.
현재 SDK URL 직접 하드 코딩 되어있는데 상수로 분리하거나 env에 넣어서 관리하면 더 좋을 것 같습니다!
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
PR 검증 결과✅ TypeScript: 통과 |
✨ 주요 변경사항
📝 작업 상세 내용
✅ 체크리스트
Close #번호추가📸 스크린샷 (선택)
2026-01-31.2.30.06.1.1.1.mov
🔍 기타 참고사항
🔗 관련 이슈