forked from code-squad/javascript-food
-
Notifications
You must be signed in to change notification settings - Fork 0
4차 리뷰 정리
Chany edited this page Aug 2, 2018
·
3 revisions
01 slider하면서 어느 부분은 일관성있게 __ - 이부분을 안 하고 넘어갔던 것 같은데 다시 한 번 체크해보고 일관성을 지키도록 하자 ! 그리고 별개로 생각 난 부분 html 추가 되면서 validation 다시 돌려보기
02 ajaxHelper
ajax말고 다른 것으로 통신할 수도 있는데 어떤 것이 들어와도 들어갈 수 있도록 하는 방식으로 !
내가 구현 한 방법은 통신방식은 고정이고 element들을 넣는 방식
음 ajax통신하는 애가 elment를 가지고 있는 것이 어색하게 느껴지긴 한다.
const tabEl = new Tab("ele", tabAjaxHelper) ;
class Tab {
constructor(el, dataHelper) {
init()
}
init() {
dataHelper.getData();
}
}
되도록이면 스타일을 바꿀 때 js로 직접 접근하는 부분은 쓰지 말도록
export default pagination default는 되도록이면 쓰지 말자?