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

[Feature] Landing page #37

Merged
merged 24 commits into from
Jul 18, 2024
Merged

[Feature] Landing page #37

merged 24 commits into from
Jul 18, 2024

Conversation

hin6150
Copy link
Member

@hin6150 hin6150 commented Jul 14, 2024

PR Type

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 리팩토링
  • 패키지 의존성 변경, CI/CD, 환경 설정 등

Overview

@hin6150

  • 아이콘 컴포넌트 작업
  • 카드 섹터 배너 작업(디테일 배너)
    • 드래그 한 번으로 뜻 확인
    • 주제별로 깔끔하게 관리
    • 용어집 디스플레이
  • Footer & 하단 배너 작업

@cobocho

  • 히어로 배너 작업

Additional Comment

  • 애니메이션은 매번 이게 맞는지 잘 모르겠습니다 ㅠ

  • apps/web/src/app/(landing)/_component와 apps/web/src/app/(landing)/_components가 중복되는 것 같습니다.
    _component 폴더 안의 컴포넌트 중 사용하는 것은 Detail Banner으로 옮겼습니다. IconBox 쪽도 작업하시는 폴더로 옮기면 좋을 것 같습니다!

Issue

#34

Check List

  • 관련된 이슈를 연결하였나요?
  • 올바른 PR 컨벤션을 작성했나요?
  • 적절한 라벨을 선택했나요?
  • Assignee와 Reviewer를 설정했나요?

References

@hin6150 hin6150 requested a review from cobocho July 14, 2024 13:04
@cobocho cobocho merged commit afc6bfd into main Jul 18, 2024
1 check failed
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.

2 participants