Skip to content

[FIX] 회원 탈퇴 확인 팝업 버튼 문구 및 스타일 수정 - #301

Open
ehye1 wants to merge 2 commits into
developfrom
fix/web/300-withdraw-modal-buttons
Open

ehye1 wants to merge 2 commits into
developfrom
fix/web/300-withdraw-modal-buttons

Conversation

@ehye1

@ehye1 ehye1 commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

ISSUE 🔗

close #300



What is this PR? 🔍

회원 탈퇴 확인 팝업의 버튼 문구, 순서, 스타일을 디자인에 맞게 수정했습니다.

배경

  • 기존 구조: 탈퇴 확인 팝업의 왼쪽 버튼은 돌아가기(흰색 border), 오른쪽 버튼은 탈퇴하기(파란색 fill)로 되어 있었습니다.
  • 발생 문제: 디자인과 달리 되돌릴 수 없는 탈퇴 버튼이 파란색으로 강조되어 있었고, 버튼 순서와 취소 버튼 문구도 디자인과 달랐습니다.
  • 해결 방향: 왼쪽에 탈퇴하기(흰색 border), 오른쪽에 취소하기(파란색 fill)가 오도록 버튼 순서와 variant, 문구를 수정했습니다.

탈퇴 확인 팝업

  • 변경 요약: 탈퇴하기 버튼을 왼쪽 border, 취소하기 버튼을 오른쪽 fill로 변경했습니다.
  • 이유: 디자인에서는 팝업을 닫는 취소하기 버튼을 강조하고, 탈퇴하기 버튼은 보조 버튼으로 표시합니다.
  • 구현 방식: 두 ModalButton의 위치와 variant를 바꿨고, 클릭 동작(취소하기 → 팝업 닫기, 탈퇴하기 → 탈퇴 요청 후 닫기)은 그대로 유지했습니다.

문구

  • 변경 요약: withdrawConfirmCancel 문구를 ko 돌아가기 → 취소하기, en Go back → Cancel로 수정했습니다.



To Reviewers

디자인만 바꾼 PR이라 크게 리뷰할 부분은 없습니다.
디자인 시스템의 ModalButton 컴포넌트는 수정하지 않았고, 앱에서 버튼 순서와 variant만 바꿨습니다.

Screenshot 📷

image



Test Checklist ✔

  • pnpm check-types 통과
  • pnpm lint 통과
  • 탈퇴 확인 팝업 버튼 문구·색상 화면 확인

취소 버튼을 fill, 탈퇴하기 버튼을 border로 변경하고 취소 문구를 수정했습니다.
@ehye1 ehye1 self-assigned this Sep 29, 2026
@vercel

vercel Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
timo-client-timo-web Ready Ready Preview Oct 1, 2026 2:51am UTC

@github-actions github-actions Bot added the ⏰ Timo-web Timo 웹 서비스 label Sep 29, 2026
@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: Team-Timo/Timo-client/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 25101cbf-1a4e-40bd-b73c-cc6df96c7117

📥 Commits

Reviewing files that changed from the base of the PR and between 2f6a7c2 and 0ba4fb9.

📒 Files selected for processing (2)
  • apps/timo-web/app/[locale]/(main)/settings/_containers/withdrawal/SettingsWithdrawModalContainer.tsx
  • apps/timo-web/messages/ko.json

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


Walkthrough

회원 탈퇴 확인 팝업에서 탈퇴 버튼을 취소 버튼보다 먼저 표시합니다. 탈퇴 버튼에는 border 스타일을, 취소 버튼에는 fill 스타일을 적용합니다. 한국어와 영어 취소 문구도 변경합니다.

Changes

회원 탈퇴 확인 팝업

Layer / File(s) Summary
버튼 순서, 스타일 및 취소 문구
apps/timo-web/app/[locale]/(main)/settings/_containers/withdrawal/SettingsWithdrawModalContainer.tsx, apps/timo-web/messages/en.json, apps/timo-web/messages/ko.json
탈퇴 버튼을 먼저 표시하고 border 스타일을 적용합니다. 취소 버튼은 뒤에 표시하고 fill 스타일을 적용합니다. 영어 문구를 “Go back”에서 “Cancel”로, 한국어 문구를 “돌아가기”에서 “취소하기”로 변경합니다.

Priority: ⬇️ Low

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

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: jjangminii

Merge Risk: ⚪ Minimal · up to 0ba4f

구현은 이슈 #300의 버튼 배치와 문구 요구를 충족하며, 확인된 코드 수준의 병합 위험은 없습니다. 순서가 유지됐다는 PR 설명은 실제 변경과 일치하지 않으므로 정정이 필요합니다.

