Skip to content

Comments

[feature] 지원하기 모달 텍스트를 변경한다#872

Merged
seongwon030 merged 2 commits intodevelop-fefrom
feature/#861-change-apply-modal-text-MOA-362
Nov 20, 2025
Merged

[feature] 지원하기 모달 텍스트를 변경한다#872
seongwon030 merged 2 commits intodevelop-fefrom
feature/#861-change-apply-modal-text-MOA-362

Conversation

@suhyun113
Copy link
Collaborator

@suhyun113 suhyun113 commented Nov 20, 2025

#️⃣연관된 이슈

ex) #861

📝작업 내용

상세 페이지 지원하기 버튼 모달 텍스트를 수정했습니다.

수정하기 전
수정하기 전
수정 후
수정 후
  • 디자인 가이드에 맞게 버튼 색상 변환
  • 모달 제목 변환
  • 모달 설명 제거

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

릴리스 노트

  • 개선사항

    • 모달 제목을 "지원서 선택"으로 변경하고 설명 텍스트를 제거했습니다.
  • 스타일

    • 버튼의 호버 상태 색상을 미세 조정했습니다.
  • 정리

    • 미사용 코드를 정리했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@suhyun113 suhyun113 self-assigned this Nov 20, 2025
@suhyun113 suhyun113 added ✨ Feature 기능 개발 💻 FE Frontend labels Nov 20, 2025
@vercel
Copy link

vercel bot commented Nov 20, 2025

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

Project Deployment Preview Comments Updated (UTC)
moadong Ready Ready Preview Comment Nov 20, 2025 5:52am

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)
Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

개요

애플리케이션 선택 모달의 제목을 "지원 분야 선택"에서 "지원서 선택"으로 변경하고, OptionButton의 hover 상태 색상을 조정하며, 사용되지 않는 import를 제거했습니다.

변경사항

응집 / 파일 변경 요약
모달 헤더 텍스트 변경
frontend/src/components/application/modals/ApplicationSelectModal.tsx
모달 제목을 "지원 분야 선택"에서 "지원서 선택"으로 변경하고 description prop 제거
모달 스타일 조정
frontend/src/components/application/modals/ApplicationSelectModal.styles.ts
OptionButton의 hover 상태 배경 및 border-color를 #ff7a00에서 #ff7543으로 변경
불필요한 import 제거
frontend/src/components/common/Modal/Modal.styles.ts
'@/styles/mediaQuery'에서 사용되지 않는 media import 제거

추정 코드 리뷰 난이도

🎯 1 (간단함) | ⏱️ ~3분

  • 텍스트 변경과 색상 값 조정, 사용되지 않는 import 제거 등 모두 단순한 변경

관련 이슈

제안된 리뷰어

  • seongwon030
  • lepitaaar

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 모달 텍스트 변경이라는 주요 변경사항을 명확하게 설명하고 있으며, 변경 내용의 핵심을 잘 반영하고 있습니다.
Linked Issues check ✅ Passed PR의 모든 변경사항이 MOA-362 이슈의 요구사항을 충족합니다: 모달 제목 변경, 설명 제거, 버튼 색상 변경이 구현되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 지원하기 모달 텍스트 및 디자인 변경과 관련된 범위 내의 변경으로, 불필요한 import 제거를 포함하여 적절합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/#861-change-apply-modal-text-MOA-362

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 095c45d and 4af6aa4.

📒 Files selected for processing (3)
  • frontend/src/components/application/modals/ApplicationSelectModal.styles.ts (1 hunks)
  • frontend/src/components/application/modals/ApplicationSelectModal.tsx (1 hunks)
  • frontend/src/components/common/Modal/Modal.styles.ts (0 hunks)
💤 Files with no reviewable changes (1)
  • frontend/src/components/common/Modal/Modal.styles.ts
🔇 Additional comments (2)
frontend/src/components/application/modals/ApplicationSelectModal.tsx (1)

43-43: 모달 제목이 명확하게 변경되었습니다.

"지원 분야 선택"에서 "지원서 선택"으로 변경되어 사용자에게 더 명확한 의미를 전달합니다. PR 목표와 일치하는 변경사항입니다.

frontend/src/components/application/modals/ApplicationSelectModal.styles.ts (1)

27-29: 버튼 호버 색상이 디자인 가이드에 맞게 일관되게 적용되었습니다.

검증 결과, 새로운 색상 #ff7543ClubApplyButtonApplicationSelectModal 컴포넌트에 일관되게 사용되고 있으며, 이전 색상 #ff7a00은 완전히 제거되었습니다. 색상 변경이 적절히 완료되었으므로 승인합니다.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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 and usage tips.

Copy link
Member

@seongwon030 seongwon030 left a comment

Choose a reason for hiding this comment

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

디자인 통일 굳입니다~ 설명도 제거 굳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FE Frontend ✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] MOA-362 지원하기 모달 텍스트를 변경한다

2 participants