ICE0208 / marvelous-react

MIT License
0 stars 0 forks source link

Refactoring #3

Open ICE0208 opened 1 year ago

ICE0208 commented 1 year ago

컴포넌트 props에 $지정

styled-components: it looks like an unknown prop "backface" is being sent through to the DOM, which will likely trigger a React console error. 위와 같은 경고 문구가 떠서 props앞에 $를 붙였는데 다른 해결 방안이 있는지 궁금합니다. 또한, 정확한 원인이 무엇인지 잘 모르겠습니다.


import 경로 수정

react-router-dom 경로를 수정해주었습니다. 자동 완성으로만 import해서 경로가 이상하게 지정되어있었습니다.


폴더명 소문자로 일관성있게 처리

폴더명을 전부 소문자로 수정해주었습니다.


HeroList 컴포넌트로 관심사 분리

HeroList 컴포넌트를 만들어서 서로 관련있는 코드끼리만 모여있게 처리해주었는데, 더 좋은 방법이 있는지 궁금합니다.


useSticky 커스텀 훅으로 분리

스크롤과 관련된 로직을 useSticky 커스텀 훅을 만들어서 분리시켜주었습니다.

netlify[bot] commented 1 year ago

Deploy Preview for marvelous-react-ice ready!

Name Link
Latest commit 73438ba0a66ea88eb61e0bf27334aa4545b67fd9
Latest deploy log https://app.netlify.com/sites/marvelous-react-ice/deploys/6505626d6363b30008de710e
Deploy Preview https://deploy-preview-3--marvelous-react-ice.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.