Skip to content
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

[feat] 어드민 기대평 여러가지 #113

Merged
merged 4 commits into from
Aug 19, 2024
Merged

Conversation

darkdulgi
Copy link
Collaborator

#️⃣ 연관 이슈

📝 작업 내용

  • 검색어를 입력하면 기대평을 검색 가능
  • 현재의 이벤트와 검색 쿼리의 문자열을 보이게 함
  • 기대평 문자열을 최대 길이 50까지만 보이게 함

길이가 50이 넘는 문자들은 ... 로 대체합니다.

@darkdulgi darkdulgi added the feat 기능 구현 label Aug 19, 2024
@darkdulgi darkdulgi self-assigned this Aug 19, 2024
Copy link
Collaborator

@lybell-art lybell-art left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.

Comment on lines +54 to +61
function truncateString(str) {
if (str.length > 50) {
return str.substring(0, 50) + "...";
} else {
return str;
}
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

만약 기대평이 한글만으로 이루어져 있을 때와, 영어로만 이루어져 있을 때를 상상해보셨나요?
한글이 영어보다 더 넓기 때문에 순수하게 문자열의 길이를 이용해 말줄임표를 구현하는 것보다는 css를 쓰는 게 적절합니다. (https://tailwindcss.com/docs/text-overflow#truncate)

@lybell-art lybell-art merged commit 29f1933 into dev Aug 19, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants