Skip to content

[김상우B]스프린트 미션 1,2#34

Open
kksw7000-lang wants to merge 4 commits intocodeit-sprint-fullstack:basic-김상우Bfrom
kksw7000-lang:basic-김상우B-sprint1

Hidden character warning

The head ref may contain hidden characters: "basic-\uae40\uc0c1\uc6b0B-sprint1"
Open

[김상우B]스프린트 미션 1,2#34
kksw7000-lang wants to merge 4 commits intocodeit-sprint-fullstack:basic-김상우Bfrom
kksw7000-lang:basic-김상우B-sprint1

Conversation

@kksw7000-lang
Copy link
Copy Markdown
Collaborator

미션 1, 2

@kksw7000-lang kksw7000-lang added 순한맛🐑 마음이 많이 여립니다.. 제출일 이후 제출한PR 제출일(일요일) 이후에 늦게 제출한 PR입니다. 최종제출 스프린트 미션 최종 제출 PR입니다. 코드리뷰 및 평가해주세요! and removed 제출일 이후 제출한PR 제출일(일요일) 이후에 늦게 제출한 PR입니다. labels Mar 3, 2026
@kksw7000-lang kksw7000-lang changed the title Basic 김상우b sprint1 [김상우B]스프린트 미션 1,2 Mar 3, 2026
@kksw7000-lang kksw7000-lang added 진행중 아직 스프린트 미션 제출일이 아닙니다. 새로 커밋된 내용에 대해 코드리뷰 해주세요! (2주 이상의 스프린트 미션 중간점검 PR 제출시 사용합니다.) and removed 최종제출 스프린트 미션 최종 제출 PR입니다. 코드리뷰 및 평가해주세요! labels Mar 3, 2026
@kksw7000-lang kksw7000-lang requested a review from greatelv March 4, 2026 12:31
Copy link
Copy Markdown
Collaborator

@greatelv greatelv left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다. 기능 구현은 잘 되었으나, 마크업 구조, 웹 접근성, 그리고 CSS 속성 중복 선언 등에서 아쉬운 점이 많이 발견되었습니다. 안정적이고 유지보수하기 쉬운 코드를 위해 라인별로 코멘트를 남겨두었으니, 꼭 확인하시고 꼼꼼하게 수정해 주시기 바랍니다.

box-sizing: border-box;
margin-bottom: 20px;
position: relative; /* 기준점 */
width: 100%;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

동일한 클래스(.input-box) 내에 width 속성이 중복 선언(640px과 100%)되어 있습니다. 의도한 하나의 값만 남겨 불필요한 속성 덮어쓰기 현상을 방지해야 합니다.

css/login.css Outdated
box-sizing: border-box;
}

.input-text-box {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

.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;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

단축 속성(text-decoration: underline;)과 구체적 세부 속성(text-decoration-line 등)이 불필요하게 중복되어 있습니다. 단축 속성 하나만 남겨 코드를 간결하게 관리하는 것을 권장합니다.

css/style.css Outdated

.section1 {
display: flex;
width: 100%px;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

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"/>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

<img> 태그에 alt 속성이 누락되어 스크린 리더 사용자의 웹 접근성이 저하됩니다. 이미지를 설명하는 적절한 텍스트(예: alt="판다마켓 로고")를 반드시 추가해야 합니다. 장식용 이미지의 경우 빈 속성(alt="")이라도 넣어야 합니다.

login.html Outdated

<section class="form-text">

<lable class="lable1">이메일</lable>
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

<lable>은 조재하지 않는 오타이며 올바른 HTML 태그인 <label>로 수정해야 합니다. 또한 웹 접근성과 사용성 개선을 위해 for 속성을 사용하여 연결될 input 요소의 id와 명시적으로 매핑해 주는 것을 강력히 권장합니다.

signup.html Outdated

<section class="form-text">
<lable class="lable1">이메일</lable>
<div class="input-box">
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

여기서도 <label> 태그의 철자 오류(lable)가 있습니다. 올바른 태그로 수정하고, forid 속성을 활용해 폼 요소와의 연결성을 명확히 해주세요.

@kksw7000-lang
Copy link
Copy Markdown
Collaborator Author

kksw7000-lang commented Mar 23, 2026

수정(?) 했고 미션 3,4 push 했습니다

@kksw7000-lang kksw7000-lang added 최종제출 스프린트 미션 최종 제출 PR입니다. 코드리뷰 및 평가해주세요! and removed 진행중 아직 스프린트 미션 제출일이 아닙니다. 새로 커밋된 내용에 대해 코드리뷰 해주세요! (2주 이상의 스프린트 미션 중간점검 PR 제출시 사용합니다.) labels Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

순한맛🐑 마음이 많이 여립니다.. 최종제출 스프린트 미션 최종 제출 PR입니다. 코드리뷰 및 평가해주세요!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants