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

[김나연]Week2 #52

Conversation

kobly
Copy link
Collaborator

@kobly kobly commented Feb 25, 2024

요구사항

기본

  • [기본] PC사이즈만 고려해 주어진 디자인을 구현했나요?
  • [기본] HTML, CSS 파일을 Netlify로 배포했나요? (참고: https://www.codeit.kr/learn/5309)
  • [기본] 랜딩 페이지의 url path는 루트(‘/’)로 설정했나요?
  • [기본] title은 Linkbrary로 설정했나요?
  • [기본] 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 했나요?
  • [기본] 화면의 너비가 1920px 보다 작아질 때, “Linkbrary” 로고의 왼쪽 여백 200px “로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 했나요?
  • [기본] Linkbrary 아이콘은 클릭 시 루트 페이지(‘/’)로 이동하나요?
  • [기본] 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 cursor: pointer를 설정했나요?
  • [기본] “로그인”은 클릭 시 로그인 페이지(‘/signin’)로 이동하나요?
  • [기본] “링크 추가하기”는 클릭 시 회원가입(‘/signup’)로 이동하나요?
  • [기본] 화면의 너비가 1920px 이상이면 내부에 있는 요소간 동일한 간격을 유지하며 가운데 정렬해야 합니다.
  • [기본] 화면의 너비가 1920px 보다 작아질 때, 최하단에 있는 “codeit-2023”의 왼쪽 여백 104px과 SNS 아이콘들의 오른쪽 여백 104px을 유지하면서 가운데 있는 “Privacy Policy”, “FAQ” 요소와 각각 동일한 간격을 유지하며 가까워져야 합니다.
  • [기본] “Privacy Policy”, “FAQ”는 클릭 시 각각 Privacy 페이지(‘/privacy’), FAQ 페이지(‘/faq’)로 이동하나요?
  • [기본] 페이스북, 트위터, 유튜브, 인스타그램 아이콘은 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동 하나요?
  • [랜딩] 아래로 스크롤 해도 “Linkbrary” 로고와 “로그인” 버튼이 있는 상단 네비게이션 바(Global Navigation Bar)영역이 최상단에 고정되게 했나요?
  • [로그인 페이지, 회원가입 페이지 공통] Linkbrary 로고 클릭시 루트 페이지(“/”)로 이동하나요?
  • [로그인 페이지, 회원가입 페이지 공통] 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일한가요?
  • [로그인 페이지, 회원가입 페이지 공통] input 요소에 focus in 일 때, 테두리 색상이 파랑색인가요?
  • [로그인 페이지, 회원가입 페이지 공통] input 요소에 focus out 일 때, 테두리 색상이 회색인가요?
  • [로그인 페이지, 회원가입 페이지 공통] SNS 아이콘들은 클릭시 각각 “https://www.google.com/”, “https://www.kakaocorp.com/page/” 으로 이동하나요?
  • [로그인 페이지] “회원 가입하기”는 클릭시 “/signup” 페이지로 이동하나요?
  • [회원가입 페이지] “로그인 하기”는 클릭시 “/signin” 페이지로 이동하나요?

심화

  • [심화] 사용자의 브라우저 font-size가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정했나요?
  • [심화] palette에 있는 color값들을 css 변수로 등록하고 사용했나요?
  • [심화] 비밀번호 input 요소 위에 비밀번호를 확인할 수 있는 눈 모양 아이콘을 추가했나요?

스크린샷

제목-없음-1

멘토에게

-사용자의 브라우저 font-size가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정했나요? 이부분이 em이나 em을 이용하는거같은데 현업에서도 많이 이용하나요? 보통 박스나 박스안의 요소들 크기를 정할때 사용한다는데 구체적으로 어디에 자주 쓰이는지 궁금합니다!

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

@kobly kobly added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Feb 25, 2024
@kobly kobly self-assigned this Feb 25, 2024
@kobly kobly requested a review from Ahseo February 25, 2024 13:34
@kobly kobly removed their assignment Feb 25, 2024
Copy link
Collaborator

@Ahseo Ahseo left a comment

Choose a reason for hiding this comment

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

실제 배포된 내용도 있으면 더 확인하기 좋을 것 같습니다 :) 고생하셨습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

DS_Store는 mac에서 시스템이 폴더에 접근할 때 생기며, 해당 폴더에 대한 메타데이터를 저장하는 파일입니다. 즉 git에 올라갈 필요가 없어요! 추후 .gitignore 등을 배우실 수도 있을 것 같은데, gitignore에 등록해두면 git에 올라가지 않는 동작을 추가할 수 있습니다. https://www.toptal.com/developers/gitignore 나중에 참고해보세요!

