Skip to content

Conversation

@juha399
Copy link

@juha399 juha399 commented Dec 30, 2024

요구사항

기본

[ X ] 랜딩 페이지의 url path는 루트(‘/’) 입니다.
[ X ] title은 “판다마켓”으로 설정해 주세요.
[ ] 화면의 너비가 1920px 이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 해주세요.

[ X ] 화면의 너비가 1920px 보다 작아질 때, “판다마켓” 로고의 왼쪽 여백 200px“로그인" 버튼의 오른쪽 여백 200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 해주세요.
[ X ] 클릭으로 기능이 동작해야 하는 경우, 사용자가 클릭할 수 있는 요소임을 알 수 있도록 cursor: pointer를 설정해 주세요.
[ X ] “판다마켓” 클릭 시 루트 페이지(‘/’)로 이동시켜주세요.

[ X ] “구경하러 가기" 클릭 시 (“/items”)페이지로 이동시켜주세요.(빈 페이지)
[ X ] “로그인”버튼 클릭 시 로그인 페이지(‘/login’)로 이동합니다
[ X ] “구경하러가기”버튼 클릭 시(’/items’)로 이동합니다.
[ X ] 페이스북, 트위터, 유튜브, 인스타그램 아이콘은 클릭 시 각각의 홈페이지로 새로운 창이 열리면서 이동 합니다.
[ X ] “Privacy Policy”, “FAQ”는 클릭 시 각각 아래 페이지로 이동합니다
- Privacy 페이지(‘/privacy’) 
- FAQ 페이지(‘/faq’)

심화

[ X ] 사용자의 브라우저가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 보세요.(설정 값은 자유입니다)

주요 변경사항

[ X ] 메인 페이지 HTML/CSS 구조 구현
[ X ] 이미지 및 로고 파일 추가
[ X ] 로그인 페이지 연결

스크린샷

멘토에게

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

@juha399 juha399 changed the title 판다마켓 초기구조 만들기 [신주하] sprint2 Dec 30, 2024
@juha399 juha399 changed the title [신주하] sprint2 [신주하] Sprint2 Dec 30, 2024
@kiJu2
Copy link
Collaborator

kiJu2 commented Dec 31, 2024

스프리트 미션 하시느라 수고 많으셨어요.
학습에 도움 되실 수 있게 꼼꼼히 리뷰 하도록 해보겠습니다. 😊

@kiJu2
Copy link
Collaborator

kiJu2 commented Dec 31, 2024

commit 단위를 더욱 자주, 작게 해보시는건 어떠실까요?

git을 다룰 때 commit은 "언제 해야 하는가"를 생각해보신 적 있으신가요?
흔히 하는 말이 있습니다:

커밋은 합칠 수 있지만 나눌 수 없습니다.

그럼 커밋을 언제 해야 할까요?

저는 다음과 같은 룰을 지키며 커밋을 하는걸 권장 드립니다:

  1. 커밋을 하는 단위는 커밋 메시지 한 줄로 설명할 수 있는 행동
  2. 하나의 목표 혹은 액션이 달성될 때

관련하여 읽으시면 좋은 아티클을 추천드릴게요:

tl;dr

관련 변경 사항 커밋
커밋은 관련 변경 사항에 대한 래퍼여야 합니다. 예를 들어 두 개의 다른 버그를 수정하면 두 개의 별도 커밋이 생성되어야 합니다. 작은 커밋을 통해 다른 개발자가 변경 사항을 더 쉽게 이해하고 문제가 발생한 경우 롤백할 수 있습니다. 준비 영역과 같은 도구와 파일의 일부만 준비하는 기능을 통해 Git을 사용하면 매우 세부적인 커밋을 쉽게 만들 수 있습니다.

자주 커밋
커밋은 커밋을 작게 유지하고 관련 변경 사항만 커밋하는 데 도움이 되는 경우가 많습니다. 또한 이를 통해 코드를 다른 사람들과 더 자주 공유할 수 있습니다. 이렇게 하면 모든 사람이 정기적으로 변경 사항을 통합하고 병합 충돌을 방지하는 것이 더 쉬워집니다. 대조적으로, 대규모 커밋을 갖고 이를 드물게 공유하면 충돌을 해결하기가 어렵습니다.

미완성 작업을 커밋하지 마십시오
논리적 구성 요소가 완료된 경우에만 코드를 커밋해야 합니다. 자주 커밋할 수 있도록 기능 구현을 빠르게 완료할 수 있는 논리적 청크로 분할합니다. 깨끗한 작업 복사본이 필요하기 때문에(브랜치 확인, 변경 사항 가져오기 등) 커밋하고 싶은 유혹이 든다면 Git의 «Stash» 기능을 대신 사용하는 것이 좋습니다.

커밋하기 전에 코드를 테스트하세요
완료되었다고 생각하는 일을 저지르고 싶은 유혹에 저항하세요. 철저하게 테스트하여 실제로 완료되었는지, 부작용이 없는지(알 수 있는 한) 확인하세요. 로컬 저장소에 설익은 것을 커밋하려면 자신만 용서하면 되지만, 코드를 다른 사람과 푸시/공유하는 경우에는 코드를 테스트하는 것이 훨씬 더 중요합니다.

원문 보기

@kiJu2
Copy link
Collaborator

