Conversation
lemoncurdyogurt
left a comment
lemoncurdyogurt
left a comment
There was a problem hiding this comment.
제일 먼저 과제 제출해주시고, 너무너무 수고하셨습니다! 기능적으로 문제는 없던 코드라서, ux적으로 고려하여 코드리뷰를 남겼던 것 같습니다.
- 추가로 전체보기로 등록된 전체 할 일을 조회하는 경우에는 할일 전체 갯수가 나오지 않는데, 그것도 추가해주면 기능적으로 통일되어 보기 편할 것 같습니다.
다음 과제로 리액트라이브러리를 사용하여 투두만드신 것들을 리팩토링 하게 되는데 컴포넌트의 사용 이점을 느껴보실 수 있을 거에요! (다음과제도 1등하실건가요???~~! 기대해보겠습니다)
There was a problem hiding this comment.
전체보기를 통해서 모든 할 일들을 조회할 수 있는 점은 좋은 듯합니다. 하지만, 전체보기 후에 오늘 날짜를 보려면, 날짜 캘린더 들어가서 클릭 후, 날짜를 클릭해야한다는 점에서 사용자 ux측면에서 불편한듯합니다ㅠ
script.js
Outdated
| return; | ||
| } | ||
|
|
||
| const todo = { id: Date.now(), text, date }; |
There was a problem hiding this comment.
현재 todo id를 Date.now으로 저장하셨는데, 같은 밀리초안에 여러 투두를 등록하게 되는 경우, id가 꼬일 수 있습니다(동시성 문제).
date.now대신 uuid를 사용해보는 것을 어떠할까요?
http://developer.mozilla.org/en-US/docs/Web/API/Crypto/randomUUID
Wannys26
left a comment
Wannys26
left a comment
There was a problem hiding this comment.
제가 좋아하는 깔끔하고 담백한 디자인이여서 좋았습니다!
미숙한 피드백일 수 도 있겠지만 잘 읽어주시면 감사하겠습니다
| width: 30px; | ||
| margin-left: 10px; | ||
| font-size: large; | ||
| } |
There was a problem hiding this comment.
지금 inputDiv가 countDisplay div까지 포함되어 있는 상태라,
할 일 수가 0이거나 전체 보기를 누르면, 중앙 정렬이 안되는 거 같습니다!
이 부분을 수정하려면
#countDisplay.hidden {
display: none;
}
위 코드를 추가하시고,
할 일 갯수가 0이 되면 해당 클래스가 추가되도록, js에서도 수정해보시는 것도 좋을 것 같습니다!
|
안녕하세요..! 과제 잘 봤습니다 ㅎㅎ |
|
우선 기능적으로 추가를 안하신 것 같은데 투두리스트이다 보니 할 일 완료했음을 표시하는 기능이 들어가면 더 좋을 것 같습니다..! |
script.js
Outdated
| resetBtn.addEventListener("click", () => { | ||
| renderTodos(todos); | ||
| countDisplay.textContent = ""; | ||
| }); |
There was a problem hiding this comment.
이게 전체보기 버튼인데, 클릭하면 할일 개수가 사라지던데 그 기능이 맞는걸까요??
countDisplay.textContent = ""; 가 할일 개수를 빈 문자열로 바꾸는 거 같은데 어떤 기능인지 잘 모르겠습니다..
특히 다른 할일을 추가하면filterByDate(date);로 다시 개수는 업데이트 되어서 이 부분이 어떤 기능을 하는지 파악이 어려운 것 같습니다 ㅜㅜ
4 participants
<vercel 배포 링크>
https://sungah-vanilla-todo-22nd-tpi2.vercel.app/
<구현기능>
1. 투두 추가 및 조회
할일을 입력하고 추가 버튼을 누르면 목록에 추가됩니다.
화면을 꺼도 local storage에 저장되어 다시 조회가 가능합니다.
추가 버튼 오른 편에 그날의 투두 개수를 볼 수 있습니다.
상단에 To do list 버튼을 누르면 현재 날짜를 기준으로 조회할 수 있습니다.
2. 투두 삭제
추가된 할일 옆에 삭제 버튼을 누르면 삭제할 수 있습니다.
3. 날짜별 조회
날짜를 누르면 그 날에 작성된 투두를 조회할 수 있습니다.
4. 전체 조회 버튼
날짜 관계없이 모든 투두를 조회할 수 있습니다.
<느낀점 및 배운점>
1. DOM은 무엇인가요?
2. 이벤트 흐름 제어(버블링 & 캡처링)이 무엇인가요?
4. 클로저와 스코프가 무엇인가요?