Skip to content
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

[Feat] 메인페이지 업데이트 + OG 이미지 변경 #408

Merged
merged 7 commits into from
Sep 6, 2024

Conversation

wuzoo
Copy link
Contributor

@wuzoo wuzoo commented Sep 4, 2024

Summary

close #405

Screenshot

2024-09-05.12.32.09.mov

Comment

  • 리크루팅 상단 띠 배너 적용
  • CTA 텍스트 "35기 YB 지원하기"로 변경
  • CTA 키컬러 적용

Copy link

height bot commented Sep 4, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@eonseok-jeon eonseok-jeon 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
Member

@lydiacho lydiacho 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
Member

Choose a reason for hiding this comment

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

요거 이렇게 구현하면 YB 모집 딱! 시작할때 시간 맞춰서 배포해야돼서 불편할 것 같아요

분명 제가 예전에 valid 시간인지 체크하도록 구현해놨는데 누가 없앳어? 하고 찾아보니... 저네요... ^^
다른 작업하다가 갑자기 얘를 지워버렸네요.. (아마 뭐 테스트해보고 싶었거나 35기엔 이 배너가 재활용되지 않을거라고 생각했던 것 같아요)

요 코드 다시 살려두면 좋을 것 같아요
미리 배포해둘 수 있게!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아하 ! checkIsTimeInRage 로 지원 기간인지 확인하여 해당 반환값인 isValidTopBanner 조건부 렌더링 하라는 말씀이신거죵 ??

Copy link
Member

Choose a reason for hiding this comment

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

네넹 맞아요 그 모집 시간 이후부터만 배너 뜰 수 있게!
지금 상태로 배포하면 모집 기간과 무관하게 계속 탑 배너가 뜰테니까유 !

animation: ${BackgroundMove} 3s linear 0s infinite alternate;
background: linear-gradient(274deg, #3c92ff 10%, #8fc0ff 74.12%, #3c92ff 100%);

animation: ${BackgroundMove} 3s linear infinite;
Copy link
Member

Choose a reason for hiding this comment

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

이 애니메이션은 마우스 hover 시 그라데이션이 마우스 움직임에 따라 수평으로 움직이도록 구현하기 위해 쓰인 친구예요 (관련 PR)

그런데 이 슬랙 메시지 보면 선화가 요청한 것은 hover 되지 않았을 때 그라데이션의 짙은 부분이 수평으로 움직이는 애니메이션이 추가되길 원한 것 같고, 현재 그 부분은 구현되어있지 않은 것 같은데용!

선화도 리소스에 보고 선택해달라고 했으니
우선 해당 PR은 이대로 머지 시키고, 다른 페이지 먼저 작업한 후에
새로 브랜치 파서 애니메이션 작업만 추가 진행해도 될 것 같아용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 넵 ! 사실 저도 언석, 승히님이 따로 말씀이 없으셔서 일단은 후순위로 나중에 구현하는 걸로 생각했던 것 같아요

추후에 작업하겠습니다 !

@eonseok-jeon
Copy link
Member

뽕쌈하네요~~~ 🚀🚀

@lydiacho
Copy link
Member

lydiacho commented Sep 5, 2024

쌈@뽕하네요~~~🚀🚀

@wuzoo wuzoo merged commit 7c19baa into develop Sep 6, 2024
1 check passed
@wuzoo wuzoo deleted the feat/#405_35-main branch September 6, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 메인페이지 업데이트 (+ OG 이미지 변경)
3 participants