Skip to content

Conversation

@cskime
Copy link
Collaborator

@cskime cskime commented Aug 24, 2025

📝 작업 내용

  • 모든 페이지에서 header의 logo를 클릭하면 메인 페이지(/)로 이동하도록 구현합니다.

💬 리뷰어에게 남길 말 (선택)

  • @nidor022
    • RollingPaperListPage에서 header를 제외한 영역을 viewport에 꽉 차게 늘리도록 수정하신 것을 확인했습니다.
      • 컨텐츠가 화면을 넘어가지 않았는데도 오른쪽에 스크롤이 생기는 문제가 있어서 관련 내용을 수정했습니다.
      • RollingPaperListPageTopContainermin-height 속성을 제거하고, OnboardingLayout에서 <main> 영역에 높이를 지정하도록 수정했습니다.
      • 이렇게 수정하면, RollingPaperListPage 외에 MainPage에서도 같은 효과를 볼 수 있습니다.
    • rolling-paper-list-page.jsx 파일이 export하는 component 이름을 파일 이름과 동일하게 수정했습니다.
      • ShowMessageList -> RollingPaperListPage

@cskime cskime changed the title [#92 [#92] Header logo를 클릭했을 때 메인 페이지로 이동 Aug 24, 2025
@cskime cskime linked an issue Aug 24, 2025 that may be closed by this pull request
1 task
@cskime
Copy link
Collaborator Author

cskime commented Aug 25, 2025

남은 작업을 빠르게 완료하기 위해 리뷰 없이 병합합니다.

@cskime cskime merged commit eaf2b06 into codeit-FE-18-part2:develop Aug 25, 2025
@cskime cskime deleted the feature/#92 branch August 25, 2025 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Header logo 클릭할 때 메인 페이지로 이동

2 participants