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

[강효성] Sprint3 #93

Conversation

Happ-yee4831
Copy link
Collaborator

@Happ-yee4831 Happ-yee4831 commented Jun 7, 2024

요구사항

기본

  • Github에 PR(Pull Request)을 만들어서 미션을 제출합니다.
  • 피그마 디자인에 맞게 페이지를 만들어 주세요.
  • React와 같은 UI 라이브러리를 사용하지 않고 진행합니다.

공통

  • 브라우저에 현재 보이는 화면의 영역(viewport) 너비를 기준으로 분기되는 반응형 디자인을 적용합니다.

랜딩 페이지

  • Tablet 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 24px, “로그인” 버튼 오른쪽 여백 24px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • Mobile 사이즈로 작아질 때 “판다마켓” 로고의 왼쪽에 여백 16px, “로그인” 버튼 오른쪽 여백 16px을 유지할 수 있도록 “판다마켓” 로고와 “로그인" 버튼의 간격이 가까워집니다.
  • 화면 영역이 줄어들면 “Privacy Policy”, “FAQ”, “codeit-2024”이 있는 영역과 SNS 아이콘들이 있는 영역의 간격이 줄어듭니다.

###심화

  • 페이스북, 카카오톡, 디스코드, 트위터 등 SNS에서 Linkbrary 랜딩 페이지(“/”) 공유 시 좌측 예시와 같은 미리보기를 볼 수 있도록 랜딩 페이지 메타 태그를 설정해 주세요.
  • 미리보기에서 제목은 “판다 마켓”, 설명은 “일상의 모든 물건을 거래해보세요”로 설정합니다.
  • 주소와 이미지는 자유롭게 설정하세요.

###배포주소

https://aesthetic-fox-bd5eb6.netlify.app/

멘토에게

-media.css의 141~143번째 줄의 코드가 먹히지 않는 것 같은데, 원인이 무엇인지 모르겠습니다.

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

@Happ-yee4831 Happ-yee4831 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label Jun 7, 2024
@Happ-yee4831 Happ-yee4831 requested a review from jyh0521 June 7, 2024 14:47
Copy link
Collaborator

@jyh0521 jyh0521 left a comment

Choose a reason for hiding this comment

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

과제 하느라 고생 하셨습니다! 리뷰 반영도 잘해주셨네요.

@@ -1,5 +1,5 @@
<!DOCTYPE html>
<html lang="en">
<html lang="ko">
Copy link
Collaborator

Choose a reason for hiding this comment

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

lang 속성 값 수정해주신 것 좋네요!

Comment on lines +5 to +9
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<meta property="og:title" content="판다마켓" />
<meta property="og:type" content="website" />
<meta property="og:url" content="https://aesthetic-fox-bd5eb6.netlify.app" />
<meta property="og:description" content="일상의 모든 물건을 거래해 보세요" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

메타태그 적용 잘해주셨네요.

@@ -18,69 +25,71 @@
<section id="topBannerPanda" class="banner">
<div class="limited-width">
<h1>
일상의 모든 물건을<br>
일상의 모든 물건을<br />
Copy link
Collaborator

Choose a reason for hiding this comment

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

닫는 표시 추가해주신 점 좋습니다.

<span class="contentSmolText">Hot item</span>
<h1>인기 상품을<br>확인해 보세요</h1>
<h2>Hot item</h2>
<h1>인기 상품을 <span class="break-on-desktop"><br /></span>확인해 보세요</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

클래스명도 새롭게 수정해주셨네요! 저번에 말씀드렸던 다른 css 방법론에 대해서 알아보시고 적용해보시는 것도 추천드려요!

<div class="topBigLogo">
<a href="/"><img class="topBigLogoImg" src="img/top_big_logo.png" alt="판다마켓"></a>
</div>
<a href="/" class="topBigLogo"><img class="topBigLogoImg" src="img/top_big_logo.png" alt="판다마켓"></a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

여긴 아직 class 명이 수정되지 않았네요!

}

footer {
padding: 32px 104px 108px 104px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

left와 right 값이 같을때 padding: 32px 104px 108px; 이런식으로도 작성 가능합니다!

<head>
<meta charset="UTF-8">
<title>판다마켓_회원가입</title>
<link rel="stylesheet" as="style" crossorigin
href="https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/static/pretendard.min.css">
<link rel="stylesheet" href="sign.css">
<link rel="stylesheet" href="media.css">
Copy link
Collaborator

Choose a reason for hiding this comment

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

link 태그에도 닫는 표시 해주시면 좋을 것 같습니다.

@jyh0521
Copy link
Collaborator

jyh0521 commented Jun 10, 2024

-media.css의 141~143번째 줄의 코드가 먹히지 않는 것 같은데, 원인이 무엇인지 모르겠습니다.

  • flex-direction: row-reverse; 이 옵션은 display: flex;가 적용되어있어야 동작합니다!

@jyh0521 jyh0521 merged commit f3e77b0 into codeit-bootcamp-frontend:Basic-강효성 Jun 10, 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