Conversation
| npm install | ||
|
|
||
| # 개발 서버 실행 | ||
| npm run dev |
There was a problem hiding this comment.
깔끔하게 잘 작성해주신 readme PR 클릭시 바로 보이면 더 좋을 것 같습니다!
| inputRef.current.focus(); | ||
| inputRef.current.select(); | ||
| } | ||
| }, [isEditing]); |
There was a problem hiding this comment.
focus와 select로 수정시 기존 내용을 전부 블럭처리 해주네요! 좋은 기능 같습니다.
| </div> | ||
| </li> | ||
| ); | ||
| } |
There was a problem hiding this comment.
각 버튼에서 px-3, py-1등의 css속성들이 중복되는데 이런 공통된 부분은 묶을 수 있지 않을까 싶습니다. 또 return 부분이 상당히 긴데 작게 분리할 수는 없을까 하는 생각도 듭니다!
| } catch (e) { | ||
| console.log(e); | ||
| } | ||
| }; |
There was a problem hiding this comment.
수정시 바로 storage에 반영되지 않는 것 같습니다. 이 상태에서 다른 동작(todo추가나 toggle등)을 하면 그때서야 변경감지되어 storage에 반영이 되는데 만약 수정한채로 추가 동작 없이 프로그램이 종료된다면 좋지 못한 사용자 경험이 될 것 같습니다. 😢
또 log()보다는 console.error()를 사용하는 것이 더 좋을 것 같습니다!
| </span> | ||
| )} | ||
| </span> | ||
| )} |
There was a problem hiding this comment.
등록시간은 todo를 눌러야 뜨는데, 코드를 보기 전까지는 어떠한 표시가 없어서 해당 기능이 있는지 모를 것 같습니다. 그리고 특정 상황에서 한번 시간이 깨졌는데 이유는 잘 모르겠습니다.
dbjoung
left a comment
dbjoung
left a comment
There was a problem hiding this comment.
totalSet처럼, 특정 작업이 반복되는 코드를 묶어서 코드의 반복을 줄인 부분이 인상 깊었습니다. 많이 배웠습니다. 과제 수행 고생하셨습니다.
3 participants
No description provided.