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

KDT0_JeongBeomHwan 라인(LINE) 클론 코딩 #59

Open
wants to merge 38 commits into
base: main
Choose a base branch
from

Conversation

Bumang-Cyber
Copy link

@Bumang-Cyber Bumang-Cyber commented Jul 28, 2023

##기간:
2023-07-27~2023-07-28

##프로젝트 내용:
원본 링크
클론코딩 링크

##자바스크립트를 사용해서 기능 구현한 부분:

  • 로딩 애니메이션
  • 배너 애니메이션
  • 스크롤에 따른 배너 축소 애니메이션
  • 스크롤에 따른 헤더 현재 위치 변경
  • 서비스 카테고리 별 분류 기능

@Bumang-Cyber Bumang-Cyber changed the title KDT0_JeongBeomHwan KDT0_JeongBeomHwan 라인(LINE) 클론 코딩 Jul 28, 2023
@JitHoon
Copy link

JitHoon commented Jul 28, 2023

dom-loaded 애니메이션 너무 잘 봤습니다~! JS 소스코드 보면서 공부해보고 싶네요!

@Bumang-Cyber
Copy link
Author

@JitHoon 좋게 봐주셔서 영광입니다!! 저도 지훈님 프로젝트 보고 많이 배워보겠습니다! 😄

@IAMISTP
Copy link

IAMISTP commented Jul 29, 2023

와 애니메이션 진심 대박인데요?! 너무 잘하시는데요?! 작성하신 코드 보고 공부해야겠습니다!!!와 .. 진짜 일주일만에 저걸 하셨다는게 대단하신데요?

@Bumang-Cyber
Copy link
Author

@IAMISTP 감사합니다! 저도 혜민님 작업물 보면서 많이 배워가겠습니다:)

Copy link

@JIYEONGYANGdev JIYEONGYANGdev left a comment

Choose a reason for hiding this comment

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

플러그인도 적용하고 js로 애니메이션도 구현하셨네요. 멋집니다!
섹션이 많은만큼 시멘틱 태그를 더 고민해보는 것도 좋을 것 같습니다.

그리고 몇가지 팁으로는,

1번 2번
20230731_5조_정범환_1차과제리뷰01_양지영 20230731_5조_정범환_1차과제리뷰02_양지영

1번) 마우스 호버되는 버튼에는 cursor: pointer css를 적용해주면 좋을 것 같아요.
2번) 문장 줄바꿈은 white-space: .. css 로 어느정도 조절이 가능합니다.

스크롤하면 해당 섹션과 연동된 내비바에 active효과도 넣으셨는데,
여기서 거꾸로 내비바를 클릭하면 해당 섹션으로 이동하는 코드도 js로 구현해보면 좋을 것 같아요. 추후에 여유가 있다면요 :)

기타 참고)

  • .DS_store 이 파일은 맥에서 생기는 건데, .gitignore 파일을 만들어 추가하면 커밋에 포함되지 않을 거예요.
  • woff 폰트도 /asset 안으로 넣어도 될 것 같아요~
  • 많은 걸 구현한 만큼 README.md를 자세히 작성 하는 것도 좋을 것 같아요.

Comment on lines +9 to +27
<!-- OpenGraph -->
<meta property="og:type" content="website" />
<meta property="og:site_name" content="LINE | 라인은 언제나 사용자와 함께합니다." />
<meta property="og:title" content="LINE | 라인은 언제나 사용자와 함께합니다." />
<meta property="og:description" content="메신저 앱 그 이상의 경험을 제공합니다. 라인은 새로운 인프라 경험, 새로운 커뮤니케이션 경험을 모두에게 전달하고자 합니다." />
<meta property="og:image" content="./images/og_bg.png" />
<meta property="og:url" content="https://line.me/ko" />

