Skip to content

캠퍼스맵 UI QA 반영 - #736

Draft
ikseong00 wants to merge 4 commits into
developfrom
fix/campus-map-qa
Draft

캠퍼스맵 UI QA 반영#736
ikseong00 wants to merge 4 commits into
developfrom
fix/campus-map-qa

Conversation

@ikseong00

@ikseong00 ikseong00 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

수정사항

  • 핀 아이콘이 잘려 보이는 문제 수정
  • 운영시간 텍스트 정렬
  • 카테고리 중복 선택 가능하도록 수정
  • 카테고리 바텀시트에 사진이 표시되는지 확인
  • 카테고리 바텀시트에 운영시간이 없을 때 표시 방법 확인
  • 지도 크기별로 표시되는 마커 확인
  • 운영시간 표기 방법 수정
  • 사진 상세보기 및 확대 기능 추가

핀 아이콘 잘림 수정

  • 선택 핀 벡터의 path가 40×48 viewport 경계까지 차 있어 1.33dp stroke 일부가 viewport 밖에서 잘리고 있었습니다.
  • 마커 크기는 기존 40×48dp로 유지하고, 벡터 내부 path만 중심 기준으로 축소해 흰색 테두리가 viewport 안에 표시되도록 수정했습니다.

운영시간 텍스트 정렬

  • 운영시간, 학기 중, 방학 중 라벨에 서로 다른 너비와 시작 패딩이 적용되어 시작선이 어긋나 있었습니다.
  • 동일한 라벨 너비를 사용하고 라벨과 주중·주말 값을 왼쪽 정렬해 같은 시작선에 맞췄습니다.

카테고리 중복 선택

  • 기존에는 단일 카테고리만 상태로 관리해 새로운 카테고리를 선택하면 이전 선택이 해제됐습니다.
  • 선택 상태를 카테고리 목록으로 변경하고, 선택한 모든 카테고리를 API 쿼리 파라미터로 전달하도록 수정했습니다.
  • 카테고리별 개별 해제와 전체 해제, 빠른 선택 변경 중 이전 응답이 최신 상태를 덮어쓰지 않도록 요청 상태도 함께 처리했습니다.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0d86b1ec-bcac-4876-9708-c56c79255814

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant