NHNNEXT / 2014-01-HUDI-WILDGOOSE

김민주/서동규/양현석/김희재 휴디팀
0 stars 5 forks source link

[클라이언트] toggleFav을 개선할 필요는 없을까? #422

Open mixed opened 10 years ago

mixed commented 10 years ago

https://github.com/NHNNEXT/2014-01-HUDI-WILDGOOSE/blob/codereview_12/wildgoose/webapp/scripts/fav.js#L10

https://github.com/NHNNEXT/2014-01-HUDI-WILDGOOSE/blob/codereview_12/wildgoose/webapp/scripts/fav.js#L27 은 data속성을 한번 검색해보고 어떻게 쓰고 왜 사용하는지 좀 확인해볼래?

36 ~ 42은 좀 깔끔하게 개선할 수 없을까?

FourwingsY commented 10 years ago

dataset을 사용해보았습니다. 뒷 부분은 card 전체를 blur처리하는것으로 변경하였습니다만, favorites 페이지가 아직 처리되지 않아 추후에 사용하게 될 것 같습니다.

mixed commented 10 years ago

사실 내가 기대했던 대답은 ajax콜백이 굉장히 중복되어 있는데 개선할 수 없얼까?

happyhj commented 10 years ago

http://callbackhell.com/ 읽어보고있는데 promise로 개선하고 싶어지네용 교수님 promise 관련 괜찮은 링크 없을까용?

mixed commented 10 years ago

http://www.html5rocks.com/ko/tutorials/es6/promises/ 이거 한번 볼래?