Skip to content

[투두 리스트 만들기] 권민재 미션 제출합니다.#4

Open
kwonminjae5700 wants to merge 24 commits intoinsertcourse24:mainfrom
kwonminjae5700:minjae
Open

[투두 리스트 만들기] 권민재 미션 제출합니다.#4
kwonminjae5700 wants to merge 24 commits intoinsertcourse24:mainfrom
kwonminjae5700:minjae

Conversation

@kwonminjae5700
Copy link
Copy Markdown

흥미로웠던 점 : javascript 를 잘하면 진짜 모든 걸 할 수 있구나 라는 것을 느꼈으며 개발을 하면 할 수록 코드를 잘 작성하는 제 모습이 코드에서 보여서 정말 뿌듯했습니다. 이번 미션 덕분에 헷갈리던 javascript 개념에 대해서 확실하게 기초를 다질 수 있었습니다. 이때까지는 이미 있는 페이지를 클론코딩 하는 식의 코딩을 했다면 이번에는 직접 처음부터 페이지를 구상하고 개발할 수 있어서 정말 즐거웠습니다. 또한 마지막에 제가 만든 투두리스트 사이트가 완벽하게 작동하는 것을 보고 프런트엔드에 정말 큰 재미가 생겼습니다.

어려웠던 점 : 처음 일정 삭제 기능을 개발할 때 그 버튼에 해당하는 일정을 삭제하는 과정을 개발하는 것이 정말 어려웠습니다. 아직 CSS 를 다루는 것이 많이 어려운 거 같습니다. 아직까지 div 의 위치를 배치하는 과정이 어렵습니다..

궁금한 점 : javascript 를 개발할 때 제 코드처럼 저렇게 변수를 많이 생성해도 되는 지 궁금합니다. 그리고 기능마다 javascript 페이지를 나눠서 개발하는 것이 효율적인 것인지도 궁금합니다!

@kimsiyeon0223
Copy link
Copy Markdown
Contributor

너무 수고많으셨어요~~~
항상 질문 많고 열정적인 모습을 보여주셔서 감사홥니다ㅠㅠ
그리고 혼자 에러에 부딫히고 해결해 나가는 모습이 아주 기특합니다!
계속 이런 모습을 유지하길 바라요 화이팅!!

css는 많은 경험이 가장 도움이 됩니다... ㅜㅜ 구글링 많이 해보세요!!ㅎㅎ

기능에서 필요한 변수라면 다 선언해줘야겠죠???
기능마다 파일을 나누는 것은 때에 따라 다를 것 같아요.. 자기 하기 나름이기도 해요!! 하지만 같은 변수를 쓰는 기능들이나 비슷한 기능들은 같은 파일 내에 작업하시는 것을 추천합니다!! 좋은 질문인 것 같아요~~ 곧 있을 리엑트 과제에서 생각해봅시다!!

Comment thread practice/practice2.js
@@ -1 +1,28 @@
// Mission 2
let coffee = 10

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 Outdated

#insert_button {
background-color : #5b75e2;
width : 70px;
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.

width를 직접적으로 주기보다는 padding을 사용하는 것이 좋을 것 같아요!!

Comment thread todolist/style.css Outdated
}

#today {
font-size : 38px;
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.

다른 단위들은 상대적인 단위들을 사용했네요!!
그러면 폰트 사이즈도 rem이라는 상대적인 단위를 사용해보면 어떨까요??

Comment thread todolist/style.css
@kwonminjae5700
Copy link
Copy Markdown
Author

피드백 해주신대로 리팩토링 하였습니다!

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