Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Say sth about #303

Open
wants to merge 29 commits into
base: saySthAbout
Choose a base branch
from

Conversation

saySthAbout
Copy link

No description provided.

saySthAbout and others added 29 commits November 24, 2021 18:58
한글을 인식하지 못하는 keypress이벤트 대신 한글도 인식할 수 있는 keydown이벤트로 변경
- 상태는 변하는 데이터, 이 앱에서 변하는 것이 무엇인가
- 로컬 스토리지는 해당 url의 로컬스토리지에 localStroage.setItem("key", "value"); 로 저장하고
localStorage.getItem("key"); 로 출력 가능하다.
- 개발자 도구의 Application탭에 로컬스토리지 쪽에 가면 저장된 값 볼 수도 있고, 삭제할 수도 있다.
App() 함수 호출할때 new App();으로 호출해야만 호출된 코드 안에서 this사용이 가능하다.
카테고리별 메뉴 수정, 삭제 그리고 리팩터링
1. 한 파일에는 한 객체만 있는게 좋다 -> 외부 파일 2개 만들고 export 시킨후 import 처리함
2. menu객체에서 상태관리 하니까 메뉴의 count를 그 객체를 통해서 한다.
3. 메뉴 수정이나 삭제 후에도 render()메소드를 불러오도록 한다.
4. addEventListener계층의 로직이 계속 추가되니까 initEventListeners라는 함수에 모아서 관리하고 init시에 초기화? 한다.
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