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.

Sign up for GitHub

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

Jump to bottom

[Mission1] 로그인 페이지 레이아웃 #1

Open
saengmotmi opened this issue Jul 14, 2021 · 0 comments · Fixed by #452
Open

[Mission1] 로그인 페이지 레이아웃 #1

saengmotmi opened this issue Jul 14, 2021 · 0 comments · Fixed by #452
Labels
필수 구현 사항

Comments

@saengmotmi
Copy link
Contributor

saengmotmi commented Jul 14, 2021 •
edited
Loading

✔️ 코드 리뷰 시 중점적으로 체크하는 사항입니다. 관련 개념 및 리뷰 내용 숙지하고 코드 작성 시에 더욱 신경써주세요!


Semantic tag의 적절한 사용

HTML에는 다양한 태그가 존재하며 그 중에서도 Semantic tag를 적절하게 사용하는 것이 가독성 측면에서도, SEO 측면에서도 너무나 중요합니다. 무조건적으로 div 태그를 남발하기 보다는 상황에 맞게 적절한 태그를 사용할 수 있도록 신경써주세요.

📎 참고자료


id, class 이름, 변수명, 함수명

요소에 id 혹은 class 를 부여하는 경우 의미가 직관적으로 드러날 수 있도록 신경써주세요. 코드는 쓰는 경우보다 읽히는 경우가 훨씬 많습니다! id, class를 비롯해 자바스크립트 변수명, 함수명을 짓는게 처음엔 쉽지 않기 때문에 지금부터 고민해서 작성하는 습관을 반드시 길러주세요.

📍 관련 PR & Review 보기

📎 참고자료


하나의 요소에 여러가지 CSS 속성을 부여하는 경우 규칙성(convention) 갖추기

하나의 요소에 복수의 CSS 속성을 부여하는 경우에도 우선순위와 관련도에 따라 나름의 convention을 갖고 CSS 코드를 작성할 수 있습니다. 예를들어 우선순위를 고려해 width, height 속성을 background-color 속성보다 먼저 정의해볼 수 있고, font-size, font-weight 와 같이 연관있는 속성들은 붙여서 작성할 수 있습니다.

📍 관련 PR & Review 보기


HTML, CSS 복습하기

westagram 프로젝트를 하는 이유는 그동안 배운 HTML, CSS, JavaScript 개념을 복습하기 위함입니다. HTML <head> 태그부터 시작해서 큰 구조를 먼저 파악해보시고, <title> 태그 내용도 기존의 repl.it 보다 멋진 이름으로 바꿔보세요!

📍 관련 PR & Review 보기


불필요한 엔터 삭제, 필요한 엔터 추가, 테스트 마친 console.log 삭제(또는 주석처리)

코드는 내가 코드를 치는 것보다 남들에게 읽히는 경우가 훨씬 많습니다. 특히 팀 프로젝트를 진행하면 내가 작성한 코드가 팀원들과 합쳐지기 때문에 코드를 깔끔하게 유지하는 것이 더욱 중요해집니다. 이 점을 염두해두로 최대한 코드의 가독성을 높이기 위해 신경써주세요.

📍 관련 PR & Review 보기


개발자 도구의 활용

개발자도구는 가로/세로 길이, 여백, 색깔, 폰트 크기, border-radius 같은 스타일 속성만 참고하는 것입니다. html이나 css를 그대로 복사해서 사용하면 개발이 복잡해지고 더 이상 스스로 유지보수 하기 어려워 집니다. 개발자 도구에서 필요한 정보와 그렇지 않은 정보를 구별해서 취해주세요.
@SK0414 SK0414 linked a pull request Sep 9, 2021 that will close this issue
[25기 홍승균] 필수구현사항 완료 #75
Closed
3 tasks
@SK0414 SK0414 removed a link to a pull request Sep 9, 2021
[25기 홍승균] 필수구현사항 완료 #75
Closed
3 tasks
@KH2Wone KH2Wone linked a pull request Dec 8, 2021 that will close this issue
[28기 강희원][Add] 추가 기능 구현중 #156
Closed
10 tasks
@KH2Wone KH2Wone removed a link to a pull request Dec 11, 2021
[28기 강희원][Add] 추가 기능 구현중 #156
Closed
10 tasks
@seoltang seoltang linked a pull request Jan 5, 2022 that will close this issue
[29기 박겸영] 필수 항목 구현 / 댓글 좋아요, 삭제 기능 #187
Closed
5 tasks
@seoltang seoltang removed a link to a pull request Jan 5, 2022
[29기 박겸영] 필수 항목 구현 / 댓글 좋아요, 삭제 기능 #187
Closed
5 tasks
@hyunkark hyunkark linked a pull request Jan 6, 2022 that will close this issue
[29기 곽승현] Add: 필수 구현 사항 추가 #171
Closed
5 tasks
@wkdwhdghks wkdwhdghks linked a pull request Mar 11, 2022 that will close this issue
[31기 장종환] ADD : login 페이지 개발 완료 #225
Closed
5 tasks
@JaySurplusYoon JaySurplusYoon mentioned this issue Mar 12, 2022
Closed
5 tasks
@parkjooyoung0608 parkjooyoung0608 linked a pull request May 4, 2022 that will close this issue
[33기 박주영] Mission 1~4 구현완료, 추가 기능 구현중 #294
Closed
5 tasks
@DevSeulgi DevSeulgi linked a pull request May 6, 2022 that will close this issue
[33기 박슬기] Mission 1 ~ 4 완료 #288
Closed
5 tasks
This was unlinked from pull requests May 23, 2022
[29기 곽승현] Add: 필수 구현 사항 추가 #171
Closed
[31기 장종환] ADD : login 페이지 개발 완료 #225
Closed
[33기 박주영] Mission 1~4 구현완료, 추가 기능 구현중 #294
Closed
[33기 박슬기] Mission 1 ~ 4 완료 #288
Closed
@AliceNam93 AliceNam93 linked a pull request Oct 28, 2022 that will close this issue
[39기 남연우] Mission 1-3 완료, 나머지 미션 수행중 #452
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
필수 구현 사항
Projects
None yet
Milestone
No milestone
1 participant
@saengmotmi