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: 랜딩페이지 제작 #132

Merged
merged 23 commits into from
Sep 4, 2023
Merged

feat: 랜딩페이지 제작 #132

merged 23 commits into from
Sep 4, 2023

Conversation

YesHyeon
Copy link
Collaborator

@YesHyeon YesHyeon commented Sep 2, 2023

🔗 연관된 이슈 번호

resolve #131

✨ 어떤 기능을 개발했나요?

  • 랜딩페이지를 개발했습니다.

✅ 어떻게 해결했나요?

  • 시작페이지를 랜딩페이지로 변경하였고 스크롤 애니메이션을 추가했습니다.
  • IntersectionObserver를 이용하여 특정 컴포넌트가 화면에 위치할 때를 판단합니다.

📌 어떤 부분에 집중하여 리뷰해야 할까요?

  • 이미지 파일 이름을 어떻게 하면 좋을지 고민이 됐는데, 직관적으로 숫자를 붙여서 표현했습니다.
    Styled-components가 많아질수록 유지보수에 어려움이 있어서 혼합형 네이밍 방식을 도입해보았습니다.

❗️이 부분은 주의해 주세요! (Option)

  • 아직 Video가 수정되지 않아서 우선 이미지로만 구현했습니다.
  • 방생성 날짜 선택 페이지 토글 기본값 기간 -> 하나씩 수정 작업도 추가했습니다.

🗂️ 참고자료 (Option)

혼합형 네이밍 방식

🚀 결과 (Option)


Copy link
Member

@corinthionia corinthionia left a comment

Choose a reason for hiding this comment

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

오 안 그래도 나중에 말씀 드리려 했는데 className으로 스타일 지정하는 방식 좋은 것 같아요! 확실히 코드가 깔끔해 지네요

또 참고하신 자료에서 말하는 S dot 네이밍 방식도 함께 사용하는 게 어떨까요? 저희가 여태 사용한 방식은 상단의 import문이 너무 길어져서 괜히 코드 가독성이 떨어진다고 느꼈어요!

@YesHyeon
Copy link
Collaborator Author

YesHyeon commented Sep 4, 2023

@corinthionia S dot 도입하는 거 좋습니다! 앞으로 유지보수 하는 페이지마다 하나씩 적용해보는 걸로 하면 좋을 것 같아요. 적용해봤는데 확실히 깔끔해지네요!

Copy link
Member

@corinthionia corinthionia left a comment

Choose a reason for hiding this comment

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

네 차차 변경해 나가면 될 것 같아요!

@YesHyeon YesHyeon merged commit e50c70d into develop Sep 4, 2023
1 check passed
@corinthionia corinthionia deleted the feature/landing branch September 6, 2023 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 랜딩페이지 제작
2 participants