최종프로젝트인만큼 깃허브에서 merge시 프론트엔드 팀원들과 각자 작성한 코드를 동료에게 이해시키는 코드리뷰시간을 갖는데 리뷰만 2시간 넘게 한것같다.. 내가 짠 코드가 아닌만큼 설명을들을떄 확실히 이해하고 동작원리를 하나하나 묻다보니 오래걸리는것같다. 그리고 내눈에는 고칠곳이 없다고판단한곳에서 리뷰를하는순간 하나하나 오류를 발견하게된다. 리뷰를 하던중 모달을 띄울떄 뒷배경이 변하는 효과를 추가해보면 좋을것같다는 판단하에 이부분을 작성하려는데 생각만큼 만만하지않았다.. css지식중 position속성에대해서 다시한번 알수있는 계기가되었다. absolute 와 fixed. 이 두가지를 가지고 구현을했는데 해결하는데 가장 걸림돌이되었던부분은 모달을 띄울시 투명도를적용시켰을떄 모달 내용또한 투명도가 적용되는것이였다. 이문제를 해결하려찾다가 rgba로 배경색을 주는방식을 알게되었고, opacity와의 차이점은 상속이 안된다는점이였다.. 물론 결론적으로는 형제관계로 모달을 구성하여 이부분을 사용하지는 않았지만, 새로운 개념을알게되었다.
그리고 각 페이지마다 마진값을 적용했을떄 재대로 적용되지않았었는데 보니까 app컴포넌트에서 margin : 0 auto값을 적용시켰어서 각페이지에서 margin 값을 부여하였을시, 마진상쇄가 적용되어 페이지에서 적용된 마진이 상쇄되었던것이였다.. 그래서 margin값으로 중앙정렬를하는대신 flex 박스를 통해 중앙정렬을 시킨후 각 페이지에서 margin값을적용시키니 제대로 적용되는것을 볼수있었다.