refactor: BottomCta 공통 컴포넌트 개선 및 페이지별 개별 구현 통합#345
Open
kanghaeun wants to merge 2 commits into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughBottomCta에 공통 배경·패딩과 선택적 그라데이션을 추가하고, 여러 페이지의 하단 CTA를 BottomCta 기반으로 통합했습니다. 기존 버튼의 이벤트와 상태 동작은 유지됩니다. ChangesBottomCta 공통화
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
작업 요약
BottomCta공통 컴포넌트로 통합배경
1. CTA의 화면 하단 밀착
모바일에서 바텀 CTA가 화면 최하단에 너무 붙어 있어 여백이 부족했음
→ 하단 패딩을
20px → 32px로 상향해 여유 확보.2. 페이지별 페이드 그라데이션 구현 방식 불일치
CTA 위 페이드 그라데이션이 페이지마다 인라인으로 개별 구현되어 있었음.
66.878px매직 넘버 사용#F4F4F6색상 하드코딩→
BottomCta공통 컴포넌트로 통합.작업 세부 내용
1.
BottomCta공통 컴포넌트 개선패딩 상향
hasGradientprop 추가#F4F4F6 0% → bg/layer/basement기본 배경 승격
bg-bg-layer-basement를 컴포넌트 기본 배경으로 승격2. 페이지별 개별 구현을
BottomCta로 대체ByWishContent)<BottomCta hasGradient>ResultClient)<BottomCta hasGradient>(z-index 30 동일 유지)3. 기존 사용처 오버라이드 정리
기본값과 중복된
bg-bg-layer-basement,pb-8,py-3오버라이드 제거.대상
스크린샷
연관 이슈
closes #344
Summary by CodeRabbit