-
Notifications
You must be signed in to change notification settings - Fork 0
[Fix] following 이미지 및 navigate 추가 #115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
📊 Coverage Report
📈 #115을 main에 병합하면 coverage가 Coverage 요약@@ Coverage Diff @@
## main #115 +/- ##
===========================================
+ Coverage 30.29% 30.39% +0.10%
===========================================
Files 105 105 0
Lines 4182 4188 +6
Branches 190 191 +1
===========================================
+ Hits 1267 1273 +6
Misses 2915 2915 0 영향받은 파일✅ 이 PR로 영향받은 파일이 없습니다 수정된 모든 파일이 현재 coverage를 유지했습니다. |
🎨 Storybook Report✨ Story가 변경되었습니다 Chromatic에서 비주얼 변경사항을 확인하세요.
|
🚀 PR Preview Report✨ Build가 성공적으로 완료되었습니다. Preview에서 변경사항을 확인하세요.
|
yoorli
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다.
| id={0} | ||
| name='얼룩말' | ||
| profileImage='https://cdn.pixabay.com/photo/2025/11/28/15/29/zebras-9983175_1280.jpg' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
id, name, profileImage가 다 동일하다면 공통으로 묶는 것도 코드도 짧아지고 수정도 쉬워져서 좋을 거 같아요.
const baseArgs = {
id: 0,
name: '얼룩말',
profileImage: '... URL...',
profileMessage: '안녕하세요! 반갑습니다',
};
export const FollowingCardTable: Story = {
args: {
...baseArgs,
// 다른 값 오버라이드
},
};| router.push(`/profile/${id}`); | ||
| }; | ||
| return ( | ||
| <div |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
지금 카드가 div로 되어 있는데, 스크린리더에서는 텍스트 콘텐츠만 알리고 클릭 가능한 요소로 인식이 안되거나, 키보드(Tab, Enter, Space 등)으로 접근이 어려울 수 있어요.
지금 당장 필요하거나 중요한 사항은 아니니 나중에 리팩토링 할 때 고려해 보는 것도 좋을 거 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
좋은 지적 감사합니다!!
📝 변경 사항
🔗 관련 이슈
Closes #
🧪 테스트 방법
📸 스크린샷 (선택)
📋 체크리스트
💬 추가 코멘트
CodeRabbit Review는 자동으로 실행되지 않습니다.
Review를 실행하려면 comment에 아래와 같이 작성해주세요