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

Review Korean Translation #2

Merged
merged 1 commit into from
Jul 9, 2022
Merged

Review Korean Translation #2

merged 1 commit into from
Jul 9, 2022

Conversation

cottonpup
Copy link
Collaborator

안녕하세요 지효님!

띄어쓰기와 더 나은 번역에 있어서 의견을 내봤습니다.

한번 읽어보시고 확인 부탁드릴게요! :)

@@ -81,15 +81,15 @@
"Carousel": "캐러셀",
"Collapse": "접기",
"Countdown": "카운트다운",
"Kbd": "단축키",
"Kbd": "키보드",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이건 고민되었는데 방향키도 있는 컴포넌트라 단축키보단 키보드가 나을 거 같아서 변경했습니다.

"Tooltip": "툴팁",
"Data input": "데이터 인풋",
"Checkbox": "체크박스",
"Text input": "텍스트 인풋",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

툴팁은 개발자들에겐 툴팁이란 용어가 더 편한 거 같고 공식 리액트 한국어 문서에서도 툴팁이란 단어를 사용해서 변경해보았습니다.
데이터 인풋, 텍스트도 같은 이유입니다.

@@ -160,9 +160,9 @@
"But you can still customize components in many ways.": "하지만 여전히 여러 방법으로 컴포넌트를 커스터마이징 할 수 있습니다.",
"Let's say, you want to customize this button:": "이 버튼을 커스터마이징 하고 싶다면:",
"You can use daisyUI utility classes:": "데이지UI 유틸리티 클래스를 사용해 커스터마이징 할 수 있습니다:",
"You can use Tailwind's utility classes:": "Tailwind 유틸리티 클래스를 사용해 커스터마이징 할 수 있습니다:",
"You can use Tailwind's utility classes:": "Tailwind 유틸리티 클래스를 사용해 커스터마이징 할 수 있습니다.",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Screen Shot 2022-07-09 at 10 38 53 PM

https://github.com/reactjs/ko.react.dev/issues/27 리액트 모범 번역사례를 참고하였습니다.

Copy link
Owner

@jihyojeon jihyojeon left a comment

Choose a reason for hiding this comment

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

감사합니다 ㅎㅎ

@jihyojeon jihyojeon merged commit e36d4c9 into master Jul 9, 2022
@cottonpup cottonpup deleted the cottonpup/ko-translation branch July 10, 2022 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants