Skip to content

Conversation

@juha399
Copy link
Collaborator

@juha399 juha399 commented Feb 5, 2025

#️⃣ Issue Number

#28



📝 요약(Summary)

  • 반응형 크기 적용
  • 반응형 따라서 좌우 버튼 따라오게 만들기
  • 슬라이드 추가
  • 아이돌 컴포넌트 클릭 시 그라데이션 넣기
  • 추가 버튼 누르면 내가 좋아하는 아이돌 섹션에 추가되게 하기
  • 추가버튼 그라데이션 넣기
  • 폰트 수정

페이지에 오류가 많은 것 같으니 의심가는 부분이나 오류 원인이 있으시면
말씀해 주시면 감사하겠습니다
(슬라이드 관련 논의 예정)



🛠️ PR 유형

어떤 변경 사항이 있나요?

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


📸스크린샷 (선택)

스크린샷 2025-02-05 192444
스크린샷 2025-02-05 192428

bandicam.2025-02-05.19-20-13-818.mp4


💬 공유사항



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



@vercel
Copy link

vercel bot commented Feb 5, 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 5, 2025 1:43pm

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.

수고하셨습니다!

Comment on lines +90 to +91
<div className="w-full min-h-screen bg-[#02000E] flex flex-col items-center font-pretendard">
<Header />
Copy link
Owner

Choose a reason for hiding this comment

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

bg-midnightBlack으로 쓰시는 게 좋을 거 같아요!

Comment on lines +94 to +95
<div className="w-full max-w-[1200px] flex flex-col items-center py-6 sm:py-10">
<h1 className="text-white text-[16px] tablet:text-[20px] pc:text-[24px] font-bold self-start">
Copy link
Owner

Choose a reason for hiding this comment

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

첫번째에 sm도 저희는 mobile로 써요!
반응형 써 놓으신 것들은 다 바꾸셔야 할 거에요!
읽으면서 보이면 다 적어 놓겠습니다!

Comment on lines +43 to +46
className={`relative w-[98px] h-[98px] md:w-[128px] md:h-[128px] p-[2px]
flex items-center justify-center rounded-full
${isSelectable ? 'cursor-pointer' : 'cursor-default'} transition-all`}
onClick={isSelectable ? toggleFavorite : undefined}
Copy link
Owner

Choose a reason for hiding this comment

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

첫번째에 md는 tablet으로 쓰기로 했어요!
반응형 써 놓으신 것들은 다 바꾸셔야 할 거에요!
읽으면서 보이면 다 적어 놓겠습니다!

Comment on lines +61 to +65
<img
src={checkIcon}
alt="check"
className="absolute w-[40%] h-[40%] top-1/2 left-1/2 transform -translate-x-1/2 -translate-y-1/2 z-30"
/>
Copy link
Owner

Choose a reason for hiding this comment

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

저 가운데로 설정하는 css를 진짜 많이 써서 기억해놓으면 좋아요! top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2 이거요!

Comment on lines +99 to +110
<style>
{`
html, body {
background-color: #02000E; /* 브라우저 전체 배경 */
margin: 0;
padding: 0;
width: 100%;
height: 100%;
overflow-x: hidden; /* 가로 스크롤 방지 */
}
`}
</style>
Copy link
Owner

Choose a reason for hiding this comment

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

이거 없어도 될 거 같아요! 없어도 크게 문제 없어 보입니다!

`}
</style>
<div className="w-full overflow-x-auto scrollbar-thin scrollbar-thumb-steelGray">
<div className="grid grid-cols-3 md:grid-cols-4 lg:grid-cols-8 gap-3 mt-4 mx-auto min-h-[150px]">
Copy link
Owner

Choose a reason for hiding this comment

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

이것도 반응형 바꾸셔야 해요!

<button
onClick={prevPage}
disabled={currentPage === 0}
className="absolute left-[1%] md:left-[-6%] lg:left-[-4%] top-1/2 transform -translate-y-1/2
Copy link
Owner

Choose a reason for hiding this comment

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

이 친구도 반응형이요!

그리고 정중앙에 놓은 top-1/2 left-1/2 -translate-x-1/2 -translate-y-1/2 이거는 자주 사용하시는 거 같아서 apply 문법 적용해보는 것도 좋아보입니다!

</button>

{/* 아이돌 리스트 */}
<div className="grid grid-cols-3 md:grid-cols-4 lg:grid-cols-8 gap-3 mt-4 mx-auto min-h-[300px]">
Copy link
Owner

Choose a reason for hiding this comment

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

반응형이요!

Comment on lines +170 to +174
className="absolute right-[1%] md:right-[-6%] lg:right-[-4%] top-1/2 transform -translate-y-1/2
w-[29px] h-[135px] rounded-[4px]
bg-[rgba(27,27,27,0.8)]
hover:bg-[rgba(27,27,27,1)] transition-all
flex items-center justify-center"
Copy link
Owner

Choose a reason for hiding this comment

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

반응형이요! 근데 flex items-center justify-center도 많이 써서 tailwind의 apply 적용해보는 것도 좋아보여요!

Copy link
Collaborator

@doctor-taco doctor-taco left a comment

Choose a reason for hiding this comment

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

효준님이 짚은 거 외에 더 짚을 거 없는 거 같아요 👀
수고하셨습니다! 👍

@juha399 juha399 merged commit b8ecaa8 into main Feb 5, 2025
3 checks passed
@juha399 juha399 deleted the fix/#28/FavoriteSecotion branch February 5, 2025 23:53
Copy link
Collaborator

@hyeonjiroh hyeonjiroh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

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.

5 participants