<!-- Twitter -->
<meta property="twitter:card" content="summary" />
<meta property="twitter:site" content="LINE | 라인은 언제나 사용자와 함께합니다." />
<meta property="twitter:title" content="LINE | 라인은 언제나 사용자와 함께합니다." />
<meta property="twitter:description" content="메신저 앱 그 이상의 경험을 제공합니다. 라인은 새로운 인프라 경험, 새로운 커뮤니케이션 경험을 모두에게 전달하고자 합니다." />
<meta property="twitter:image" content="./images/og_bg.png" />
<meta property="twitter:url" content="https://line.me/ko" />

<!-- Favicon -->
<link rel="shortcut icon" href="./images/favicon.png" type="image/x-icon" />

Choose a reason for hiding this comment

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

메타태그랑 파비콘까지 작업하셨군요! 👍

Comment on lines +491 to +520
.main-products .main-products__service {
display: flex;
height: 560px;
position: relative;
margin-top: 200px;
margin-bottom: 85px;
}
.main-products .main-products__service.reversed {
justify-content: flex-end;
}
.main-products .main-products__service article {
margin-left: 40px;
width: 520px;
height: 424px;
}
.main-products .main-products__service article h3 {
margin-top: 24px;
font-size: 48px;
font-family: "SF pro Display", Arial, sans-serif;
font-weight: 700;
}
.main-products .main-products__service article h4 {
margin-top: 20px;
font-size: 18px;
font-weight: 500;
color: #07b53b;
}
.main-products .main-products__service article p {
margin: 8px 0 60px;
}
Copy link

@JIYEONGYANGdev JIYEONGYANGdev Jul 31, 2023

Choose a reason for hiding this comment

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

ScSS 을 활용한다면 네스팅 구조를 적용할 수 있을 것 같네요!

.main-products {
  .main-products__service {
     article..

Copy link
Author

Choose a reason for hiding this comment

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

오오 SCSS 잘 배워서 활용해보겠습니다. 그리고 cursor hover애니메이션 디테일도 놓쳤었는데 짚어주셔서 감사합니다!

Copy link

@TaePoong719 TaePoong719 left a comment

Choose a reason for hiding this comment

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

범환님은 border로 등장 애니메이션을 처리하셨네요, 되게 창의적이신거 같습니다. 애니메이션 만드는데 고생하셨겠는데요? 뿐만아니라 세부적인 것도 퀄리티있게 잘 하셨네요 대단하네요 잘 배우고 갑니다!

@moana16
Copy link

moana16 commented Aug 2, 2023

처음 시작화면을 완전 잘 구현하셨네요! 그리고 hover 부분도 다 구현하신게 멋집니다 코드 잘 보고 배울게요~

@dbstjrals
Copy link

어우 퀄리티가 정말 높네요. 저는 em이나 rem 같은 상대 단위를 잘 활용을 못하겠던데 반응형 웹 개발에 참고할 수 있을 것 같아요. 또 sticky로 구현된 좌측 내비게이터 바에서 페이지 이동까지 되는 거 보고 깜짝 놀랐습니다.. 많이 배워갑니다 !

Copy link

@JSH99 JSH99 left a comment

Choose a reason for hiding this comment

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

요소도 정말 많고, 애니메이션도 구현하는 데 정말 어려웠을 것 같은데 기간 내에 다 해낸 거 대단하시네요..👍🏻 공부 많이 되셨을 것 같아요! 저도 많이 배우고 갑니다!!

*/

document.addEventListener("DOMContentLoaded", function () {
//DOM이 자바스크립트 파일을 다 읽는 이벤트를 기점으로 시작합니다.
Copy link

Choose a reason for hiding this comment

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

애니메이션 관련 js 코드에 주석을 꼼꼼하게 잘 달아주셔서 코드 리뷰하는 데 도움이 됐어요! 완성된 로딩 애니메이션 정말 멋집니다..⭐️

@Bumang-Cyber
Copy link
Author

하하 다들 감사합니다! 여러분들 코드도 잘 관찰하면서 배웠습니다 감사해요!

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.

8 participants