Skip to content

refactor: BottomCta 공통 컴포넌트 개선 및 페이지별 개별 구현 통합 - #345

Merged
kanghaeun merged 3 commits into
devfrom
refactor/344-common-bottom-cta-gradient
Jul 17, 2026
Merged

refactor: BottomCta 공통 컴포넌트 개선 및 페이지별 개별 구현 통합#345
kanghaeun merged 3 commits into
devfrom
refactor/344-common-bottom-cta-gradient

Conversation

@kanghaeun

@kanghaeun kanghaeun commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 모바일에서 바텀 CTA가 화면 최하단에 붙어 보이던 문제 개선 (하단 패딩 20px → 32px)
  • 페이지마다 인라인으로 제각각 구현되던 CTA 상단 페이드 그라데이션을 BottomCta 공통 컴포넌트로 통합

배경

1. CTA의 화면 하단 밀착

모바일에서 바텀 CTA가 화면 최하단에 너무 붙어 있어 여백이 부족했음

→ 하단 패딩을 20px → 32px로 상향해 여유 확보.

2. 페이지별 페이드 그라데이션 구현 방식 불일치

CTA 위 페이드 그라데이션이 페이지마다 인라인으로 개별 구현되어 있었음.

  • 66.878px 매직 넘버 사용
  • #F4F4F6 색상 하드코딩
  • 수동 fixed 컨테이너 중복 구현

BottomCta 공통 컴포넌트로 통합.

작업 세부 내용

1. BottomCta 공통 컴포넌트 그라데이션 prop 추가

패딩 상향

위치 변경
12px (유지)
아래 20px → 32px
양옆 20px (유지)

hasGradient prop 추가

  • CTA 상단에 높이 36px 페이드 그라데이션 표시
  • 색상: #F4F4F6 0% → bg/layer/basement

기본 배경 승격

  • 모든 사용처가 공통으로 쓰던 bg-bg-layer-basement를 컴포넌트 기본 배경으로 승격

2. 페이지별 개별 구현을 BottomCta로 대체

페이지 기존 변경
위시에서 가져오기 (ByWishContent) 인라인 그라데이션 div + 수동 fixed 컨테이너 <BottomCta hasGradient>
토너먼트 결과 (ResultClient) "홈으로 가기" 수동 fixed 컨테이너 <BottomCta hasGradient> (z-index 30 동일 유지)

3. 기존 사용처 오버라이드 정리

기본값과 중복된 bg-bg-layer-basement, pb-8, py-3 오버라이드 제거.

대상

  • 아이템 편집 폼 (아카이브 위시 ×2, 토너먼트 아이템)
  • 마이페이지 수정
  • 회원 탈퇴

스크린샷

image

연관 이슈

closes #344

Summary by CodeRabbit

  • 개선 사항
    • 여러 화면의 하단 고정 버튼 영역을 통일된 레이아웃으로 개선했습니다.
    • 토너먼트 생성 및 결과 화면의 하단 CTA가 일관된 방식으로 표시됩니다.
    • 하단 영역의 배경과 여백을 정리해 화면별 표시 방식이 개선되었습니다.
    • 필요한 경우 CTA 상단에 그라데이션 효과가 적용됩니다.
    • 기존 뒤로 가기, 다음 단계, 홈 이동 및 로딩·비활성화 동작은 동일하게 유지됩니다.

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
depromeet Ready Ready Preview, Comment Jul 17, 2026 1:07pm
piki Ready Ready Preview, Comment Jul 17, 2026 1:07pm

@github-actions
github-actions Bot requested review from soyeong0115 and ychany July 15, 2026 11:29
@github-actions github-actions Bot added the refactor Extra attention is needed label Jul 15, 2026
@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

BottomCta에 공통 배경·패딩과 선택적 그라데이션을 추가하고, 여러 페이지의 하단 CTA를 BottomCta 기반으로 통합했습니다. 기존 버튼의 이벤트와 상태 동작은 유지됩니다.

Changes

BottomCta 공통화

