Skip to content

fix: 튜토리얼 완료 화면 헤더 제거 - #249

Merged
whghdus merged 1 commit into
devfrom
fix/248-tutorial-complete-remove-header
Sep 19, 2026
Merged

whghdus merged 1 commit into
devfrom
fix/248-tutorial-complete-remove-header

Conversation

@whghdus

@whghdus whghdus commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • 튜토리얼 완료 화면(TutorialComplete)의 빈 상단 헤더(StatusBar)를 onboarding/replay 두 케이스 모두 제거
  • AdaptiveScrollArea(콘텐츠가 넘칠 때만 스크롤 허용/짧으면 중앙 정렬)를 걷어내고, 다른 튜토리얼 스텝과 동일하게 항상 스크롤 가능한 section 구조로 통일
  • 더 이상 쓰이지 않는 AdaptiveScrollArea 컴포넌트 삭제
  • 헤더가 사라진 만큼 상단 패딩(pt-10)을 추가해 콘텐츠가 화면 끝에 붙지 않도록 함 (rem 기반이라 기존 반응형 폰트 스케일과 함께 자연스럽게 스케일됨)

Closes #248

Test plan

  • pnpm lint / pnpm typecheck / pnpm build 통과 (pre-push 훅)
  • /tutorial 전체 스텝을 진행해 onboarding 완료 화면 확인 (헤더 없음, 레이아웃 정상)
  • /tutorial/replay/steps 진행해 replay 완료 화면 확인 (헤더 없음, 레이아웃 정상)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 변경 사항
    • 튜토리얼 완료 화면의 콘텐츠 영역이 직접 스크롤 가능한 형태로 변경되었습니다.
    • 콘텐츠 높이에 따라 스크롤을 자동으로 적용하던 기존 동작이 제거되었습니다.
    • 완료 화면의 상태 표시줄이 제거되었습니다.
    • 보상 표시와 튜토리얼 완료 버튼의 기능은 기존과 동일하게 유지됩니다.

- TutorialComplete에서 빈 상단 헤더(StatusBar)를 완전히 제거 (replay/non-replay 공통)
- AdaptiveScrollArea의 조건부 스크롤 토글 대신 항상 스크롤 가능한 구조로 변경, 다른 튜토리얼 스텝과 동일한 패턴 적용
- 더 이상 사용되지 않는 AdaptiveScrollArea 컴포넌트 삭제

Closes #248

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
brifo-front Ready Ready Preview Sep 19, 2026 1:42pm UTC

@coderabbitai

coderabbitai Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: Team-BRIFO/BRIFO-front/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: ee1ae227-e222-4d08-b671-ddf57a437f24

📥 Commits

Reviewing files that changed from the base of the PR and between 775b930 and 1ca2083.

📒 Files selected for processing (2)
  • src/components/feature/tutorial/AdaptiveScrollArea.tsx
  • src/components/feature/tutorial/TutorialComplete.tsx
💤 Files with no reviewable changes (1)
  • src/components/feature/tutorial/AdaptiveScrollArea.tsx

Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

AdaptiveScrollArea를 삭제했습니다. TutorialComplete에서 StatusBar를 제거하고, 패딩과 세로 스크롤을 직접 지정한 section으로 콘텐츠를 감쌌습니다.

Changes

튜토리얼 완료 화면 레이아웃

Layer / File(s) Summary
직접 스크롤 영역으로 교체
src/components/feature/tutorial/AdaptiveScrollArea.tsx, src/components/feature/tutorial/TutorialComplete.tsx
AdaptiveScrollArea의 조건부 스크롤과 크기 감시 로직을 삭제했습니다. TutorialComplete에서 StatusBar와 관련 import를 제거하고, 패딩·세로 스크롤·콘텐츠 정렬을 지정한 section을 사용합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix · Severity of issue fixed: Low

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 튜토리얼 완료 화면에서 헤더를 제거하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Linked Issues check ✅ Passed 이슈 #248의 세 가지 코딩 요구사항을 모두 충족합니다. TutorialComplete.tsx에서 onboarding과 replay 공통의 StatusBar를 제거했습니다. 콘텐츠를 overflow-y-auto와 overscroll-y-contain을 가진 section으로 변경해 항상 스크롤 가능한 구조로 통일했습니다. 상단 여백은 `…
Out of Scope Changes check ✅ Passed 변경 범위는 이슈 #248과 직접 연결됩니다. TutorialComplete의 헤더 제거와 레이아웃 변경은 요구사항을 구현합니다. 미사용 AdaptiveScrollArea 삭제도 조건부 스크롤 구조를 제거하는 작업과 직접 연결됩니다. 관련 없는 변경은 확인되지 않습니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@whghdus
whghdus merged commit defe982 into dev Sep 19, 2026
4 checks passed
@whghdus
whghdus deleted the fix/248-tutorial-complete-remove-header branch September 19, 2026 13:44

This branch was successfully deployed

1 active deployment
Preview — 1ca2083c Deployed Sep 19, 2026 by vercel[bot]
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.

튜토리얼 완료 화면 헤더 제거 및 레이아웃 수정

1 participant