Skip to content

[나원지] feat: 초급, 중급 todolist 구현#7

Open
rowanna wants to merge 4 commits into
asac-assignments:mainfrom
rowanna:rowanna
Open

[나원지] feat: 초급, 중급 todolist 구현#7
rowanna wants to merge 4 commits into
asac-assignments:mainfrom
rowanna:rowanna

Conversation

@rowanna
Copy link
Copy Markdown

@rowanna rowanna commented Dec 10, 2024

작업 동기

  • 1차 자바스크립트 과제 초급, 중급 구현

스크린샷

  • 초급(simple.html)
image image image image
  • 중급(detail.html)
image image image image image image image

작업 내용

요구사항

  • simple.html

    • 할일 목록 추가
  • detail.html

    • 할 일 추가
    • 할 일 수정
    • 할 일 완료
    • 할 일 삭제
    • 할 일 목록 필터링

개발 중 발견했던 버그사항 및 개선방법

keyup이벤트로 addTodo함수 실행시 한글 입력시 이벤트가 두번 발생되는 상황

  • keydown이벤트로 변경하였더니 해결되었습니다.
  • (아래 링크에서는 keydown으로 하면 안된다고 하는데 난 왜 해결됐는지 모르겠음.)
  • https://kimyeongseo.tistory.com/60

리뷰 시 봐주었으면 하는 부분

  • 기능을 함수로 쪼개는 단위가 어느정도 범위여야 하는지 잘 모르겠어서 궁금합니다.

체크리스트

  • 무엇을 변경했는지 충분히 설명하고 있나요?
  • 새로운 기술을 사용했다면, 그 기술을 설명하고 있나요?
  • 이해하기 어려운 비즈니스 로직에 관해서 부연 설명을 하고 있나요?
  • 작업 범위에 대해서 테스트를 작성하셨나요?

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.

1 participant