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

[김지윤] #67

Conversation

21ow
Copy link
Collaborator

@21ow 21ow commented Aug 31, 2024

요구사항

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

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

envvoo added 30 commits August 21, 2024 20:06
각 요소에 padding으로 배치
-> 최대 너비 1200px로 할당 후 중앙 정렬
-> 두 번째 요소만 기본축 end 정렬
미디어 쿼리 값과 기본 값 바뀌어서 변경
전체 페이지 중앙 정렬 삭제
-> 로그인-회원가입 페이지 로고 위 공통 margin: 60px 할당
input의 font-family: Pretendard 추가
-> 브라우저 기본 값 때문에 적용이 안 되어서 직접 할당

placeholder의 line-height 삭제
-> chrome의 기본 값 !important 때문에 line-height 적용 안 됨

아이콘 CDN 링크 삭제
로고 클래스명 겹침으로 인한 크기 오류
-> 랜딩 페이지 / 로그인-회원가입 페이지를 기준으로 클래스명을 분리
--> 랜딩 페이지의 로고는 재사용 가능성이 있으므로, 로그인-회원가입
페이지만 변경

로고 이미지 깨짐 현상으로 lg 사이즈 이미지로 전부 변경
-> 기존 사이즈별 이미지 전부 삭제
-> lg 이미지 적용 후 각 조건에 맞게 너비와 높이 지정

랜딩 페이지 break-point: 375px 기준 로고 변경을 위한 이미지 파일 추가
@21ow 21ow closed this Aug 31, 2024
@21ow 21ow deleted the part1-김지윤-sprint4 branch September 4, 2024 06:40
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.

1 participant