0807-1day1Pr
intro
키보드 충전한 다는 걸 잊어서 … ㅜ
오늘 또 늦잠 12:00
플래닝
키워드 1day1Pr & 운동은 지키고 있다. 이것만은 한다 라는 마인드
- 오늘 Step5 리뷰해주시는 데로 리뷰 반영 및 -> 리팩토링
- Step6
nextStep
오늘 배운 것은???
1 | 삽질 끝에 배운 점 |
- 크롱 피드백을 통해 배운 점 인자들은 되도록 변수화 시켜서 매개변수로 받고 주기 !!!
- 되도록이면 스타일 코드들 분리할 수 있는 것은 css로 -> 분리 하기 + 저번에 비슷한 피드백이 나왔는데 또 같은 지적을 받았는데
심지어 이번 커밋도 …js안에서 했네 얼른 고치자 …
아쉬움 잘한점
잘한점
어떻게든 pr를 날리고는 있다.
그래도 3끝까지 그 문제를 해결하려고 했다.
아쉬운 점
삽질 계속 같은 걸 시도 한 시간이 많았다. 시행착오 -> 이렇게는 안 되는 구나를 보고 다른 시도를 해야 하는데
같은 시도를 왜 안되지 하면서 너무 많이 했다.하드코딩으로 해결했다.
이거 뭡니까 !!!
const padArr = [...new Array(this.padElCounts)]
this.slideEl.insertAdjacentHTML('beforeend', padTemplate(padArr));
this.slideEl.insertAdjacentHTML('beforeend', cardTemplate(slideData.slice(0,showListItems)));
this.slideEl.insertAdjacentHTML('afterbegin', padTemplate(padArr));
this.slideEl.insertAdjacentHTML('afterbegin', cardTemplate(slideData.slice(-showListItems+this.padElCounts)));
Question
주입한 Pagination … setting들을 넘겨서 SliderConstructor에서 해주고
next나 right Button Click되도 그 idx update method를 실행시켜주는데 너무 다 직접 부르고 있다 _= 완전 자식 같은 컴포넌트라 이래도 자연스러운 것 같기도 하고 ;;;
ETC
- 배민 사이트를 돌리면서 힌트를 얻었다.
- over-flow hidden풀고 보면 더 동작이 명확해지는 구나
- Data의 유동성 고려를 못했는데 api가 딱 맞아 떨어지더라도 내가 […] 스프레드를 통해서 갯수를 변화시켜서 테스트 해볼 수 있다. 루미랑 얘기를 하면서 풀어보려 하는게 도움이 많이 되었다. 사람들이랑 얘기를 많이 하자 !!!
- 다른 사람들 사이트에 비해서 스타일링 너무 안하고 넘어간 부분이 많다 . 하루에 하나씩 발견한 것 고쳐나가자 !!!