netlify[bot]
commented
1 year ago Deploy Preview for seongminn ready!
| Name | Link |
|---|---|
| Latest commit | a9c39b9296ddead32ee0927f555ef0bb041b816e |
| Latest deploy log | https://app.netlify.com/sites/seongminn/deploys/64311aa8fc1ce10008c9f0bb |
| Deploy Preview | https://deploy-preview-3--seongminn.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site settings.
작업 내용
pages들과layout간의 의존성을 느슨하게 하기 위한 시도를 했습니다.기존에는
Layout내부에서 Header, Body, Footer를 관리했고, 각 페이지에서 넘겨주는props에 따라layout컴포넌트가 알아서 아이템을 렌더링 했습니다. 그러다 보니layout컴포넌트의 역할이 무거워지게 되고, 각 페이지에서 하고자 하는 대부분의 작업이layout을 거쳐야 한다는 문제가 발생하였습니다.이러한 이유로
layout에서 header 역할을 하던 background image를 제거했고, 이를 각 페이지에서layout컴포넌트에 넘겨주는 방식으로 리팩토링 했습니다.