Skip to content

Conversation

@juha399
Copy link
Collaborator

@juha399 juha399 commented Feb 7, 2025

#️⃣ Issue Number

#38



📝 요약(Summary)

-추가하기 버튼 클릭시 클릭 내가 관심있는 아이돌 섹션에 추가
-닫기 버튼 클릭시 삭제



🛠️ PR 유형

어떤 변경 사항이 있나요?

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


📸스크린샷 (선택)



스크린샷 2025-02-07 123015

스크린샷 2025-02-07 123024

스크린샷 2025-02-07 123032

0207.mp4

💬 공유사항



📚 코드 이해에 필요한(혹은 본인이 이해하는데 사용한) 레퍼런스 목록



@vercel
Copy link

vercel bot commented Feb 7, 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 7, 2025 3:46am

@hyeonjiroh
Copy link
Collaborator

고생하셨습니다!

{/* 관심 있는 아이돌 섹션 */}
<div className="w-full max-w-[1200px] flex flex-col items-center py-6 mobile:py-10">
<h1 className="text-white text-[16px] tablet:text-[20px] pc:text-[24px] font-pretendard font-bold self-start">
내가 관심있는 아이돌
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 피그마에는 '관심있는'으로 되어있는데 '관심 있는'으로 수정하면 좋을 것 같습니다!

@hyeonjiroh
Copy link
Collaborator

지금은 관심 있는 아이돌로 추가되고 나서 아래의 추가할 수 있는 리스트에 관심 아이돌이 그대로 남아있는데요, 이미 관심 아이돌로 추가된 아이돌은 이 밑의 목록에서 없어지거나 선택할 수 없게 비활성화(disabled) + 회색 처리? 같은 걸 해줘서 다시 선택할 수 없다는 걸 표시해주면 좋을 것 같습니다.

};

return (
<div className="w-full min-h-screen bg-midnightBlack flex flex-col items-center font-pretendard">
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tablet, Mobile 화면에서 요소들이 너무 화면 좌우에 딱 붙어있는 것 같습니다. 피그마를 확인해보면 padding이 24px씩 되어있네요.
image
여기에 px-[24px] 해주면 어떨까 싶습니다.

@hyeonjiroh
Copy link
Collaborator

hyeonjiroh commented Feb 7, 2025

반응형 화면에 문제가 있는 것 같습니다.

  • 1200px 이상
    image

  • 1200px 미만 1024px 정도 이상
    image

  • 1024px 정도 미만 768px 정도 이상
    image

  • 768px 정도 미만
    image

저희 tailwind.config.js 파일 확인해보시면

screens: {
        pc: '1200px',
        tablet: '768px',
        mobile: '375px',
      },

이렇게 break point가 설정되어 있으니까, 여기에 맞춰서 다시 반응형 손봐주셔야될 것 같습니다!
(PC는 1200px 이상, tablet은 768px 이상 1200px 미만, mobile은 375px 이상 768px 미만입니다.)

Copy link
Collaborator

@jihye5081 jihye5081 left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다~!

@jihye5081 jihye5081 merged commit 728eccc into main Feb 7, 2025
3 checks passed
@jihye5081 jihye5081 deleted the feat/#87/closeButton branch February 7, 2025 11:00
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