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

[김희성] sprint1+2 #23

Conversation

HEES56
Copy link
Collaborator

@HEES56 HEES56 commented Dec 12, 2024

요구사항 (스프린트 미션1)

기본

  • [o] React와 같은 UI 라이브러리를 사용하지 않고 진행합니다
  • [o] PC사이즈만 고려해 주어진 디자인으로 구현합니다.
  • [o] HTML, CSS 파일을 [Netlify](https://www.netlify.com/) 로 배포해 주세요.
  • [o] 랜딩 페이지의 url path는 루트(‘/’)로 설정합니다.
  • [o] title은 “판다마켓”로 설정합니다.
  • [o] “판다마켓” 로고 클릭 시 루트 페이지(‘/’)로 이동합니다.
  • [o] '로그인' 버튼 클릭 시 로그인 페이지(‘/login’)로 이동합니다 (빈 페이지)
  • [o] “구경하러 가기”버튼 클릭 시(’/items’)로 이동합니다. (빈 페이지)
  • [o] “Privacy Policy”, “FAQ”는 클릭 시 각각 Privacy 페이지(‘/privacy’), FAQ 페이지(‘/faq’)로 이동합니다.(모두 빈 페이지)
  • [o] 페이스북, 트위터, 유튜브, 인스타그램 아이콘을 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동합니다.
  • [o] 아래로 스크롤해도 “판다 마켓” 로고와 “로그인” 버튼이 있는 상단 내비게이션 바(Global Navigation Bar)가 최상단에 고정게 해 주세요.
  • [o] 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다.
  • [o] 화면의 너비가 1920px 보다 작아질 때, “판다마켓” 로고의 왼쪽 여백 200px, “로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소 간 거리가 가까워지도록 설정합니다.
  • [o] 화면의 너비가 1920px 이상이면 내부에 있는 요소 간 동일한 간격을 유지하며 가운데 정렬해야 합니다.
  • [o] 화면의 너비가 1920px 보다 작아질 때, 최하단에 있는 “codeit-2024”의 왼쪽 여백 200px과 SNS 아이콘들의 오른쪽 여백 200px을 유지하면서 가운데 있는 “Privacy Policy”, “FAQ” 요소와 각각 동일한 간격을 유지하며 가까워져야 합니다.
  • [o] 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 CSS 속성 cursor: pointer로 설정합니다.

심화

  • [o] reset.css를 설정해 주세요.
  • [o] 사용자의 브라우저 설정에 따라 기본 폰트 크기 설정이 변화함에 따라서 페이지의 요소 간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 주세요.

기본

  • [o] “판다마켓" 로고 클릭 시 루트 페이지(“/”)로 이동합니다.
  • [o] 로그인 페이지, 회원가입 페이지 모두 로고 위 상단 여백이 동일합니다.
  • [o] SNS 아이콘들은 클릭 시 각각 “https://www.google.com/”, “https://www.kakaocorp.com/page/”으로 이동합니다.
  • [o] input 요소에 focus in 일 때, 테두리 색상은 ##3692FF입니다.
  • [o] input 요소에 focus out 일 때, 테두리는 없습니다.
  • [o] 로그인 페이지의 “회원가입”버튼 클릭 시 “/signup” 페이지로 이동합니다.
  • [o] 회원가입 페이지의 “로그인”버튼 클릭 시 “/login” 페이지로 이동합니다.

심화

  • [o] palette에 있는 color값들을 css 변수로 등록해서 사용합니다.
  • [o] 구글 애널리틱스로 방문자 수 확인하기 할 수 있도록 설정합니다.
  • [o] 비밀번호, 비밀번호 확인 input 요소 오른쪽에 비밀번호를 확인할 수 있는 눈 모양 아이콘을 추가합니다.

스크린샷

판다 마켓 메인

판다 마켓 로그인

판다 마켓 회원가입

멘토에게

-https://voluble-moxie-3153e3.netlify.app/

  • 최대한 정정 많이 해서 깎아왔습니다!
  • 처음은 매운맛 부탁드려요!
  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

Copy link
Collaborator

@junguksim junguksim left a comment

Choose a reason for hiding this comment

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

전반적으로 너무 잘 짜주셨습니다!
클래스나 id 명명 규칙도 잘 지켜주셔서 읽기 편했습니다.

다양한 선택자를 사용해주셨는데요!
그에 대한 개인적인 피드백만 추가했습니다.

정말 잘해주셨습니다 :)

width: 100%;
}

.containor {
Copy link
Collaborator

Choose a reason for hiding this comment

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

container 에서 오타가 났네요. ㅎㅎ

gap: 5%;
}

.head-img1 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

img1, img2 같이 애매하게 쓰기 보다는, logo-img, logo-text-img 같이 이미지에 대한 간단한 설명을 할 수 있으면 좋겠습니다!

border-color: var(--inputfocuscolor);
}

.pass-containor > a img{
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분이나 다양한 부분에서 선택자를 자식 선택자와 자손 선택자를 사용하셨는데요!
협업 단계 + 복잡한 프로젝트로 가면 이게 상상치 못한 side effect 를 일으킬 수 있습니다.

예를 들어 div p 으로 div 속의 모든 p 태그에 스타일을 적용했는데, 다른 p 요소를 추가할 때마다 모두 스타일을 적용받게 됩니다.
또한 선택자 간의 적용 우선순위가 있는데, 사용하는 선택자 종류가 많아지면 이것도 부작용을 일으킬 수 있습니다.

저는 협업 단계에서는 id 와 클래스만 사용합니다.

회원가입 버튼처럼 고유할 수 있는 요소는 id, 그 외에 중복될 수 있는 요소는 클래스를 줘서, 해당 요소가 어떤 요소인지 설명하는 기능도 할 수 있습니다. ( id 와 클래스명으로 )

이 경우, 클래스 명으로 toggle-visibility-button 등으로 주면 html 코드 상에서 이 버튼에 대한 설명도 되고 좋을 것 같습니다!

@junguksim junguksim merged commit 5bc8bb8 into codeit-sprint-fullstack:basic-김희성 Dec 17, 2024
1 check passed
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.

2 participants