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.

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

KDT5_BaekDongEun #50

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

KDT5_BaekDongEun #50

wants to merge 3 commits into from

Conversation

debeck6
Copy link

@debeck6 debeck6 commented Apr 5, 2023

프론트엔드 과정 5기 백동은입니다.

원본 사이트 경로: https://google.com/
클론 사이트 경로:https://googleclone0000.netlify.app/

검색 기능까지는 구현하였으나 상단 메뉴는 완전히 구현하지 못했습니다

@happyhermann
Copy link

h1태그는 한 페이지에 단 한개 밖에 쓰지 못하기에 아쉬운 부분이있네요
그리고 h1 태그내에 텍스트가 아닌 span태그가 들어간 것은 시멘틱하게 볼 수 없어서 나중에는 유의하시면 될거 같습니다
form태그로 검색 기능 구현한 것은 정말 좋았습니다!

전체적으로 깔끔하네요

고생하셨습니다

@ChoEun-Sang
Copy link

검색 기능 구현하신 부분 잘하신 거 같아요!

@dev-junehee
Copy link

SCSS 활용하신 부분 좋은 것 같아요! 배우고 갑니다 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants