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 #4

Conversation

JiinSeok
Copy link
Collaborator

@JiinSeok JiinSeok commented Apr 24, 2024

요구사항

기본

  • 랜딩 페이지의 url path는 루트(‘/’)로 설정합니다.
  • title은 “판다마켓”로 설정합니다.
  • 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다.
  • 화면의 너비가 1920px 보다 작아질 때, “판다마켓” 로고의 왼쪽 여백 200px“로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다.
  • 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 CSS 속성 cursor: pointer 로 설정합니다.
  • “판다마켓” 클릭 시 루트 페이지(‘/’)로 이동합니다.
  • '로그인'버튼 클릭 시 로그인 페이지(‘/login’)로 이동합니다 (빈 페이지)
  • “구경하러가기”버튼 클릭 시(’/items’)로 이동합니다.(빈 페이지)
  • “Privacy Policy”, “FAQ”는 클릭 시 각각 Privacy 페이지(‘/privacy’), FAQ 페이지(‘/faq’)로 이동합니다.(모두 빈 페이지)
  • 페이스북, 트위터, 유튜브, 인스타그램 아이콘을 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동합니다.

심화

  • palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.

주요 변경사항

스크린샷

스크린샷 2024-04-24 오후 5 48 12

멘토에게

  • 피그마에 있는 심화 요구사항(반응형)은 구현하지 못했습니다.
  • google fonts에 피그마의 기본 폰트 Pretendard가 없어서 폰트는 적용하지 않았습니다.
  • .gitignore를 배워서 사용했습니다.

@JiinSeok JiinSeok added enhancement New feature or request 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. labels Apr 24, 2024
@JiinSeok JiinSeok marked this pull request as draft April 24, 2024 09:01
@JiinSeok JiinSeok closed this Apr 24, 2024
@JiinSeok JiinSeok reopened this Apr 25, 2024
@JiinSeok JiinSeok marked this pull request as ready for review April 25, 2024 23:03
@dev-kjy dev-kjy self-requested a review April 26, 2024 04:24
index.html Show resolved Hide resolved
style.css Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
}

.login {
background-color: #3692FF;
Copy link
Collaborator

Choose a reason for hiding this comment

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

image

현재 .login 클래스를 포함 해 아래에 있는 여러 클래스에서 background-colorcolor 속성에 대해 색상이 변수로 지정되어 있지 않습니다 😭 변수화는 공통적인 부분을 체계적으로 관리하는 방식이므로 중요한 부분이기에 다음 PR에서 도전해보시는 것을 추천드립니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@dev-kjy 이름을 어떻게 짓는 게 더 적절할까요?

  1. bg-color 같은 의미형
  2. color-blue 같은 색상명칭형

index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
@dev-kjy dev-kjy merged commit 63c1d08 into codeit-bootcamp-frontend:Basic-석지인 Apr 29, 2024
@JiinSeok JiinSeok self-assigned this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants