-
Notifications
You must be signed in to change notification settings - Fork 3
[hotfix] 유튜브링크 줄바꿈 문제 해결 #1049
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
[hotfix] 유튜브링크 줄바꿈 문제 해결 #1049
Conversation
…psis-MOA-510 [fix] 한글 url 말줄임이 안 되는 오류를 해결한다
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 코호트 / 파일(들) | 변경 요약 |
|---|---|
ClubProfileCard 스타일 수정 frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.styles.ts |
SocialUrl span 요소에 white-space: nowrap 속성 추가하여 텍스트 줄바꿈 방지 |
추정 코드 리뷰 시간
🎯 1 (Trivial) | ⏱️ ~3 minutes
관련될 수 있는 PR들
- [fix] 한글 url 말줄임이 안 되는 오류를 해결한다 #1048: ClubProfileCard.styles.ts의 동일한 SocialUrl 스타일 컴포넌트를 수정하여 URL 말줄임표 동작을 개선하는 직접 관련 PR
- [feature] SNS 링크 표시 방식을 개선한다 #1038: ClubProfileCard.styles.ts의 SocialUrl 스타일 컴포넌트를 수정하여 줄바꿈/말줄임표 동작을 제어하는 관련 PR
제안 라벨
🛠Fix
제안 리뷰어
- suhyun113
- PororoAndFriends
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목은 변경사항의 핵심 내용인 유튜브 링크의 줄바꿈 문제 해결을 명확하게 설명하고 있으며, 실제 코드 변경(SocialUrl의 white-space: nowrap 추가)과 일치합니다. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
- 📝 Generate docstrings
📜 Recent review details
Configuration used: Organization 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 (1)
frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.styles.ts
🔇 Additional comments (1)
frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.styles.ts (1)
165-189: LGTM!text-overflow: ellipsis동작을 위한 올바른 수정입니다.
text-overflow: ellipsis가 정상 작동하려면 세 가지 CSS 속성이 필요합니다:
overflow: hidden✓white-space: nowrap✓ (이번 추가)- 너비 제한 (
max-width) ✓기존에는
white-space: nowrap이 없어서 텍스트가 줄바꿈되었고, 따라서 가로 방향 오버플로우가 발생하지 않아 ellipsis가 표시되지 않았습니다. 특히 한글이 포함된 URL의 경우 CSS word-breaking 규칙으로 인해 예기치 않은 줄바꿈이 발생할 수 있어 이 수정이 필수적입니다.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
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.
suhyun113
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.
빠르게 수정갑시당~
#️⃣연관된 이슈
📝작업 내용
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.