-
Notifications
You must be signed in to change notification settings - Fork 7
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
Changes from 6 commits
74b1e6a
d96a368
a8eaca0
c1ba253
2c2e1a6
b80fe00
d0b9740
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 요거 이렇게 구현하면 YB 모집 딱! 시작할때 시간 맞춰서 배포해야돼서 불편할 것 같아요 분명 제가 예전에 valid 시간인지 체크하도록 구현해놨는데 누가 없앳어? 하고 찾아보니... 저네요... ^^ 요 코드 다시 살려두면 좋을 것 같아요 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 아하 ! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 네넹 맞아요 그 모집 시간 이후부터만 배너 뜰 수 있게! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 애니메이션은 마우스 hover 시 그라데이션이 마우스 움직임에 따라 수평으로 움직이도록 구현하기 위해 쓰인 친구예요 (관련 PR)
그런데 이 슬랙 메시지 보면 선화가 요청한 것은 hover 되지 않았을 때 그라데이션의 짙은 부분이 수평으로 움직이는 애니메이션이 추가되길 원한 것 같고, 현재 그 부분은 구현되어있지 않은 것 같은데용!
선화도 리소스에 보고 선택해달라고 했으니
우선 해당 PR은 이대로 머지 시키고, 다른 페이지 먼저 작업한 후에
새로 브랜치 파서 애니메이션 작업만 추가 진행해도 될 것 같아용
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 넵 ! 사실 저도 언석, 승히님이 따로 말씀이 없으셔서 일단은 후순위로 나중에 구현하는 걸로 생각했던 것 같아요
추후에 작업하겠습니다 !