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

code update #2

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

code update #2

wants to merge 4 commits into from

Conversation

joji7752
Copy link

No description provided.

index.html Outdated

<nav>
<ul>
<a href="#"> <li class="filter filter-p"><img src="img/blue_p.png" alt=""></li></a>
Copy link
Contributor

Choose a reason for hiding this comment

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

<li> 태그 안에 <a>가 들어가야 할 거 같네요!

src/main.js Outdated
Comment on lines 1 to 7
const mainUl = document.querySelector('.product-ul')
const filterP = document.querySelector('.filter-p')
const filterT = document.querySelector('.filter-t')
const filterS = document.querySelector('.filter-s')
const filterBlue = document.querySelector('.filter-blue')
const filterYellow = document.querySelector('.filter-yellow')
const filterPink = document.querySelector('.filter-pink')
Copy link
Contributor

Choose a reason for hiding this comment

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

전반적으로 main.js 파일에 세미콜론이 존재하지 않네요.
물론 자바스크립트는 줄바꿈을 세미콜론으로 인식하기 때문에 세미콜론이 없어도 작동은 합니다!
근데 세미콜론이 필요하지만 자바스크립트가 그걸 인식하지 못하는 경우도 있어 에러가 발생할 수 있다고 하네요!
자세한 사례는 링크의 에러 예제부분을 참고하시면 좋을 거 같아요! ☺

p.s 저 같은 경우는 'prettier' 라는 비주얼 스튜디오 코드 확장 프로그램을 사용 중인데, 알아서 필요한 부분에 세미콜론을 붙여줘서 좋더라고요! 영지님도 한번 사용해보세요~!

src/main.js Outdated
return value.src.contain('pink')
})
test.forEach((value) => listshow(value))
})
Copy link
Contributor

Choose a reason for hiding this comment

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

지금 마지막에 ⛔ 이러한 이모티콘이 있는 이유는 개행을 하지 않아서인데요,
파일 끝에는 항상 개행을 해야 한다고 하네요!
그렇지 않으면 이 파일이 아직 끝나지 않았다고 인식을 하기 때문인데요,
이거를 EOL(End of Line)이라고 표현하더라고요.
자세한 내용은 참고 링크를 보시고 이해하시면 될 거 같습니다!

p.s 이또한 'prettier' 확장 프로그램을 사용하시면 자동으로 EOL을 해주기 때문에, 'prettier' 사용을 추천드려요~!

src/main.js Outdated
Comment on lines 1 to 7
const mainUl = document.querySelector('.product-ul')
const filterP = document.querySelector('.filter-p')
const filterT = document.querySelector('.filter-t')
const filterS = document.querySelector('.filter-s')
const filterBlue = document.querySelector('.filter-blue')
const filterYellow = document.querySelector('.filter-yellow')
const filterPink = document.querySelector('.filter-pink')
Copy link

Choose a reason for hiding this comment

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

yeongjiCho님 코드 잘 봤어요😘 질문 몇 가지 하려구요!! 혹시 변수 선언을 하시고 ; 을 안하신 이유가 있을까요? 제 의견으로는 혹시나 이런 사소한 게 나중엔 큰 오류를 가져 올 것 같습니다!

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.

3 participants