kiJu2 commented Dec 31, 2024

커밋 타입도 한 번 고려해보세요 !

tl;dr:

커밋 메시지 형식

type: Subject

body

footer

기본적으로 3가지 영역(제목, 본문, 꼬리말)으로 나누어졌다.

메시지 type은 아래와 같이 분류된다. 아래와 같이 소문자로 작성한다.

feat : 새로운 기능 추가
fix : 버그 수정
docs : 문서 내용 변경
style : 포맷팅, 세미콜론 누락, 코드 변경이 없는 경우 등
refactor : 코드 리팩토링
test : 테스트 코드 작성
chore : 빌드 수정, 패키지 매니저 설정, 운영 코드 변경이 없는 경우 등

원문보기


<div class="feature-content">
<h2 class="feature-tag">Hot item</h2>
<h1>인기 상품을<br/>확인해 보세요</h1>
Copy link
Collaborator

Choose a reason for hiding this comment

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

<h1> 태그는 문서에 하나만 넣는게 어떨까요?

이하 MDN 발췌

페이지 당 하나의 <h1>만 사용하세요. 여러 개를 써도 오류는 나지 않겠지만, 단일 <h1>이 모범 사례로 꼽힙니다. 논리적으로 생각했을 때도, <h1>은 가장 중요한 제목이므로 전체 페이지의 목적을 설명해야 할 것입니다. 두 개의 제목을 가진 책이나, 여러 개의 이름을 가진 영화는 볼 수 없죠! 또한 스크린 리더 사용자와 SEO에도 더 적합합니다.

MDN <h1>-<h6>: HTML 구획 제목 요소

Comment on lines +90 to +103
<div id="socialMedia">
<a href="https://www.facebook.com/" target="_blank" rel="noopener noreferrer">
<img src="logo1.jpg" alt="페이스북" width="20" /></a>


<a href="https://twitter.com/" target="_blank" rel="noopener noreferrer">
<img src="logo2.jpg" alt="트위터" width="20" /></a>

<a href="https://www.youtube.com/" target="_blank" rel="noopener noreferrer">
<img src="logo3.jpg" alt="유튜브" width="20" /></a>

<a href="https://www.instagram.com/" target="_blank" rel="noopener noreferrer">
<img src="logo1.jpg" alt="인스타그램" width="20" /></a>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

(제안/선택)요소들이 순서 없는 목록을 의미한다면 <ul><li>로도 나타낼 수 있습니다 !

<ul class="footer-social">
  <li>
	<a target="_blank" href="https://www.facebook.com/">
	  <img src="images/ic_facebook.svg" alt="페이스북" />
	</a>
  </li>
  <li>
	<a target="_blank" href="https://www.twitter.com/">
	  <img src="images/ic_twitter.svg" alt="트위터" />
	</a>
  </li>
  <li>
	<a target="_blank" href="https://www.youtube.com/">
	  <img src="images/ic_youtube.svg" alt="유튜브" />
	</a>
  </li>
  <li>
	<a target="_blank" href="https://www.instagram.com/">
	  <img src="images/ic_instagram.svg" alt="인스타그램" />
	</a>
  <li>
</ul>

MDN: HTML <ul> 요소는 정렬되지 않은 목록을 나타냅니다. 보통 불릿으로 표현합니다.

li 태그의 기본 스타일을 제거하고 싶다면 다음과 같이 작성할 수 있습니다 ! { list-style: none; }

Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 버튼은 css로 작성하시는게 바람직합니다 !

paddingbackground-color 그리고 border-radius 등으로 같은 UI를 출력할 수 있습니다 😊

Copy link
Collaborator

Choose a reason for hiding this comment

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

로고 파일 하나만 있어도 됩니다 !

하나의 로고파일로 크기를 조정하여 개발할 수 있습니다 !
다음과 같이요 !

<img src="..." class="logo" />

// ...
.logo { width: 200px }

Copy link
Collaborator

Choose a reason for hiding this comment

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

또한 벡터 이미지는 svg로 저장하시는걸 추천드려요 !

이하 MDN
SVG(Scalable Vector Graphics)는 2차원 벡터 그래픽을 서술하는 XML 기반의 마크업 언어입니다. SVG는 텍스트 기반의 열린 웹 표준 중 하나로, 모든 사이즈에서 깔끔하게 렌더링 되는 이미지를 서술하며 CSS, DOM, JavaScript, SMIL (en-US) 등 다른 웹 표준과도 잘 동작하도록 설계됐습니다. SVG는 달리 말하자면 HTML과 텍스트의 관계를 그래픽에 적용한 것입니다

image

이미지 파일은 최소 단위가 픽셀로 되어있으며 확대했을 때 이미지가 깨질 수 있어요 !
피그마에서 export하실 때에 svgexport할 수 있습니다 😊

Rester vs Vector

Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 이미지 파일도 css를 통하여 개발되어야 합니다 😊

@kiJu2
Copy link
Collaborator

kiJu2 commented Dec 31, 2024

수고하셨습니다 주하님 !
다음 번에는 css도 함께 적용해보시는걸 추천드릴게요. 😊😊

미션 수행 하시느라 고생 많으셨습니다 !

@kiJu2 kiJu2 merged commit dcd9318 into codeit-bootcamp-frontend:Basic-신주하 Dec 31, 2024
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.

2 participants