2022-12-12

Fact (사실)

• 스프린트

Feeling (느낌)

스프린트

오늘은 저번주에 마저하던 버그나 수정사항들을 수정했습니다.
딱히 어려운 작업들은 없었는데 PC버전의 디자인만 생각하고 모바일을 염두하지않아서 깨진 페이지를 수정했는데
PC에선 하나를 선택할때마다 선택사항들이 더 나오는 그런 페이지인데 모바일에선 그 선택페이지들이 한번에 보이지않으니
스크롤로 이동시켜주면 좋겠다라는 생각이 들어 scrollTo와 useEffect를 사용하여 무언가 선택이 되면 자동으로 넘어가게끔 해줬네요
이 작업할때 모바일마다 높이가 다르니 window.innerHeight을 사용해줬는데 제가 놓친 예외사항이 없을까 걱정되긴하네요..!
추후 문제가 생긴다면 다시 고민해봐야할거같습니다

Finding (교훈)

요즘 PR을 올릴때마다 궁금해할만한 부분을 공식문서를 참고하여 자체리뷰를 올려주는데
이렇게 자체리뷰를 하면서 올리니까 제가 놓친점이나 제대로 이해하지않고 사용하는것을 방지할수있어서 괜찮은거같네요
PR적는데 시간은 좀 걸리긴하지만요..!