[김상우B]스프린트 미션 1,2#34
Hidden character warning
[김상우B]스프린트 미션 1,2#34kksw7000-lang wants to merge 4 commits intocodeit-sprint-fullstack:basic-김상우Bfrom
Conversation
| box-sizing: border-box; | ||
| margin-bottom: 20px; | ||
| position: relative; /* 기준점 */ | ||
| width: 100%; |
There was a problem hiding this comment.
동일한 클래스(.input-box) 내에 width 속성이 중복 선언(640px과 100%)되어 있습니다. 의도한 하나의 값만 남겨 불필요한 속성 덮어쓰기 현상을 방지해야 합니다.
css/login.css
Outdated
| box-sizing: border-box; | ||
| } | ||
|
|
||
| .input-text-box { |
There was a problem hiding this comment.
.input-text-box 선택자가 중복 선언되어 이전 스타일(border, background 등)이 무시되고 있습니다. 내부의 input 요소를 타겟팅하려던 의도라면 .input-text-box input과 같은 형태로 수정하는 것이 좋습니다.
css/login.css
Outdated
| font-style: normal; | ||
| font-weight: 500; | ||
| line-height: normal; | ||
| text-decoration-line: underline; |
There was a problem hiding this comment.
단축 속성(text-decoration: underline;)과 구체적 세부 속성(text-decoration-line 등)이 불필요하게 중복되어 있습니다. 단축 속성 하나만 남겨 코드를 간결하게 관리하는 것을 권장합니다.
css/style.css
Outdated
|
|
||
| .section1 { | ||
| display: flex; | ||
| width: 100%px; |
There was a problem hiding this comment.
width: 100%px;와 같이 단위가 잘못 입력된 오타가 있습니다. 단위를 % 또는 px 중 하나로 통일하여 명확하게 수정해야 합니다. (예: 100% 또는 100px)
index.html
Outdated
| <div class="gnb"> | ||
| <div class="gnb-container"> | ||
| <a href="/" class="logo"> | ||
| <img src="./img/logo.png"/> |
There was a problem hiding this comment.
<img> 태그에 alt 속성이 누락되어 스크린 리더 사용자의 웹 접근성이 저하됩니다. 이미지를 설명하는 적절한 텍스트(예: alt="판다마켓 로고")를 반드시 추가해야 합니다. 장식용 이미지의 경우 빈 속성(alt="")이라도 넣어야 합니다.
login.html
Outdated
|
|
||
| <section class="form-text"> | ||
|
|
||
| <lable class="lable1">이메일</lable> |
There was a problem hiding this comment.
<lable>은 조재하지 않는 오타이며 올바른 HTML 태그인 <label>로 수정해야 합니다. 또한 웹 접근성과 사용성 개선을 위해 for 속성을 사용하여 연결될 input 요소의 id와 명시적으로 매핑해 주는 것을 강력히 권장합니다.
signup.html
Outdated
|
|
||
| <section class="form-text"> | ||
| <lable class="lable1">이메일</lable> | ||
| <div class="input-box"> |
There was a problem hiding this comment.
여기서도 <label> 태그의 철자 오류(lable)가 있습니다. 올바른 태그로 수정하고, for와 id 속성을 활용해 폼 요소와의 연결성을 명확히 해주세요.
|
수정(?) 했고 미션 3,4 push 했습니다 |
미션 1, 2