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

Conversation

BossTeemo
Copy link

@BossTeemo BossTeemo commented May 24, 2024

요구사항

기본

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

심화

  • palette에 있는 color값들을 css 변수로 등록하고 사용해 주세요.
  • 사용자의 브라우저가 크고 작아짐에 따라 페이지의 요소간 간격, 요소의 크기, font-size 등 모든 크기와 관련된 값이 크고 작아지도록 설정해 보세요.(설정값은 자유입니다)

주요 변경사항

-netlify에 업로드하였습니다.
링크

  • html파일 작성 (index, faq, items, login, privacy)
  • css파일 작성 (style.css)
  • 아이콘들은 icons폴더에 저장
  • 이미지들은 images폴더에 저장

스크린샷

스크린샷 2024-05-25 오전 1 00 47 스크린샷 2024-05-25 오전 1 00 36

멘토에게

-banner-top안의 text와 banner-bottom의 text가 같은 css속성인데 다르게 동작합니다.
창을 줄였을 때, banner-top의 text는 그대로 있으면서 화면밖으로 나가는 만큼 잘리는 방식으로 동작하고(의도한 바가 아님)
banner-bottom의 text는 css에서 .text에 지정한 아래의 코드가 제대로 적용되어 동작합니다.
왜 banner-top에서는 이상하게 동작하는지 모르겠습니다.
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;

  • �또, banner-top에서 판다이미지를 어떻게 바닥에 붙여야 좋을지 잘 모르겠습니다.
    저는 일단 relative속성을 이용해 bottom: 으로 -px값을 줘서 붙였는데, 이렇게하면서
    이미지크기를 유동적으로 만들수가 없습니다.
    어떻게하는것이 좋은방법일까요?

@BossTeemo BossTeemo added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label May 24, 2024
@BossTeemo BossTeemo requested a review from 201steve May 24, 2024 16:25
Copy link
Collaborator

@201steve 201steve left a comment

Choose a reason for hiding this comment

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

banner-top안의 text와 banner-bottom의 text가 같은 css속성인데 다르게 동작합니다.
창을 줄였을 때, banner-top의 text는 그대로 있으면서 화면밖으로 나가는 만큼 잘리는 방식으로 동작하고(의도한 바가 아님)
banner-bottom의 text는 css에서 .text에 지정한 아래의 코드가 제대로 적용되어 동작합니다.
왜 banner-top에서는 이상하게 동작하는지 모르겠습니다.
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;

  • banner-top은 text까지 가는데에 banner-top,banner-top-container,banner-top-component를 거쳐서 가야하는 반면, banner-bottom는 banner-bottom,banner-bottom-container이렇게 2개의 div하위에 위치하고있습니다.
.banner-top-component {
    overflow: hidden;
}
  • 이 속성을 추가해서 부모 요소가 overflow:hiddend이 되어야 자식요소에서 말줄임표 처리가 가능해집니다.

또, banner-top에서 판다이미지를 어떻게 바닥에 붙여야 좋을지 잘 모르겠습니다.
저는 일단 relative속성을 이용해 bottom: 으로 -px값을 줘서 붙였는데, 이렇게하면서
이미지크기를 유동적으로 만들수가 없습니다.
어떻게하는것이 좋은방법일까요?

.banner-top-image{
   align-self: flex-end 
}

요렇게 하면 됩니다.
부모 요소인 banner-top-container가 flex box여서, 자식요소 각각이 독립적으로 align 스타일링이 필요할 때 align-self 속성을 쓸 수 있습니다.

Comment on lines +13 to +18
<a class="goto_root" href="./index.html">
<img src="./icons/Property 1=Variant3.svg" alt="main logo">
</a>
<a class="button-login" href="./login.html">
로그인
</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<a class="goto_root" href="./index.html">
<img src="./icons/Property 1=Variant3.svg" alt="main logo">
</a>
<a class="button-login" href="./login.html">
로그인
</a>
<a class="goto-root" href="./index.html">
<img src="./icons/Property 1=Variant3.svg" alt="main logo">
</a>
<a class="button-login" href="./login.html">
로그인
</a>

선택자, 변수, 함수, 등등 식별자를 이름지어야 할 때 naming convention을 여러개 혼용해서 사용하지 않는게
코드를 읽기 쉽고, 이해하기 쉽게 만드는 방법중 하나 입니다.

naming convention은 종류가 많은데, 일반적으로 널리 알려진것은
camelCase, snake_case, kebaa-case, UPPERCASE,PascalCase 등이 있습니다.

예를들어, 자바스크립트는 대부분 camelCase를 사용하고, 종종 PascalCase를 사용하기도 합니다.

중요한것은 제품을 만드는 사람이 편한 것보다, 읽는 사람이 이해하기쉽게 볼 수 있게 만드는게 중요합니다.

</section>

<main>
<section class="section1">
Copy link
Collaborator

Choose a reason for hiding this comment

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

소프트웨어는 항상 바뀔 수 있습니다.
지금은 section1이지만, 나중엔 이 순서 혹은 위치가 바뀔 수도 있습니다.
역할을 생각해보시면 좀더 변경에 유연한 이름 짓기가 될꺼에요.

line-height: 24px;
}

main{
Copy link
Collaborator

Choose a reason for hiding this comment

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

태그 선택자는 지양하는 것이 좋습니다.
지금은 혼자서 개발하지만, 팀 단위로 개발하면 선택자 사용에 중복되거나, 의도치않은 영향을 줄 수 있습니다.
class 선택자를 사용하는걸 권장합니다. 하나의 태그에 복수의 class를 선언하는 것도 가능합니다.

Comment on lines +110 to +112
main section .text {
margin: 12px 0 24px ;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

특정 class를 부모 요소부터 찾아서 잘 적용하셨습니다.

특정 class를 나타내는 class를 하나 더 부여해서 사용하는 다른 방법도 있습니다.

예를들어,

<main>
  <section>
     <div class="text">...</div>
  </section>
</main>

main section .text 로 찾는경우 main부터 찾게 되는데,
<div class="text subtitle">
이렇게 class를 하나 더 추가해주면

조부모, 부모 등을 거치지않고 .subtitle로 바로 특정지을 수 있습니다.

height: 720px;
display: flex;
align-items: center;
gap: 0px 64px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
gap: 0px 64px;
gap: 0 64px;

0은 단위를 붙이지않아도 됩니다

Comment on lines +40 to +42
<div class="tag">Hot item</div>
<div class="text">인기 상품을<br>확인해 보세요</div>
<div class="text-info">가장 HOT한 중고거래 물품을<br>판다 마켓에서 확인해 보세요</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<div class="tag">Hot item</div>
<div class="text">인기 상품을<br>확인해 보세요</div>
<div class="text-info">가장 HOT한 중고거래 물품을<br>판다 마켓에서 확인해 보세요</div>
<div class="tag">Hot item</div>
<div class="text">인기 상품을<br>확인해 보세요</div>
<div class="text-info">가장 HOT한 중고거래 물품을<br>판다 마켓에서 확인해 보세요</div>

화면상 이 영역이 모두 영역을 나타내는 태그로 쓰여야 하는지,
아니면 어떤 역할을 하는지 생각해보시고 적절한 태그를 사용해보세요.

@201steve 201steve merged commit 3d20d1d into codeit-bootcamp-frontend:Basic-엄세환 May 28, 2024
1 check passed
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