Basic 김현제 #15
Basic 김현제 #15
Conversation
ranflir
added
매운맛🔥
|
Sprint mission 1, 2 |
| <!-- 여기까지 1차 라이브 코딩 내용 --> | ||
|
|
||
| <section id="homes"> | ||
| <section class="home1"> |
There was a problem hiding this comment.
class naming에 숫자를 넣는걸 지양해주세요 좀 더 의미있는 네이밍을 추구해주셨으면 합니다.
| <img id="logo" src="imgs\logo.svg" /> | ||
| </a> | ||
|
|
||
| <div class="title"> |
There was a problem hiding this comment.
form 태그에 대해서 더 공부해보고 감싸보면 좋을거 같아요
| <img id="logo" src="imgs\logo.svg" /> | ||
| </a> | ||
|
|
||
| <div class="title"> |
There was a problem hiding this comment.
title이라는 class 네이밍이 조금 이상해보이는데요. 클래스 이름 지을때도 네이밍에 대해서 고민해보면 좋을거 같아요.
| </label> | ||
| </div> | ||
|
|
||
| <div class="signup-button">회원가입</div> |
There was a problem hiding this comment.
실제 간편 로그인은 아니고 oauth 로그인이라 네이밍에도 이게 반영되면 좋을거같고 추후에 oauth 인증에 대해서 공부를 하면 좋을거 같습니다.
|
|
||
| <div id="already"> | ||
| 이미 회원이신가요? | ||
| <a id="al-login" href="login.html">로그인</a> |
There was a problem hiding this comment.
class네이밍을 정말 다른사람들이 알아볼 수 있게 쓰시면 좋을거 같습니다. al은 already에 축약형이라고 생각하는 사람이 많지는 않을꺼 같아서요.
| </div> | ||
| </div> | ||
|
|
||
| <div id="already"> |
There was a problem hiding this comment.
id는 필요할 때 unique 할때만 사용하고 class를 우선적으로 사용하면 좋을거 같습니다. 실제로 나중에 추가하다보면 unique하지 않을수도 있어서요.
|
|
||
| <footer> | ||
| <div>©codeit - 2024</div> | ||
| <div id="footerMenu"> |
There was a problem hiding this comment.
div 말고 시맨틱 태그를 이용해 다른 html 요소를 표현해보면 좋을거 같아요. 그리고 html tag가 어떤걸 의미하는지도 공부해보면 좋을거 같습니다.
|
감사합니다!
다시 살펴보겠습니다!
2025년 12월 7일 (일) 오전 5:39, seungjinWoo ***@***.***>님이 작성:
… ***@***.**** commented on this pull request.
------------------------------
In Index.html
<#15 (comment)>
:
> + </section>
+ <footer id="body-bottom">
+ <div class="container2">
+ <h1>
+ 믿을 수 있는
+ <br />
+ 판다마켓 중고 거래
+ </h1>
+ </div>
+ <img src="imgs\Img_home_bottom.png" alt="믿을 수 있는 중고마켓" />
+ </footer>
+ </main>
+
+ <footer>
+ <div>©codeit - 2024</div>
+ <div id="footerMenu">
div 말고 시맨틱 태그를 이용해 다른 html 요소를 표현해보면 좋을거 같아요. 그리고 html tag가 어떤걸 의미하는지도
공부해보면 좋을거 같습니다.
—
Reply to this email directly, view it on GitHub
<#15 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/B2DLLZWOKYCZOOKGIFRIFVD4AM5JFAVCNFSM6AAAAACOELMHAKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTKNBYGI2TIOJXHA>
.
You are receiving this because you were assigned.Message ID:
<codeit-sprint-fullstack/11-sprint-mission-fe/pull/15/review/3548254978@
github.com>
|
2 participants
No description provided.