Comment on lines +3 to +24
@import url("https://fonts.googleapis.com/css2?family=Acme&display=swap");

.acme-regular {
font-family: "Acme", sans-serif;
font-weight: 400;
font-style: normal;
}

/* color palette */

:root {
--primary: #6d6afe;
--red: #ff5b56;
--black: #111322;
--white: #ffffff;
--gray1: #3e3e43;
--gray2: #9fa6b2;
--gray3: #ccd5e3;
--gray4: #e7effb;
--gray5: #f0f6ff;
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

나중에 파일이 커지면 컬러 및 폰트 파일은 추후 분리해보는 것도 좋을 것 같아요.

Comment on lines +19 to +20
<div id="wrap">
<header id="header">
Copy link
Collaborator

Choose a reason for hiding this comment

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

일부는 아이디로 하고 일부는 classname으로 작성을 하셨네요. 혹시 이렇게 다르게 작성하신 이유가 있으실까요~?

Copy link
Collaborator

Choose a reason for hiding this comment

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

일반적으로 특별한 이유가 없다면 통일을 하는게 추후 관리하기 더 용이합니다!

</h1>
<h1>쉽게 저장하고 관리해 보세요</h1>
<a href="./signup/" class="link_add">링크 추가하기</a>
<img src="./assets/images/main/mainImg.png" alt="" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

이미지 태그에는 alt 속성을 추가하여 이미지가 로드되지 않았거나, 시각 장애가 있는 사용자를 위한 대체 텍스트를 제공하는 것이 좋습니다 :)

<div class="cont_item">
<div class="cont_txt">
<h1>
<span class="gradient_txt gradient_link">원하는 링크</span>를
Copy link
Collaborator

Choose a reason for hiding this comment

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

classname이 일관성을 가지면 추후 classname만 보더라도 어떤 영역인지 쉽게 파악할 수 있습니다. 다양한 명명규칙이 있는데 https://medium.com/witinweb/css-%EB%B0%A9%EB%B2%95%EB%A1%A0-1-bem-block-element-modifier-1c03034e65a1 이 부분도 가볍게 참고해주세요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

이미지 파일 이름의 규칙이 다 다른 것 같아요! 통일을 해도 좋을 것 같습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

svg와 png를 혼용해서 쓰신 것 같아요. 혹시 두 이미지 파일 형식의 차이에대해 알고계신가요~?

Copy link
Collaborator

Choose a reason for hiding this comment

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

이름만 보고도 어떤 곳에 쓰이는 이미지인지 알 수 있으면 좋을 것 같습니다.

<title>Linkbrary</title>
<link rel="stylesheet" href="./assets/css/common.css" />
<link rel="stylesheet" href="./assets/css/reset.css" />
<link rel="stylesheet" href="./assets/css/signin.css" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

signup.html에서 signin.css를 가져오는 것이 애매해보여요! signin.css를 auth.css로 의미를 포괄할 수 있게 변경하는 것은 어떨까요?

Comment on lines +47 to +52
<input
id="password"
name="password"
placeholder="******"
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.

placeholder가 위와 아래가 다르네요! 통일해도 좋을 것 같긴 합니다.

Copy link
Collaborator

@Ahseo Ahseo left a comment

Choose a reason for hiding this comment

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

질문 답변내용)
em과 rem 단위는 웹 디자인에서 요소의 크기를 설정할 때 사용자의 브라우저 설정이나 상위 요소의 크기에 따라 동적으로 조정되도록 하는 데 유용합니다. 이러한 단위를 사용하는 주된 목적은 접근성과 반응형 디자인을 향상시키는 역할을 하는데요, 사용자가 시스템 또는 브라우저 설정에서 기본 폰트 크기를 변경했을 때, 페이지의 텍스트 크기뿐만 아니라 간격, 크기 등도 비례하여 조정되어야 최적의 사용자 경험을 제공할 수 있습니다. 현업에서 많이 쓰입니다!!

Ahseo
Ahseo previously approved these changes Feb 26, 2024
@Ahseo Ahseo dismissed their stale review February 26, 2024 04:32

main branch로 올리셨네요! 수정해서 올려주시면 될 것 같습니다.

@kobly kobly changed the base branch from main to part1-김나연 February 26, 2024 04:54
@kobly kobly closed this Feb 26, 2024
@kobly kobly deleted the part1-김나연-week2 branch February 26, 2024 05:05
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