Skip to content

로그인 후 페이지 높이 상속 복원 - #586

Merged
goder-0 merged 1 commit into
mainfrom
fix/#585-auth-page-height-inheritance
Aug 26, 2026
Merged

로그인 후 페이지 높이 상속 복원#586
goder-0 merged 1 commit into
mainfrom
fix/#585-auth-page-height-inheritance

Conversation

@goder-0

@goder-0 goder-0 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

  • 로그인 후 SideNavigation 셸을 사용하는 /home, /chat/[id], /all-link, /mypage의 페이지 <main>h-full을 적용했습니다.
  • 레이아웃의 h-dvh 높이가 각 페이지 컴포넌트까지 이어지도록 높이 상속을 복원했습니다.
  • /home의 GreetingBlock과 QueryBox가 다시 뷰포트 중앙에 배치되도록 수정했습니다.
  • 채팅 답변이 표시된 뒤에도 입력창이 뷰포트 하단에 유지되도록 수정했습니다.
  • 전체 링크의 내부 스크롤과 마이페이지의 수직 정렬이 셸 높이를 기준으로 동작하도록 수정했습니다.

검증

  • TypeScript tsc --noEmit --incremental false 통과
  • 변경 파일 ESLint 통과
  • 변경 파일 Prettier 검사 통과
  • git diff --check 통과
  • 로컬 백엔드 연결 환경에서 로그인 후 페이지 흐름 확인

@goder-0
goder-0 force-pushed the fix/#585-auth-page-height-inheritance branch from 5ce0505 to cbc3359 Compare August 21, 2026 10:04
@goder-0 goder-0 changed the title 인증 페이지 높이 상속 복원 /home 콘텐츠 높이 상속 복원 Aug 21, 2026
@goder-0
goder-0 marked this pull request as ready for review August 21, 2026 10:06
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 50 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8a6c2d0a-0beb-474b-8507-5e08616b8e0a

📥 Commits

Reviewing files that changed from the base of the PR and between cbc3359 and 69ea1c4.

📒 Files selected for processing (3)
  • src/app/(route)/all-link/page.tsx
  • src/app/(route)/chat/[id]/page.tsx
  • src/app/(route)/mypage/page.tsx

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 69583c5f-eaa4-4a64-8fe7-1c2402ea149d

📥 Commits

Reviewing files that changed from the base of the PR and between dcd66eb and cbc3359.

📒 Files selected for processing (1)
  • src/app/(route)/home/page.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

홈 페이지의 main 요소에 h-full 클래스가 추가되었습니다. 상위 인증 레이아웃의 높이가 홈 페이지 콘텐츠 영역으로 상속됩니다.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed <main>h-full을 추가해 직접 연결된 #585의 높이 상속 문제를 해결합니다.
Out of Scope Changes check ✅ Passed 변경은 /home 높이 상속 복원에 한정되며 관련 없는 코드 변경이 없습니다.
Title check ✅ Passed 제목은 로그인 후 페이지의 높이 상속을 복원하는 주요 변경 사항을 간결하게 설명합니다.
Description check ✅ Passed PR 설명은 템플릿의 관련 이슈와 PR 설명 섹션을 포함합니다. 변경 내용과 검증 결과도 구체적으로 작성되어 있습니다.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#585-auth-page-height-inheritance

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@goder-0 goder-0 changed the title /home 콘텐츠 높이 상속 복원 로그인 후 페이지 높이 상속 복원 Aug 25, 2026
@goder-0
goder-0 force-pushed the fix/#585-auth-page-height-inheritance branch from cbc3359 to 69ea1c4 Compare August 25, 2026 18:58
@goder-0

goder-0 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

@Seong-Myeong 홈화면 뿐만아니라, 다른 각 페이지들에서도 동일 문제를 발견하여 이슈와 PR 을 수정했습니다.

@goder-0
goder-0 requested a review from Seong-Myeong August 25, 2026 19:02
@goder-0
goder-0 merged commit c6540cb into main Aug 26, 2026
3 checks passed
@goder-0
goder-0 deleted the fix/#585-auth-page-height-inheritance branch August 26, 2026 13:20
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.

로그인 후 페이지 높이 상속 누락으로 레이아웃이 깨지는 문제

2 participants