🚥 Pre-merge checks | ✅ 4 | ❓ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Linked Issues check ❓ Inconclusive 변경 내용은 Issue #300의 버튼 순서, variant, 문구 요구사항과 대체로 일치합니다. 그러나 PR 목표에는 한국어 문구가 취소인지 취소하기인지, 화면 이미지의 버튼 위치가 무엇인지 서로 다른 정보가 있습니다. Issue #300의 최신 요구사항과 실제 화면을 기준으로 버튼 위치와 한국어 withdrawConfirmCancel 값이 취소하기인지 확인하세요. 확인 결과를 PR 설명과 이미지에 동일하게 반영하세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Out of Scope Changes check ✅ Passed 변경 파일은 탈퇴 확인 모달과 관련된 컴포넌트 및 한국어·영어 번역 파일로 제한됩니다. PR 목표를 벗어난 변경은 확인되지 않습니다.
Title check ✅ Passed PR 제목은 회원 탈퇴 확인 팝업의 버튼 문구와 스타일 변경을 명확하게 설명합니다. 변경된 버튼 순서도 제목의 범위와 일치합니다.
Description check ✅ Passed PR 설명은 버튼 문구, 순서, variant 변경과 각 버튼의 동작을 변경 사항과 연결하여 설명합니다.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@github-actions github-actions Bot added 🐛 Bug 기능이 정상적으로 작동하지 않는 문제 수정 ♥️ 혜원 혜원양 labels Sep 29, 2026
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Timo Performance Report

Bundle Size — timo-web
라우트 크기 First Load JS
/[locale]/home 275.81 kB 🔴 481.62 kB
/[locale]/today 260.02 kB 🔴 465.83 kB
/[locale]/focus 222.32 kB 🔴 428.13 kB
/[locale]/settings 228.90 kB 🔴 434.71 kB
/[locale]/statistics 210.79 kB 🔴 416.60 kB
/[locale]/[...rest] 0 B 🟡 205.81 kB
/[locale]/login 282.28 kB 🔴 488.09 kB
/[locale]/oauth/calendar/callback 121.37 kB 🟡 327.18 kB
/[locale]/oauth/callback 121.03 kB 🟡 326.85 kB
/[locale]/onboarding 294.75 kB 🔴 500.57 kB
/[locale] 120.35 kB 🟡 326.16 kB
/[locale]/policy 121.32 kB 🟡 327.13 kB
/robots.txt/route 0 B 🟡 205.81 kB
/sitemap.xml/route 0 B 🟡 205.81 kB

공유 번들: 205.81 kB
🟢 < 200kB  |  🟡 < 350kB  |  🔴 ≥ 350kB (First Load JS · gzip)

Lighthouse — timo-web
URL Perf A11y LCP CLS TBT
/en/home 🟡 71 🟢 96 🔴 16.3s 🟢 0.000 🟡 215ms
/en/today 🔴 69 🟢 96 🔴 16.2s 🟢 0.000 🟡 288ms
/en/focus 🔴 69 🟢 96 🔴 16.1s 🟢 0.000 🟡 276ms
/en/statistics 🟡 70 🟢 96 🔴 16.1s 🟢 0.000 🟡 236ms

Perf ≥ 70 / A11y ≥ 85 목표
LCP 🟢 < 2.5s 🟡 < 4s 🔴 ≥ 4s  |  CLS 🟢 < 0.1 🟡 < 0.25 🔴 ≥ 0.25  |  TBT 🟢 < 200ms 🟡 < 600ms 🔴 ≥ 600ms

Image Optimization — timo-web
파일 크기 포맷 상태
favicon.png 27.84 kB PNG ⚠️ 🟢
images/google-calendar.png 36.20 kB PNG ⚠️ 🟢
images/google-logo.png 26.79 kB PNG ⚠️ 🟢
og.png 437.44 kB PNG ⚠️ 🟡

총 4개 · 528.28 kB  |  🟢 < 200KB  |  🟡 < 500KB  |  🔴 ≥ 500KB
⚠️ 4개 파일 WebP/AVIF 변환 권장

측정 커밋: 76742a8

@yumin-kim2 yumin-kim2 left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다-!! 빠른 반영 굿굿입니당👍

@jjangminii jjangminii left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image 확인해 보니 모달 최종 디자인이 나와서 다시 반영해야 할 것 같아요 😢

탈퇴하기 버튼을 왼쪽, 취소하기 버튼을 오른쪽으로 옮기고 취소 문구를 취소하기로 수정했습니다.
@ehye1

ehye1 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

다시 최종 디자인 반영했습니다!

@ehye1
ehye1 requested a review from jjangminii October 1, 2026 02:55
@github-actions github-actions Bot added the 🫵🏻 Reminded 리마인드 완료 label Oct 1, 2026

This branch was successfully deployed

1 active deployment
Preview — 0ba4fb95 Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♥️ 혜원 혜원양 🐛 Bug 기능이 정상적으로 작동하지 않는 문제 수정 🫵🏻 Reminded 리마인드 완료 ⏰ Timo-web Timo 웹 서비스

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 회원 탈퇴 확인 팝업 버튼 문구 및 스타일 수정

3 participants