KDT0_JangSuBin2 알툴즈 클론코딩 (기존 PR close 후 새로운 PR)#72
Open
wkdtnqls0506 wants to merge 3 commits intomainfrom
Open
KDT0_JangSuBin2 알툴즈 클론코딩 (기존 PR close 후 새로운 PR)#72wkdtnqls0506 wants to merge 3 commits intomainfrom
wkdtnqls0506 wants to merge 3 commits intomainfrom
Conversation
|
고생하셨어요 👍
|
|
고생 많으셨고 너무 잘하신 것 같습니다!
|
|
리드미도 깔끔하고 클래스 이름을 보기 쉽게 작성해주신 것 같아요! |
|
다들 생각이 비슷했던거 같습니다 ㅋㅋ 저도 리드미에 모든 페이지가 잘 정리되어 있어서 한 눈에 보기 편했어요! Navbar 저랑 같은 디자인으로 구현하셨길래 정말 반가웠습니다. 페이지도 깔끔하게 잘 만드셨습니다. 수고하셨어요~ |
jungHyeonS
reviewed
Aug 5, 2023
jungHyeonS
left a comment
jungHyeonS
left a comment
There was a problem hiding this comment.
수빈님 정말로 고생하셨습니다~
너무 깔끔하게 작성해주셔서 제가 따로 코드리뷰를 크게 해드릴 부분이 없는거같습니다
스크립트 에 defer 속성까지 정말로 잘작성해주신거같고 추후에는 웹 접근성 부분한 한번 고민해주시면 좋을꺼깥습니다~
| <p class="menu1__title">Tool</p> | ||
| <ul class="hover__tool"> | ||
| <li class="hover__tool__item"> | ||
| <img class="tool__item__img" src="./images/icon_알집.png" /> |
There was a problem hiding this comment.
웹 접근성을 위해 이미지 태그에 alt 태그를 꼭 넣어주시면 좋을꺼같습니다~
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💊 알툴즈 홈페이지 클론코딩 💊
🧷 알툴즈 url : https://altools.co.kr/
🧷 알툴즈 클론코딩 url : https://64c3784440213655395c11f1--friendly-cendol-995833.netlify.app/
🗓️ 과제 기간
2023.07.24 ~ 2023.07.28
🌀 과제 요구사항
필수 요구사항
[✔️] 과제에 대한 설명을 포함한
README.md파일을 제공하세요![✔️] 과제 결과와 비교할 수 있는 실제 사이트(페이지)의 주소를 명시하세요!
[✔️] 과정에서 사용한 프로젝트 폴더/파일이 모두 포함돼야 합니다, 일부 파일만 제출하지 마세요!
[✔️] 실제 서비스로 배포하고 접근 가능한 링크를 추가해야 합니다.
선택 요구사항
[✔️]
<header>,<section>등 시멘틱 태그를 최대한 활용해보세요.[✔️] 실제 사이트의 레거시 코드 활용보단 최신의 CSS Flex 혹은 Grid 등을 활용해보세요.
[ ] 부분적으로 BEM 방법론을 도입해보세요.
[✔️] JS가 필요한 부분은 되도록 생략하되 이유를 명시해보세요.(CSS로 대체 가능한지 피드백이 있을 수 있겠죠?!)
[✔️] JS가 필요한 부분 중 구현할 부분이 있다면 자유롭게 구현해보세요.(JS 과제가 아니니까 가볍게 구현하시길 추천해요)
[ ] SCSS 등의 CSS 전처리도구를 도입해보세요.
[ ] SCSS 컴파일에 Webpack이나 Parcel 같은 번들러를 활용해보세요.
🖥️ 화면 구성
✔️ JavaScript를 활용한 구현
📖 더 보완해야 할 점
◻️ 이미지 슬라이드 부분의 구현 - css로만 구현하려고 하니 꽤나 복잡해서 이미지만 삽입했습니다..🥲
◻️ 반응형 모바일 웹 사이트도 구현해보기
◻️ css를 작성할 때, 공통이 되는 부분은 root를 사용하여 자주 쓰는 조건들을 따로 변수로 설정하기
◻️ html을 좀 더 간결하게 표현될 수 있도록 수정하기
◻️ html class 이름을 좀 더 직관적으로 보일 수 있게 보완하기