Layer / File(s) Summary
BottomCta 스타일 및 옵션 확장
apps/web/src/components/bottom-cta/index.tsx
hasGradient prop을 추가하고 기본 배경·하단 패딩 및 조건부 상단 그라데이션을 구현했습니다.
위시 기반 토너먼트 CTA 전환
apps/web/src/app/tournament/[id]/create/by-wish/_components/ByWishContent.tsx
기존 고정 CTA 구조를 BottomCta 기반으로 변경했으며 뒤로 가기, 다음 버튼의 비활성화·로딩·클릭 동작은 유지했습니다.
기존 CTA 사용처 통합
apps/web/src/app/archive/wish/[id]/_components/ItemEditForm.tsx, apps/web/src/app/mypage/edit/_components/EditForm.tsx, apps/web/src/app/mypage/withdraw/_components/WithdrawConfirmDialog.tsx, apps/web/src/app/tournament/[id]/item/[itemId]/_components/ItemEditForm.tsx, apps/web/src/app/tournament/[id]/result/_components/ResultClient.tsx
여러 화면에서 BottomCta에 전달하던 개별 스타일을 제거하고 결과 화면의 고정 CTA를 BottomCta 래퍼로 교체했습니다.

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

Suggested reviewers: iodio89

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Issue #344의 BottomCta 그라데이션 추가, 페이지별 인라인 CTA 교체, 기존 사용처 정리 요구와 대체로 일치합니다.
Out of Scope Changes check ✅ Passed 요구사항과 무관한 변경은 보이지 않으며, 수정된 파일들도 모두 BottomCta 공통화 범위 안에 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed BottomCta 공통 컴포넌트 개선과 페이지별 개별 CTA 구현 통합을 잘 요약한 제목입니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/344-common-bottom-cta-gradient

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

@kanghaeun
kanghaeun merged commit be30cbe into dev Jul 17, 2026
5 checks passed
@kanghaeun
kanghaeun deleted the refactor/344-common-bottom-cta-gradient branch July 17, 2026 13:08
iOdiO89 pushed a commit that referenced this pull request Jul 19, 2026
* feat: 바텀 CTA 상단 그라데이션 옵션 추가 및 패딩 스펙 반영

* refactor: 페이지별 바텀 CTA를 BottomCta 공통 컴포넌트로 교체
iOdiO89 added a commit that referenced this pull request Jul 19, 2026
* fix: svgo removeViewBox 비활성화로 아이콘 축소 렌더링 시 잘림 방지

* feat: 바텀탭 디자인 변경 및 탭 4개로 분리

* refactor: 바텀탭 button > link 태그로 교체

* feat: 마이페이지에 바텀탭 추가

* refactor: 바텀탭 z-index를 Z_INDEX 상수로 관리

* refactor: 마이 탭 신설에 따라 헤더 프로필 아이콘 제거

* feat: 바텀탭 포인터 다운 시 스케일업 인터랙션 추가

* feat: 바텀탭 인디케이터 드래그로 탭 전환하는 인터랙션 추가

* feat: 바텀탭 liquid glass 효과 추가

* feat: 바텀탭 포커스되지 않은 탭 클릭해도 드래그 가능하도록 변경

* feat: 바텀탭 렌즈 변신 조건 정리 및 배경 틴트 상태 분리

* refactor: 바텀탭 wrapper 컴포넌트 내부로 이동

* style: 바텀탭 blur 완화

* fix: 탭바 새 제스처 시작 시 예약된 라우팅 취소

* fix: 탭바 착지 애니메이션 완료 후 라우팅되도록 지연 정렬

* fix: 탭바 새 제스처 시작 시 예약된 라우팅 취소

