Skip to content

Conversation

@jihye5081
Copy link
Collaborator

@jihye5081 jihye5081 commented Feb 10, 2025

#️⃣ Issue Number

#146


📝 요약(Summary)

  • 이달의 차트 모달창 PC, 태블릿, 모바일 버전 무한 스크롤 구현
  • 모달창에서 크레딧 부족 시 크레딧 부족 모달창 나타나게 수정
  • 모달창에서 투표 완료 후 리스트 페이지에 바로 업데이트
  • components 폴더에서 IdolCard 컴포넌트 불러와서 화면에 나오도록 수정
  • IdolCard 컴포넌트를 불러오면서 모달창이 나왔을 때 ListPage에 있는 이미지 보더가 겹쳐서 나오는 부분 수정

🛠️ PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📸스크린샷 (선택)

수정 부분이 너무 많아 사진으로 다 담기 어려울 거 같아 vercel에서 제공하는 PR 사이트 참고해 주시면 감사하겠습니다!! :)


@jihye5081 jihye5081 self-assigned this Feb 10, 2025
@vercel
Copy link

vercel bot commented Feb 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fandom-k ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 10, 2025 5:43am

@github-actions github-actions bot requested review from juha399 and yoonc01 February 10, 2025 05:43
Copy link
Collaborator

@juha399 juha399 left a comment

Choose a reason for hiding this comment

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

지혜님 수고하셨습니다~!

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
Owner

@yoonc01 yoonc01 left a comment

Choose a reason for hiding this comment

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

modal component에서

const [isMobile, setIsMobile] = useState(window.innerWidth < 768);

로 수정하고 16번째에서 24번째 줄을 이렇게 바꾸는 것은 어떤가요???

  useEffect(() => {
    const handleResize = () => setIsMobile(window.innerWidth < 768);
    window.addEventListener('resize', handleResize);
    return () => window.removeEventListener('resize', handleResize);
  }, []);

Comment on lines -28 to +29
<div className="fixed flex flex-col top-0 left-0 size-full bg-midnightBlack">
// z-[9999] : 모달창이 열렸을 때 리스트 페이지에 있는 아이돌 이미지 보더가 같이 보이는 현상 해결을 위해 작성
<div className="fixed flex flex-col top-0 left-0 size-full bg-midnightBlack z-[9999]">
Copy link
Owner

@yoonc01 yoonc01 Feb 10, 2025

Choose a reason for hiding this comment

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

여기서 idolCard에 있는 border의 z인덱스를 지우는 것은 어떤가용 그게 필요가 없어 보이는 거 같아서요!

Copy link
Owner

@yoonc01 yoonc01 left a comment

Choose a reason for hiding this comment

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

commit 단위를 조금 더 세분화했으면 읽는데 더 편했을 거 같아요!

@yoonc01 yoonc01 merged commit 085f654 into main Feb 10, 2025
3 checks passed
@yoonc01 yoonc01 deleted the fix/#146/MonthlyChartModal branch February 10, 2025 06:41
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.

4 participants