Skip to content

[투두 리스트 만들기] 심현진 미션 제출합니다.#3

Open
hxeonxins wants to merge 22 commits intoinsertcourse24:mainfrom
hxeonxins:hxeonxins
Open

[투두 리스트 만들기] 심현진 미션 제출합니다.#3
hxeonxins wants to merge 22 commits intoinsertcourse24:mainfrom
hxeonxins:hxeonxins

Conversation

@hxeonxins
Copy link
Copy Markdown

흥미 있었던 점:
이전 당근 클론 코딩에서는 자바스크립트가 많이 안쓰였는데 투두리스트에선 자바스크립트가 대부분이었다. 2개의 js미션을 통해서 자바스크립트에 더 익숙해질 수 있었던 것 같다. 또, 자바스크립트 파일을 원래 head다가 불러왔는데 기능이 안먹혀서 body 맨 밑으로 옮기니까 구현이 되서 신기했다. 이번 과제를 통해 화면구현 시간에 배운 내용도 복습하고 자바스크립트를 익히는 과정이 유익했다.

어려웠던 점:
수업 시간에 배운 개념들도 있어서 크게 어렵게 느껴지진 않았다. 하지만 기능을 추가하던 중 js코드에는 문제가 없는데 html에서 시멘틱 태그로 감싼곳에 스타일을 주면 js가 아예 안되는 에러가 있었다. 최대한 스타일을 안주는 것으로 해결했다..

궁금한 점:
위에 있었던 에러의 이유가 궁금하다.. 분명 html기본 틀에서 js코드는 정상적으로 작동하는데 시멘틱 태그를 감싸고 스타일을 주면 안된다. 이유를 못찾아서 디자인이 거의 없다.

@kimsiyeon0223
Copy link
Copy Markdown
Contributor

kimsiyeon0223 commented Aug 1, 2024

정말 수고하셨어요!!!~~ 넘 잘하셨습니다

html 문서는 코드의 맨 위에서부터 맨 아래로 파싱되기 때문에 script 태그를 만나게 되면 파싱을 중단하고 스크립트를 먼저 불러오게 됩니다. 스크립트를 다 불러오면 그때 html을 다시 파싱하기 시작합니다. 이로 인해서 자바스크립트의 용량이 커지면, 사용자가 웹페이지를 보기까지의 시간이 오래 걸린다는 단점이 있죠~~ 아주 좋은 시도였지만 코드를 짜고 수정해나갈때 '왜 그런거지?' or '왜?'라는 생각을 가지도록 합시다!!!

오잉... 시멘틱 태그에 스타일을 줬을 때 자바스크립트가 실행이 안 되는 경우는 저도 처음 보네요... 그럴리가 없는데 말이죠... 그런 이상한 오류가 있다면 저희에서 질문해주세요ㅠㅜㅠㅜㅠ 저희도 같이 배워간답니다... 안됐던 코드 카톡으로 보내주세요!!! 같이 고민해봅시다~~~

그리고 마지막으로 마감기한!!!!! 꼭 지키도록 합시다. 첫 과제도 이번 과제도 항상 마감기한을 가장 강조했던 것 같은데... 미리 연락을 취한 것은 좋다만 불가피한 사정이 아니라면 마감기한을 지켜야하는 것이 맞겠죠???? 다음부터는 꼭 지키도록 합시당~~

Comment thread practice/practice1.js Outdated
calculateResult();

//4단계 - 화살표 함수 익히기
const calculate2 = {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

위에 만들어둔 calculate 객체를 사용해도 될 것 같아요!

Comment thread practice/practice2.js Outdated
const length = inputString.length;
const isEmpty = length === 0;

if (!!isEmpty) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if (!!inputString)

이렇게 사용하면 length와 isEmpty가 없어도 공백을 구분할 수 있을 것 같습니다!

Comment thread todolist/script.js

const checkbox = document.createElement('input');
checkbox.type = 'checkbox';
checkbox.checked = newTodo.completed || false; // 초기 상태는 완료 여부 반영
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

주석을 달아가며 공부하는 습관은 너무 좋은 것 같습니다~~

Comment thread todolist/style.css
border: none;
border-radius: 5px;
background-color: #f5f5f5;
font-size: 20px;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

다른 단위들은 상대적인 단위들을 사용하셨네요!! 좋습니다~
그렇다면 font-size도 rem이라는 상대적인 단위를 사용해보는 것이 어떨까요?

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.

2 participants