search

DaleStudy / leaderboard

달레 스터디 리더보드
http://leaderboard.dalestudy.com/
3 stars 0 forks source link

feat : create header component #45

Closed SamTheKorean closed 2 weeks ago

SamTheKorean commented 3 weeks ago

체크리스트

데모 영상

https://github.com/user-attachments/assets/4ed377d2-7d76-47d0-9c82-b29c50e3b0fa

SamTheKorean commented 3 weeks ago

아직 테스트 작성을 못했는데 test 파일에서 컴포넌트가 렌더링 되지 않아 스타일 테스트를 할 수 없어서 트러블 슈팅중에 있습니다. 다만 시간이 생각보다 지체돼서 우선 완성한 부분들까지 올려두었습니다. 해당 문제는 이어서 해결하도록 하겠습니다.

SamTheKorean commented 2 weeks ago

테스팅의 범위를 어디까지 잡고 해야할까 고민하다가 기능적으로 꼭 필요하다고 생각하는 부분만 넣었는데 혹시 피드백 주신다면 보완해보도록 하겠습니다!

DaleSeo commented 2 weeks ago

테스팅의 범위를 어디까지 잡고 해야할까 고민하다가 기능적으로 꼭 필요하다고 생각하는 부분만 넣었는데 혹시 피드백 주신다면 보완해보도록 하겠습니다!

@SamTheKorean 이 정도면 일단 충분한 것 같아요 :)

DaleSeo commented 2 weeks ago

구현하신 Header를 각 페이지 컴포넌트에 연결을 해주셔야 비로서 요건이 만족했다고 볼 수 있을 것 같습니다.

SamTheKorean commented 2 weeks ago

나눠주신 피드백 중 의문점이 드는 부분은 답변을 달았고 대부분은 바로 수정하고 commit하였습니다! 시간내서 피드백주셔서 감사드립니다ㅎㅎ

DaleSeo commented 2 weeks ago

@SamTheKorean 메인 컨텐츠의 윗 부분이 헤더에 뒤로 가서 가려지면 안 될 것 같습니다!

image
SamTheKorean commented 2 weeks ago

@SamTheKorean 메인 컨텐츠의 윗 부분이 헤더에 뒤로 가서 가려지면 안 될 것 같습니다!

image

header 가 fixed position이라 다른 모든 컴포넌트에 상위 마진을 추가해주거나 하는 방식으로 해결해야할 것 같은데 다른 컴포넌트를 위한 pr에서 작업하는 것보다 현재 pr에서 모든 컴포넌트의 스타일을 추가하며 해결하는게 더 적절할까요?

SamTheKorean commented 2 weeks ago

생각해보니 현재 병합이 되면 바로 production(https://leaderboard.dalestudy.com/)으로 가게 되니 병합되기전에 해결하는게 바람직해 보여서 여기서 고치겠습니다.!

SamTheKorean commented 2 weeks ago

focus 사용에 관해 미결정 사항이 있지만 우선 디자이너님 의견에 따라 커밋하고 집어주신 피드백도 수정하고 commit하였습니다!