* refactor: 로그인 진입 시 세션 조회 왕복 제거 및 스플래시 전환 개선 (#348)

* refactor: access token JWT에서 role 추출하는 유틸 추가

* refactor: 로그인 세션 검사(getMe) 제거하고 JWT role 기반으로 전환

* refactor: 게스트 세션 재활용 판정을 클릭 시점 refresh로 일원화

* fix: 루트 스플래시 배경 FOUC 제거

* feat: 로그인 진입 시 문구·버튼 fade-in 애니메이션 추가

* refactor: /archive 탭을 위시리스트·내 토너먼트 페이지로 분리 (#347)

* refactor: /archive 탭 쿼리 파라미터를 /archive/wish, /archive/tournament 경로로 분리

* refactor: 구버전 /archive(?tab=) 경로를 신규 경로로 리다이렉트

* refactor: 옛 /wish 페이지 잔재 정리 및 archive 콜로케이션 재배치

* fix: 위시 페이지 체류 중 후속 공유 인텐트가 무시되던 문제 수정

* fix: 보관 탭 활성 판정 경로 경계 추가 및 위시 추가 후 중복 라우팅 제거

* fix: 공유 인텐트 실패 URL 잠금 해제 및 링크 담기 실패 시 다이얼로그 유지

* refactor: JWT role을 명시적 검증으로 좁혀 타입 단언 제거

* fix: share intent 처리 후 URL 잠금 해제하여 재공유 허용

* refactor: font preload 삭제

* refactor: Pretendard 폰트 CDN Dynamic Subset Variable로 교체

* chore: TanstackQuery devtool Dynamic Import로 변경

* chore: Tanstack Query Devtools ssr false 설정

* fix: Android 12+ 스플래시 로고 저해상도 문제 해결 (RN 오버레이로 전환)

* fix: Android 시스템 스플래시 저품질 로고 미노출 처리 (배경색만 표시)

* Revert "fix: Android 시스템 스플래시 저품질 로고 미노출 처리 (배경색만 표시)"

This reverts commit adede7e.

* Revert "fix: Android 12+ 스플래시 로고 저해상도 문제 해결 (RN 오버레이로 전환)"

This reverts commit 74d09aa.

* fix: 안드로이드 카카오 로그인 무한 로딩 수정 (리다이렉트 수신 액티비티 등록) (#350)

* fix: 카카오 로그인 안드로이드 리다이렉트 수신 액티비티 등록

* chore: 서명 자격증명 및 빌드 산출물 gitignore 추가

* fix: 안드로이드 스플래시 로고 저해상도 노출 수정 (RN 오버레이 전환) (#352)

* fix: Android 12+ 스플래시 로고 저해상도 문제 해결 (RN 오버레이로 전환)

* fix: Android 시스템 스플래시 저품질 로고 미노출 처리 (배경색만 표시)

* fix: SplashScreen.hideAsync rejection 처리 추가

* refactor: BottomCta 공통 컴포넌트 개선 및 페이지별 개별 구현 통합 (#345)

* feat: 바텀 CTA 상단 그라데이션 옵션 추가 및 패딩 스펙 반영

* refactor: 페이지별 바텀 CTA를 BottomCta 공통 컴포넌트로 교체

* chore: iOS ShareBottomSheet 에셋 파일명 교체 (#353)

* chore: iOS 앱 공유 바텀 시트 에셋 추가

* feat: iOS 앱 공유 바텀시트 성공/에러 UI 추가

* chore: app sentry cli 추가

* chore: patch app version

* chore: Playwright E2E 테스트 환경 구축 (#319)

* chore: playwright 설치

* chore: Playwright 설정 및 테스트 스크립트 추가

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

* feat: 게스트 storageState 생성 setup 추가

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

* feat: page.route 기반 API 목킹 fixture 및 목 데이터 추가

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

* test: 홈 및 토너먼트 준비 페이지 E2E 테스트 추가

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

* fix: SSR 접근 권한 조회용 목 스텁 서버 추가 및 준비 페이지 테스트를 진입 플로우로 변경

* chore: playwright/.auth gitignore 패턴 수정

* chore: E2E 테스트 CI 잡 추가

* fix: UI 모드와 CLI 동시 실행 시 목 스텁 포트 충돌 해결

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

* fix: dev 전용 react-grab 스크립트 차단으로 trace 스냅샷 빈 화면 해결

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

* chore: CI Node 버전 22 LTS로 변경

* refactor: E2E 테스트를 도메인 폴더 구조로 변경

* refactor: @e2e alias 도입

* feat: E2E 이미지 목킹 추가 (가짜 CDN URL + /_next/image 인터셉트)

* feat: api 목킹 fixture에 patch, delete 메서드 추가

* docs: E2E 테스트 사용 가이드 추가

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Jung Sun A <amber0809@naver.com>

---------

Co-authored-by: joyeongchan <tigerbone@naver.com>
Co-authored-by: kanghaeun <145974230+kanghaeun@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
iOdiO89 pushed a commit that referenced this pull request Jul 19, 2026
* feat: 바텀 CTA 상단 그라데이션 옵션 추가 및 패딩 스펙 반영

* refactor: 페이지별 바텀 CTA를 BottomCta 공통 컴포넌트로 교체
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: 바텀 CTA 그라데이션 공통화

1 participant