Conversation
ghost
closed this
ghost
requested a review
from 
|
수고 하셨습니다 예진님 ! 위클리 미션 하시느라 수고 많으셨습니다 ! |
commit 단위를 더욱 자주, 작게 해보시는건 어떠실까요?git을 다룰 때 commit은 "언제 해야 하는가"를 생각해보신 적 있으신가요?
그럼 커밋을 언제 해야 할까요?저는 다음과 같은 룰을 지키며 커밋을 하는걸 권장 드립니다:
관련하여 읽으시면 좋은 아티클을 추천드릴게요:tl;dr관련 변경 사항 커밋 자주 커밋 미완성 작업을 커밋하지 마십시오 커밋하기 전에 코드를 테스트하세요 또한 깃 커밋 메시지 컨벤션도 함께 읽어보세요:tl;dr:커밋 메시지 형식 type: Subject
body
footer기본적으로 3가지 영역(제목, 본문, 꼬리말)으로 나누어졌다. 메시지 type은 아래와 같이 분류된다. 아래와 같이 소문자로 작성한다. feat : 새로운 기능 추가 |
There was a problem hiding this comment.
파일명 잘 지었네요 ! 딱 봐도 어떤 이미지인지 알겠어요 👍
| @@ -0,0 +1,99 @@ | |||
| <!DOCTYPE html> | |||
| <head> | |||
There was a problem hiding this comment.
<html> 태그가 빠진 것으로 보여요 !
| <head> | |
| <html> | |
| <head> |
| <body> | ||
| <header> | ||
| <a href="/"> | ||
| <img src="images/panda-market-logo.png" width="153"> |
There was a problem hiding this comment.
alt를 작성해볼까요?
alt는 스크린 리더 사용자에 대한 보조 텍스트가 될 수 있으므로 "어떠한 이미지 인지"를 작성해주는 것이 좋아요 !
꼭 작성하시는걸 �제안드립니다 !
alt의 목적
- 인터넷 연결이 끊겼을 때 대체되는 이미지
- 스크린 리더 사용자를 위한 대체 텍스트
- 이미지를 볼 수 없는 환경에서 이미지를 대체하기 위한 텍스트
등 목적을 알게 된다면 alt를 어떻게 사용하시면 될지 알 수 있을 것 같아요.
다음은 하버드 에듀케이션에서 제안하는 alt 규칙입니다:
tl;dr
- Write Good Alt Text
- Add alt text all non-decorative images.
- Keep it short and descriptive, like a tweet.
- Don’t include “image of” or “photo of”.
- Leave alt text blank if the image is purely decorative
- It's not necessary to add text in the Title field.
| background-color: #1967d6; | ||
| } | ||
|
|
||
| #sign-button { |
There was a problem hiding this comment.
네이밍을 일관적으로 작성하시는게 어떨까요?
파스칼 케이스, �케밥 케이스, 카멜케이스 등 여러가지 섞여있는 것 같아요. 네이밍을 일관적으로 작성하시는게 어떨까요 ?
| a { | ||
| text-decoration: none; | ||
| color: inherit; | ||
| } |
There was a problem hiding this comment.
reset.css를 작성하셔도 될 것 같아요.
해당 스타일은 reset.css 파일을 만들어서 작성하셔도 될 것 같아요 =)
요구사항
기본
심화
주요 변경사항
스크린샷