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

Conversation

itscold96
Copy link
Collaborator

@itscold96 itscold96 commented May 24, 2024

배포 주소: panda-market-itscold96.netlify.app

요구사항

기본

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

심화

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

주요 변경사항

  • 반응형 요소 사이즈 변화 구현
  • css 모듈화

스크린샷

스크린샷 2024-05-24 오후 10 51 56

스크린샷 2024-05-24 오후 10 44 15

멘토에게

  • 부족한 부분이 많을 겁니다. 가감없이 꼭 알려주시기 바랍니다.

@itscold96 itscold96 added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label May 24, 2024
@itscold96 itscold96 requested a review from Taero-Kim May 24, 2024 13:56
index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
javascript/functions.js Outdated Show resolved Hide resolved
index.html Show resolved Hide resolved
@Taero-Kim
Copy link
Collaborator

Taero-Kim commented May 24, 2024

작다면 작은 이 프로젝트에서 정말 엄청난 정성이 느껴지는 코드입니다.
미디어 쿼리와 rem을 사용해 브라우저 사이즈에 따른 반응형을 정말 잘 구현하셨습니다!

건드릴게 크게 없어보이는데 그 와중에 사소하게 몇가지 개선하면 좋을 부분에 대해서 코멘트 남겼습니다.

제가 남긴 리뷰에 대한 질문이나 기타 질문이 있으시면 언제든 편하게 말씀해주세요!

+) 리뷰를 남긴 코멘트 위의 p(1~5); 마크의 의미는 아래와 같습니다! 참고하면 좋을 것 같아요!

p1; 꼭 반영해주세요 (Request changes)
리뷰어는 PR의 내용이 서비스에 중대한 오류를 발생할 수 있는 가능성을 잠재하고 있는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우, P1 태그를 통해 리뷰 요청자에게 수정을 요청합니다. 리뷰 요청자는 p1 태그에 대해 리뷰어의 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다.

p2; 적극적으로 고려해주세요 (Request changes)
작성자는 P2에 대해 수용하거나 만약 수용할 수 없는 상황이라면 적합한 의견을 들어 토론할 것을 권장합니다.

p3; 웬만하면 반영해 주세요 (Comment)
작성자는 P3에 대해 수용하거나 만약 수용할 수 없는 상황이라면 반영할 수 없는 이유를 들어 설명하거나 다음에 반영할 계획을 명시적으로(JIRA 티켓 등으로) 표현할 것을 권장합니다. Request changes 가 아닌 Comment 와 함께 사용됩니다.

p4; 반영해도 좋고 넘어가도 좋습니다 (Approve)
작성자는 P4에 대해서는 아무런 의견을 달지 않고 무시해도 괜찮습니다. 해당 의견을 반영하는 게 좋을지 고민해 보는 정도면 충분합니다.

p5; 그냥 사소한 의견입니다 (Approve)
작성자는 P5에 대해 아무런 의견을 달지 않고 무시해도 괜찮습니다.

@Taero-Kim Taero-Kim self-requested a review May 24, 2024 17:16
@Taero-Kim Taero-Kim self-assigned this May 24, 2024
itscold96 added 3 commits May 29, 2024 22:53
- const로 선언 후 매개변수 이름을 바꾸어 좀더 직관적으로 매개변수의 역할을 알 수 있도록 하였음
- 기존 단순 파일에서 util 폴더를 만들고 location.js로 이름을 바꾸어 폴더 구조와 파일 이름을 직관적으로 수정함
- 버튼 태그가 아님에도 .home-footer-buttons로 되어 있던 클래스명을
.home-footer-anchors로 수정

- footer에서 a 태그들의 높이가 박스가 가지는 최대 범위로 가지고 있던 것을
height: fit-content 속성을 추가하여 높이를 컨텐츠 크기에 맞춤
- 상수화된 변수를 생성
- 현재 진행하고 있는 프로젝트의 특성 상, HTML 파일 내에서는 JSX를 사용할 수 없기 때문에 템플릿 리터럴로 상수를 import 해오는 것이 불가능함
- 물론 getElementBy 류 메서드를 활용하여 속성과 내부 HTML을 생성하는 것은 가능한 부분이지만 그것은 이 스프린트 프로젝트의 활용 범위를 넘어서는 것이라 생각함
@itscold96 itscold96 closed this May 30, 2024
@itscold96 itscold96 reopened this May 30, 2024
@Taero-Kim Taero-Kim closed this Jun 4, 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