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

[김지민] Week3 #101

Conversation

jimin0209
Copy link

@jimin0209 jimin0209 commented Mar 3, 2024

요구사항

기본

  • [기본] 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용했나요?
    PC: 1200px 이상
    Tablet: 768px 이상 ~ 1199px 이하
    Mobile: 375px 이상 ~ 767px 이하
    375px 미만 사이즈의 디자인은 고려하지 않습니다
  • [랜딩 페이지] 페이스북, 카카오톡, 디스코드, 트위터 등 SNS에서 Linkbrary 랜딩 페이지(“/”) 공유 시 미리보기를 볼 수 있도록 랜딩 페이지 메타 태그를 설정했나요?
  • [랜딩 페이지] 미리보기에서 제목은 “Linkbrary”, 설명은 “세상의 모든 정보를 쉽게 저장하고 관리해 보세요”로 설정했나요?
  • [랜딩 페이지] 주소와 이미지는 자유롭게 설정했나요?
  • [랜딩 페이지] Tablet 사이즈에서 화면의 너비가 1199px 이하로 작아질 때 “Linkbrary” 로고와 “로그인” 버튼 사이의 간격은 변하지 않게 고정값으로 유지하되 좌우 여백이 줄어드나요?
  • [랜딩 페이지] Tablet 사이즈에서 최소 좌우 여백은 “Linkbrary” 로고의 왼쪽에 여백 32px, “로그인” 버튼 오른쪽 여백 32px을 유지하고“Linkbrary” 로고와 “로그인" 버튼의 간격이 가까워지나요?
  • [랜딩 페이지] PC, Tablet 사이즈의 이미지 크기는 고정값을 사용했나요?
  • [랜딩 페이지] Mobile 사이즈의 이미지는 좌우 여백 32px을 제외하고 이미지 영역이 꽉 차나요?
    (이때 가로가 커지는 비율에 맞춰 세로도 커져야 합니다.)
  • [랜딩 페이지] Mobile 사이즈 너비가 커지면, “Privacy Policy”, “FAQ”, “codeit-2023”이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 커지나요?
  • [로그인, 회원가입 페이지] Tablet 사이즈에서 디자인은 PC사이즈와 동일한가요?
  • [로그인, 회원가입 페이지] Mobile 사이즈에서 좌우 여백 32px 제외하고 내부 요소들이 너비를 모두 차지하나요?
  • [로그인, 회원가입 페이지] Mobile 사이즈에서 내부 요소들의 너비는 기기의 너비가 커지는 만큼 커지지만 400px을 넘지 않나요?

심화

  • [심화] 랜딩 페이지 Mobile 사이즈에서 제품 소개 영역의 순서를 제목, 설명, 이미지 => 제목, 이미지, 설명 순서로 변경했나요?

주요 변경사항

스크린샷

  • 랜딩 페이지-태블릿
    landing-tablet

  • 랜딩 페이지-모바일
    landing-mobile

  • 로그인 페이지-태블릿
    signin-tablet

  • 로그인 페이지-모바일
    signin-mobile

  • 회원가입 페이지-태블릿
    signup-tablet

  • 회원가입 페이지-모바일
    signup-mobile

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@jimin0209 jimin0209 requested a review from Jay-WKJun March 3, 2024 08:00
@jimin0209 jimin0209 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Mar 3, 2024
Copy link
Collaborator

@Jay-WKJun Jay-WKJun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 👍

모든 요구사항을 완수해주셨네요!
페이지도 깔끔하게 나오네요 😁

  • 전반적으로 css 이름이 정돈된 느낌입니다. 👍
  • css import를 체계적으로 잘 해주셨네요!
  • commit message도 잘 나눠주셨네요! 👍

나머지는 comment로 달아드렸습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

png를 빼고, svg를 넣어주셨네요. svg와 png, jpg 같은 이미지 에셋들의 차이점은 무엇인지 알아보면 좋을 것 같습니다. 👍

<html lang="ko">
<head>
<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

viewport가 어떤 뜻이 있는지도 알아보면 좋을 것 같습니다. 👍

<head>
<meta charset="UTF-8" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<link rel="stylesheet" href="style.css" />
<link rel="stylesheet" href="./src/style.css" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

어떤건 src에 넣어주셨고, 어떤건 아니네요. 이유가 있을까요?

깔끔하고 직관적인 파일구조 정리도 중요합니다!

Comment on lines -14 to 40
></a>
<a href="./signin.html"><div class="top-login">로그인</div></a>
</nav>
</header>
<main>
<!--1번째 파트-->
<section class="sec1">
<h1 class="sec1-title">
<span class="sec1-sub">세상의 모든 정보</span>를<br />
<div class="landing-header">
<h1 class="header-title">
<span class="header-gradient">세상의 모든 정보</span>를<br />
쉽게 저장하고 관리해 보세요
</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

글자의 띄어쓰기를 보장하려고 하나하나 분해해주셨네요!

하지만, 모바일 모드가 되서 글자가 일렬로 되면 이렇게 깨져버립니다.

letter-spacing이 잘못 적용된 것 같네요.

image

이렇게 여러 element로 글자를 복잡하게 쪼개면 css 조정이 어려울 꺼에요.

white-space라는 css 속성도 이용해보세요.

Comment on lines +58 to +60
나중에 읽고 싶은 글, 다시보고 싶은 영성,<br class="br_tag" />
사고 싶은 옷, 기억하고 싶은 모든 것을<br class="br_tag" />
한 공간에 저장하세요.
Copy link
Collaborator

Choose a reason for hiding this comment

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

br tag 뿐만아니라, \n같은 줄바꿈을 뜻하는 특수문자도 사용할 수 있습니다.

\r\n 같은 문자열에서 사용되는 특수문자들도 알아보면 좋을 것 같습니다. 👍

(HTML에서만 사용되는 escape 문자들도 있습니다.) https://www.zerocho.com/category/HTML&DOM/post/587f50b1308ed50018a00d51

@@ -0,0 +1,168 @@
@import './global.css';
Copy link
Collaborator

Choose a reason for hiding this comment

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

import관계를 잘 설정해주셨네요 👍

Comment on lines +1 to +2
/* user agent stylesheet 초기화 */

Copy link
Collaborator

Choose a reason for hiding this comment

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

이와 동일하게 style초기화를 목적으로 하는 여러 라이브러리들이 있습니다.

찾아보시면 좋을 것 같아요!

align-items: center;
}

nav {
Copy link
Collaborator

Choose a reason for hiding this comment

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

지금은 element가 한정적이어서 문제가 없지만, 수많은 element들이 생성되는 대규모 app에선 element에 직접 style을 주는 것이 문제가 될 수 있습니다.

가능하면, class를 사용해주세요.

Comment on lines +103 to +119
section:nth-of-type(odd) {
grid-template:
'. img'
'title img'
'description img'
'. img'
/30rem 55rem;
}

section:nth-of-type(even) {
grid-template:
'img .'
'img title'
'img description'
'img .'
/55rem 30rem;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

grid로 순서바꾸기 잘 적용해주셨네요! 😁

</div>
<div class="sign-input-box box-password">
<label for="password" class="password">비밀번호</label>
<input id="password" name="password" type="password" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

password라는 id를 2개를 사용해주셨네요.

id attribute는 DOM에서 단 1개만 존재해야합니다.

https://developer.mozilla.org/ko/docs/Web/HTML/Global_attributes/id

@Jay-WKJun Jay-WKJun merged commit aff4bba into codeit-bootcamp-frontend:part1-김지민 Mar 5, 2024
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.

2 participants