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

Conversation

ghkjiop92
Copy link
Collaborator

@ghkjiop92 ghkjiop92 commented Oct 26, 2024

요구사항

기본

여기 저의 네트리파이 주소 입니다.
https://pandamarket-codeit.netlify.app/

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

image

멘토에게

-아직 부족 하지만 열심히 하겠습니다
확인부탁 드립니다 . 감사합니다

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

@ghkjiop92 ghkjiop92 changed the title [강동우]Sprint12 [강동우]Sprint1 Oct 26, 2024
@ghkjiop92 ghkjiop92 added the bug Something isn't working label Oct 28, 2024
Copy link
Contributor

@withyj-codeit withyj-codeit left a comment

Choose a reason for hiding this comment

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

미션 하느라 수고 많으셨어요~👍

이미지를 바로 넣었던 부분이 많은데,
그래픽적 요소만 이미지를 활용하고 이외에는 html + css로 구성하도록 수정해보셔요~

Copy link
Contributor

Choose a reason for hiding this comment

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

구경하러 가기, 로그인 버튼은 이미지가 아닌 html + css로 만들어 주셔야 해요

Copy link
Contributor

Choose a reason for hiding this comment

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

image
화면의 너비가 넓어졌을 때 하늘색 배경이 덮어주려면
이미지가 아닌 html + css로 만들어 주셔야 해요.
펜더가 포함된 이미지 이외에 문구 + 배경은 만들어 주세요.

<div style="width:882px; height: 720px; background-color: #FFFFFF;">

<div id="desktop_02">
<img src="images/desktop_02.png" alt="인가상품확인">
Copy link
Contributor

Choose a reason for hiding this comment

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

alt 설정 좋아요~👍

margin-right: 1000px;

}
</style>
Copy link
Contributor

Choose a reason for hiding this comment

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

css 파일로 분리하면 스타일링에 대한 관심사를 분리할 수 있어서 좋아요


<body>

<header id="nav">
Copy link
Contributor

Choose a reason for hiding this comment

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

스타일링만을 위한 선택자는 class를 권장해요.

  • id 는 보통 개발작업시 요소 선택을 위한 방편으로 많이 사용해요.
  • id 선택자는 class보다 우선순위가 높아 id에 속한 스타일을 class로 덮어쓸 수 없어 유지보수와 확장성에 불리할 수 있어요.
  • id는 페이지 내에서 고유해서 재사용에 좋지 않지만, class는 재사용 가능해요.

Copy link
Contributor

Choose a reason for hiding this comment

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

다른 페이지로 이동하기 쉽도록 상단에 링크를 두는 요소는 nav태그가 더 좋을 것 같아요.
header태그는 콘텐츠의 제목이나 소개 또는 탐색하는 영역에 사용해요.

</section1>

<section2 id="container2">
<div style="width:882px; height: 720px; background-color: #FFFFFF;">
Copy link
Contributor

Choose a reason for hiding this comment

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

inline style 사용할 경우 id, class 보다 우선순위가 높아 css파일로 스타일 적용이 안되고, 유지보수 확장성에 좋지 않아 일반적으로 사용하진 않고, JavaScript로 값 변경이 필요한 경우 제한적으로 사용하는 걸 권장해요.

</div>
</footer>
</body>
</html>
Copy link
Contributor

Choose a reason for hiding this comment

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

파일의 마지막에 빈 줄이 포함되는 것이 좋아요.
동작에 문제가 있는 건 아니지만 잠재적 문제를 없애기 위함 입니다. (참고)
사용하는 에디터에서 end of line 또는 eol이라는 키워드로 설정할 수 있어요

@withyj-codeit withyj-codeit merged commit 369776d into codeit-bootcamp-frontend:Basic-강동우 Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants