HAT-Here-And-There / here-and-there-frontend

This is here and there project frontend repository of 한이음 ict
0 stars 0 forks source link

/travel-plan-all 페이지 스타일링 리팩터링 관련 이슈 #41

Open Programming-Seungwan opened 2 months ago

Programming-Seungwan commented 2 months ago

현재 travel-plan-all 페이지 관련 스타일링이 문제가 좀 있는 것 같아 의견 공유를 하고자 합니다.

  1. 현재 BookmarkedPlaceList 컴포넌트에서 배경색이 피그마와 좀 차이가 있는 것 같습니다. 그리고, 너비를 1/3, 2/3로 해놓으셨는데 이 방식보다는 BookmarkedPlaceList 컴포넌트는 1/3으로 잡되, 나머지 오른쪽 부분은 flex의 grow 속성을 활용하는 것이 더 좋지 않나 싶습니다.
  2. BookmarkedPlaceList 컴포넌트에서 각 place 카드를 현재에는 그냥 컴포넌트 내부에 기술하고 있는데, 이를 분리하여 BookmarkedPlaceListItem이라는 명칭으로 분할하는 것이 코드 가독성도 좋고 추후에 유지보수 하기도 더 좋지 않을까 싶습니다.
  3. 2에서 말한 map 되는 컴포넌트가 중앙 정렬이 지금 padding 값을 이용해서 진행되고 있는 것 같아요. 지난번 코드리뷰에서도 말씀드렸다시피, padding이나 margin 같은 것으로 중앙 정렬을 하는 방식은 디바이스가 조금이라도 달라지면 어그러질 수 있기 때문에 fle를 쓰는게 더 좋다고 생각합니다.

이 부분 추후에 더 진행해보죠! 내일이나 모레즈음에 137 티켓 작업도 슬슬 진행하겠습니다. 우선 찬영님께서 제가 각 차수마다 어떻게 저장된 장소 상태를 가질지, redux를 어느 상황에 써야하는지 이해해주시면 좋을 것 같아요

shroqkf commented 2 months ago

알겠습니다! jira에 이 부분 적어두고 추후에 진행해보는 걸로 할게요 👍