Skip to content

[FE] refactor: 홈 페이지 캐러셀 이미지 최신 UI로 교체 및 아코디언 초기 상태 열림으로 변경 #1185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 22, 2025

Conversation

soosoo22
Copy link
Contributor

@soosoo22 soosoo22 commented Apr 14, 2025


🚀 어떤 기능을 구현했나요 ?

  • 홈 페이지 캐러셀 이미지를 최신 UI에 맞게 교체했습니다. 모아보기에 대한 설명도 추가했습니다.
  • 캐러셀 좌우 화살표가 잘 보이지 않는 것 같아서 반투명 배경을 추가해 봤습니다.. (<-- 이 부분이 마음에 들지 않는다면 꼭 말해주십쇼...)
  • 추가로...! 7기 크루 UT 이후에 아코디언을 열어두는 게 좋을 것 같다고 결론이 나서 열림으로 변경했습니다.

🔥 어떻게 해결했나요 ?

2025-04-15.12.36.34.mov

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 혹시.. 변경했으면 하는 부분이 있다면... 꼭꼭꼭 말해주세요...🥹

Copy link

cloudflare-workers-and-pages bot commented Apr 14, 2025

Deploying 2024-review-me-release with  Cloudflare Pages  Cloudflare Pages

Latest commit: a233f16
Status: ✅  Deploy successful!
Preview URL: https://a6c57d6b.2024-review-me-release.pages.dev
Branch Preview URL: https://fe-chore-1184-update-home-ca.2024-review-me-release.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Apr 14, 2025

Deploying 2024-review-me-develop with  Cloudflare Pages  Cloudflare Pages

Latest commit: a233f16
Status: ✅  Deploy successful!
Preview URL: https://afb5a37f.2024-review-me-develop.pages.dev
Branch Preview URL: https://fe-chore-1184-update-home-ca.2024-review-me-develop.pages.dev

View logs

@soosoo22 soosoo22 changed the title [FE] chore: 홈 페이지 캐러셀 이미지 최신 UI로 교체 [FE] refactor: 홈 페이지 캐러셀 이미지 최신 UI로 교체 및 아코디언 초기 상태 열림으로 변경 Apr 14, 2025
Copy link
Contributor

@ImxYJL ImxYJL left a comment

Choose a reason for hiding this comment

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

숙원사업(?)이 해결됐군요!!!
바꾼 alt 멘트랑 버튼 배경 다 좋습니다!
"형광펜 기능" 문구 추가가 제일 좋네요 👍다만 강조 색상이 연한 노란색이라 눈에 조금 덜 띄는 것 같아유

@@ -95,7 +95,7 @@ const ReviewCollectionPageContents = () => {
});

return (
<Accordion title={parsedQuestionName} key={`${selectedSection.value}-${index}`} isInitiallyOpened={false}>
<Accordion title={parsedQuestionName} key={`${selectedSection.value}-${index}`} isInitiallyOpened={true}>
Copy link
Contributor

Choose a reason for hiding this comment

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

저는 고르라면 열림 파이긴 한데 이 정도면 그냥 매일매일 열고 닫기를 랜덤으로 줘야 할지도요ㅋㅋㅋㅋ

@soosoo22
Copy link
Contributor Author

⭐️ 변경 사항 ⭐️

올리의 의견을 적극 수용하여 형광펜 강조 색상을 눈에 확 띄게 primary 컬러로 변경했습니다~

스크린샷 2025-04-15 오후 12 53 32

Copy link
Contributor

@chysis chysis left a comment

Choose a reason for hiding this comment

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

언젠간 바꾸겠지... 했던 게 쑤쑤 덕분에 완성되었네요🫠
이대로 좋아요!

@soosoo22 soosoo22 merged commit 326115e into develop Apr 22, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 홈 페이지 캐러셀 이미지를 최신 UI에 맞게 변경한다.
3 participants