[FIX] 9월 2주차 QA 사항 - 지현#245
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Walkthrough멤버 리스트 프로필 이미지에 1px 테두리가 추가되었고, 역할 색상 매핑에서 민트 색 적용 조건이 문학/문헌 대신 인문/인문학/철학 계열로 변경되었습니다. 기록 아이템의 블러 강도가 2px에서 3px로 조정되었습니다. 로직/공개 API 변경은 없습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/components/members/MemberList.styled.ts (1)
66-87: 역할 색상 매핑 충돌 — 중복 분기 제거 및 '철학' 하늘색 반영 필요파일: src/components/members/MemberList.styled.ts (Lines 66–87)
- 문제:
인문관련 조건이 중복되어 앞 분기에서 mint를 반환하므로 이후의 skyblue(인문/철학) 분기가 도달 불가.- 문제: PR 요구사항(철학자 칭호 색상 = 하늘색) 및 디자인 토큰과 불일치.
- 문제: 문학(literature) → mint 매핑이 누락됨.
아래 제안된 분기 정리(diff)는 그대로 적용 가능함:
export const MemberRole = styled.div<{ roleType?: string }>` color: ${({ roleType }) => { if (!roleType) return semanticColors.text.point.green; const role = roleType.toLowerCase(); - if (role.includes('예술') || role.includes('art')) { + // 예술 + if (role.includes('예술') || role.includes('art')) { return semanticColors.text.character.pink; } - if (role.includes('인문') || role.includes('humanities') || role.includes('철학')) { - return semanticColors.text.character.mint; - } - if (role.includes('사회') || role.includes('sociology')) { + // 문학 + if (role.includes('문학') || role.includes('literature')) { + return semanticColors.text.character.mint; + } + // 사회 + if (role.includes('사회') || role.includes('sociology')) { return semanticColors.text.character.orange; } - if (role.includes('인문') || role.includes('humanities')) { - return semanticColors.text.character.skyblue; - } - if (role.includes('과학') || role.includes('science')) { + // 인문/철학 + if (role.includes('인문') || role.includes('humanities') || role.includes('철학') || role.includes('philosophy')) { + return semanticColors.text.character.skyblue; + } + // 과학 + if (role.includes('과학') || role.includes('science')) { return semanticColors.text.character.lavender; } return semanticColors.text.point.green; }};
🧹 Nitpick comments (2)
src/components/members/MemberList.styled.ts (1)
50-50: 기본 프로필에도 1px 보더 추가 LGTM
ProfileImageWithSrc와 시각적 일관성이 생겼습니다. 현재 두 곳 모두#888하드코딩인데, 추후 테마 토큰으로 통일하면 유지보수에 유리합니다.src/components/members/MemberList.tsx (1)
81-81: 프로필 이미지 존재 시 보더 적용 일관성 OK
ProfileImage/ProfileImageWithSrc모두1px solid #888로 통일되어 시각적 일관성이 확보됐습니다. 추후colors/semanticColors토큰 사용으로 하드코딩 제거를 고려해 주세요.
📜 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.
📒 Files selected for processing (3)
src/components/members/MemberList.styled.ts(3 hunks)src/components/members/MemberList.tsx(1 hunks)src/components/memory/RecordItem/RecordItem.styled.ts(1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/components/members/MemberList.styled.ts (1)
src/styles/global/global.ts (1)
semanticColors(80-153)
🔇 Additional comments (1)
src/components/memory/RecordItem/RecordItem.styled.ts (1)
6-6: 블러 강도 상향(2px → 3px) 적용은 OK. 단, 실제 가독성만 확인 부탁긴 리스트 스크롤 시 필터는 레이어 생성으로 약간의 퍼포먼스 비용이 있습니다. QA 단말(저사양/저해상도 포함)에서 텍스트 판독성과 스크롤 저하 없는지 확인해 주세요.
1 participant
#️⃣ 연관된 이슈
#244
📝 작업 내용
Summary by CodeRabbit