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

[0423] Refactor #19

Closed
12 of 17 tasks
GleamingStar opened this issue Apr 23, 2021 · 0 comments · Fixed by #23
Closed
12 of 17 tasks

[0423] Refactor #19

GleamingStar opened this issue Apr 23, 2021 · 0 comments · Fixed by #23
Assignees
Labels
Seong assigned in seong
Projects

Comments

@GleamingStar
Copy link
Owner

GleamingStar commented Apr 23, 2021

수정할 것

  • Section 만든 후 가로 크기 조정
    • DD와 함께 가로정렬 해보자
  • Modal
    • Modal Button에 기능 추가
      • 그냥 버튼 클릭시 재고없음 버튼으로 토글되게?
    • Card나 BestList등 fetch API쓰는 곳에 로딩 중 혹은 에러시 얼리리턴으로 에러처리조항 추가
      • .then().catch( { isError : true } ) 이런식으로 가짜객체 던져주고
      • if(isError) return (<div>적절한 에러 알림<div/>) 이렇게 에러처리?
    • Modal top좌표 현재 스크롤 인식해서 동적 렌더링
      • DD가 화면 센터로 옮기면 width height 100%의 검은 가림막? 두고 그위에 flex로 가운데정렬
    • Modal 하단 @JJJJ-Carousel 적용해서 추천상품리스트 추가
        1. 현재 이미지 링크들만 있는 객체 이용해 이미지만
        1. json의 API 이용
      • (API기능 추가 된다면) Card SMALL type도 맞춰 추가 할 것
  • fetch API async로 바꾸기! (굳이 해야하나?) [0421] Refactor #11

codesquad-members-2021#19

  • 헤더 Menu 컴포넌트로 분리
  • svg 따로 파일 이름으로 참고
  • Modal 32,33line replace와 regex로 한번에
    • 실패!!
  • Modal 컴포넌트 분리
  • 삼항연산자 : "" 쓸거면 ||연산자로
  • 3자리수마다 콤마넣기 유틸함수로 빼기
    • 지금 굳이? 나중에 util.js 생기면 그때 옮기자
  • ThumbNail.jsx 38-45 line useEffect 확인할것
    • fetch때문아니었나
    • 아니네... 그래도 top image 초기화가 안되므로 effect 하나는 놔두자
  • 이벤트이름 clickTab, clickCard따위로 그냥 handler ㄴㄴ
  • Card.jsx 115-117
    • 나중에 inline function과 useCallback 함수와 성능차이도 공부해보시면 좋을 것 같습니다 ㅎㅎ ??
  • CardPrice 컴포넌트 분리
  • ThumbNail.jsx foo 네이밍 + [...thumb_images, "", "", ... ""].slice(0,5) 5개 다넣고 자르기

  • JS폴더 / JSX폴더?
@GleamingStar GleamingStar added the Seong assigned in seong label Apr 23, 2021
@GleamingStar GleamingStar added this to todo in sidedish via automation Apr 23, 2021
@GleamingStar GleamingStar self-assigned this Apr 23, 2021
@GleamingStar GleamingStar changed the title [0423] 기타 추가 사항 [0423] Refactor Apr 26, 2021
@GleamingStar GleamingStar moved this from todo to doing in sidedish Apr 26, 2021
GleamingStar pushed a commit that referenced this issue Apr 26, 2021
[ DDeong ] 1차 완성 / Carousel npm 배포
@GleamingStar GleamingStar linked a pull request Apr 26, 2021 that will close this issue
sidedish automation moved this from doing to done Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Seong assigned in seong
Projects
Development

Successfully merging a pull request may close this issue.

1 participant