Skip to content

Conversation

@thwjddlqslek
Copy link
Member

@thwjddlqslek thwjddlqslek commented Apr 17, 2025

🤔 문제 및 해결방안

  • zIndex 계층 구조로 인해 [원하는 모임 장소 추가] 모달의 header와 bottomsheet가 화면에 표시되지 않는 문제 발생
  • 후보지 추가 화면의 header에 있는 XIcon이 표시되지 않는 문제 발생
  • 카드 리스트가 가로 스크롤 시 드래그로 컨텐츠 이동이 불가능한 문제 발생
  • "유력" 태그가 화면에서 잘리는 UI 이슈 발견
  • 투표 수가 0인 경우 적절한 UI가 표시되지 않는 문제 발생

✍️ 구현 설명

  • zIndex에 tooltip, header 계층을 추가하고 관련 컴포넌트들이 올바르게 표시되도록 코드를 수정했습니다
  • 디자인 가이드와 일치하지 않는 XIcon을 수정했으며, 다른 곳에서 재사용되는 파일에서는 디자인이 달랐기 때문에 SmallXIcon 컴포넌트를 별도로 추가했습니다
  • Swiper 라이브러리를 도입하여 터치/드래그 기반 가로 스크롤 기능 구현했습니다.
  • slidesOffsetBefore 속성을 통해 첫 슬라이드 앞에 여백을 추가하여 "유력" 태그가 잘리지 않도록 수정했습니다.
  • 투표 수(votedCount)가 0인 경우 NoVoteIcon을 표시하고 "아무도 콕하지 않은 장소" 텍스트가 나타나도록 개선했습니다.

📷 이미지 첨부 (Option)

2025-04-18.1.23.20.mov

⚠️ 유의할 점! (Option)

  • 새로운 의존성(swiper)이 추가되었으므로 PR 체크아웃 후 pnpm install 실행 필요합니다.

@thwjddlqslek thwjddlqslek changed the title fix: XIcon 추가/수정 및 zIndex 추가/수정 fix: XIcon 추가/수정, zIndex 계층 추가 및 Swiper를 이용한 가로 스크롤 구현 Apr 17, 2025
@thwjddlqslek thwjddlqslek changed the title fix: XIcon 추가/수정, zIndex 계층 추가 및 Swiper를 이용한 가로 스크롤 구현 fix: UI 개선 - XIcon 수정, zIndex 계층 추가, Swiper 스크롤 및 투표 UI 개선 Apr 17, 2025
@thwjddlqslek thwjddlqslek merged commit 763fd41 into main Apr 17, 2025
3 checks passed
@thwjddlqslek thwjddlqslek deleted the fix/xicon-zindex branch April 17, 2025 16:26
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.

3 participants