Conversation
- answer 모드에서 span으로 렌더링하여 제목 뒤에 * 자연스럽게 붙도록 수정 - RequiredStar에 margin-left: 4px 추가
[fix] 모바일 지원서 UI 깨짐 수정
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | 요약 |
|---|---|
QuestionTitle 컴포넌트 및 스타일frontend/src/components/application/QuestionTitle/QuestionTitle.styles.ts, frontend/src/components/application/QuestionTitle/QuestionTitle.tsx |
QuestionTitleContainer 내보내기 제거, QuestionTitleTextContainer를 QuestionTitleTextWrapper로 재명명 (inline-flex, width: 100% 업데이트). QuestionRequired를 RequiredStar 및 QuestionTitleReadOnly 새 내보내기로 대체. 모드 전환 로직을 추가하여 빌더 모드에서는 편집 가능한 textarea, 답변 모드에서는 읽기 전용 표시 렌더링. useLayoutEffect 확장으로 빌더 모드에만 자동 높이 조정 제한. |
ShortText 컴포넌트frontend/src/components/application/questionTypes/ShortText.tsx |
루트 JSX 래퍼를 div에서 React Fragment (<>...</>)로 변경하여 불필요한 DOM 노드 제거. |
InputField 반응형 스타일frontend/src/components/common/InputField/InputField.styles.ts |
작은 화면에서 전체 너비를 적용하기 위해 모바일 미디어 쿼리 중단점 규칙 추가. |
예상 코드 리뷰 노력
🎯 3 (Moderate) | ⏱️ ~20 minutes
관련 가능성 있는 PR
- [hotfix] 모바일 지원서 UI 깨짐 수정 #1029: QuestionTitle.styles.ts와 QuestionTitle.tsx의 동일한 파일 및 내보내기 기호(QuestionTitleContainer/TextContainer 제거, RequiredStar 및 QuestionTitleReadOnly 추가)를 수정하여 직접 관련.
- [hotfix] 지원서 제목이 정상적으로 바뀌지않는 오류 수정 #678: QuestionTitle 컴포넌트를 수정하고 편집/읽기 전용 처리를 변경하여 동일한 파일 및 내보내기를 업데이트하므로 직접 관련.
- [fix]지원서가 만들어지지 않던 문제 해결 #679: contentEditable div를 textarea로 교체하고 제어되는 onChange/readOnly 동작을 업데이트하여 직접 관련.
제안 레이블
📈 release, 🛠Fix
제안 리뷰어
- lepitaaar
- seongwon030
- suhyun113
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Title check | ❓ Inconclusive | PR 제목은 버전 릴리스를 나타내지만, 실제 변경사항인 QuestionTitle 필수 지시자 위치 조정, QuestionTitle 번호 정렬, InputField 모바일 반응형 스타일 추가 등의 구체적 내용을 반영하지 않습니다. | PR 제목을 더 구체적으로 변경하세요. 예: '[hotfix] FE QuestionTitle required indicator and InputField mobile responsive' 또는 주요 변경사항을 명시적으로 포함하세요. |
✅ Passed checks (2 passed)
| Check name | Status | Explanation |
|---|---|---|
| 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. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
#️⃣연관된 이슈
#1028
📝작업 내용
모바일 지원서 UI 긴급 핫픽스
QuestionTitle 필수 표시(*) 위치 및 번호 정렬 수정
InputField 모바일 반응형 스타일 추가
🫡 참고사항
임시 대응으로, 추후 InputField 및 QuestionTitle 컴포넌트 리팩토링 필요
Summary by CodeRabbit
새로운 기능
개선 사항
✏️ Tip: You can customize this high-level summary in your review settings.