-
Notifications
You must be signed in to change notification settings - Fork 3
[fix] 한글 url 말줄임이 안 되는 오류를 해결한다 #1048
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
[fix] 한글 url 말줄임이 안 되는 오류를 해결한다 #1048
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| 집단 / 파일(들) | 요약 |
|---|---|
소셜 URL 스타일 업데이트frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.styles.ts |
SocialUrl 스타일 컴포넌트에 white-space: nowrap 추가하여 URL 텍스트 줄바꿈 방지 |
예상 코드 리뷰 소요 시간
🎯 1 (Trivial) | ⏱️ ~2분
관련 이슈
- [feature] MOA-510 SNS 링크 표시 방식을 개선한다 #1031: 클럽 프로필의 SNS 링크 렌더링 방식 개선과 직접 관련 있음
- MOA-510: 유튜브 링크 표시 방식 개선 목표 달성을 위한 스타일 조정
관련 PR
- [feature] SNS 링크 표시 방식을 개선한다 #1038: 동일한 ClubProfileCard.styles.ts의 SocialUrl 스타일 컴포넌트를 수정하는 관련 변경
검토자 추천
- seongwon030
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | PR 제목이 변경 사항과 직접적으로 관련되어 있으며, 한글 URL 말줄임 오류 해결이라는 주요 변경 사항을 명확하게 설명합니다. |
| Linked Issues check | ✅ Passed | 코드 변경이 MOA-510 이슈의 유튜브 링크 표시 방식 개선 목표를 충족하며, SocialUrl에 white-space: nowrap 추가로 URL 말줄임이 모든 언어에서 작동하도록 개선했습니다. |
| Out of Scope Changes check | ✅ Passed | 변경 사항이 ClubProfileCard 스타일 파일 내 SocialUrl 컴포넌트의 CSS 속성만 수정하였으며, 이슈 범위 내의 변경으로 범위 외 코드 변경은 없습니다. |
| 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! 한글 URL 말줄임 문제에 대한 올바른 수정입니다.
white-space: nowrap추가는text-overflow: ellipsis가 정상 작동하기 위한 필수 조건입니다. 한글 문자열은 브라우저에서 줄바꿈 가능으로 인식되어 기존 스타일만으로는 말줄임이 동작하지 않았는데, 이 수정으로 한글/영문 URL 모두에서 말줄임이 정상 적용됩니다.
✏️ 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.
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.
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)
frontend/src/pages/ClubDetailPage/components/ClubProfileCard/ClubProfileCard.styles.ts (1)
196-198:IntroSection의 미디어 쿼리 블록에 닫는 중괄호가 누락되었습니다.196-197줄의
${media.tablet}블록이}없이 종료되어 CSS 구문 오류가 발생합니다. 198줄의 템플릿 리터럴 종료(`;)가 미디어 쿼리 블록을 닫지 않고 있으므로, 다음과 같이 수정이 필요합니다:🐛 수정 제안
export const IntroSection = styled.section` padding: 16px; border-radius: 14px; background-color: ${colors.base.white}; ${media.tablet} { background-color: ${colors.gray[100]}; + } `;
📜 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! 한글 URL 말줄임 문제에 대한 올바른 수정입니다.
white-space: nowrap추가는text-overflow: ellipsis가 정상 작동하기 위한 필수 조건입니다. 한글 문자열은 브라우저에서 줄바꿈 가능으로 인식되어 기존 스타일만으로는 말줄임이 동작하지 않았는데, 이 수정으로 한글/영문 URL 모두에서 말줄임이 정상 적용됩니다.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
seongwon030
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.
영어는 줄바꿈이 안되고 한글은 되군요 .. 고생하셨습니다
#️⃣연관된 이슈
📝작업 내용
문제 상황
문제 원인
기존
Social URL은 말줄임 처리가 영어 URL로 정상 동작했습니다.이는 영어를 브라우저가 한 단어로 인식하기 때문에, 줄바꿈이 불가하여
max-width초과 시 바로 말줄임(...)이 적용되었던 것입니다.그런데, 한글 문자열은 브라우저가 줄바꿈이 가능하다고 인식해서
max-width를 초과해도 말줄임이 적용되지 않는 문제가 있었습니다.해결
따라서 한글 문자열을 위해
SocialUrl스타일에white-space: nowrap을 추가해주어 한글/영문 URL 모두 말줄임(...)이 적용되도록 했습니다.중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.