Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add i18n supports for Korean #333

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

niceman114
Copy link

Hi! I really like mainroad theme.
However, since there is no Korean language support, I make PR.
I hope for your positive review. 😃

@niceman114
Copy link
Author

niceman114 commented Jul 14, 2023

@Vimux Check this pr asap, plz! 😄

Copy link
Owner

@Vimux Vimux left a comment

Choose a reason for hiding this comment

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

Hi. Thank you for the translation. There are a couple of minor things that need to fix before merging.


Check this pr asap, plz! 😄

Speed of light 😅

translation: "주의"

- id: sidebar_recommendation
translation: "하나 이상의 사이드바 위젯(params.sidebar.widgets)을 활성화해 주시기 바랍니다."
Copy link
Owner

Choose a reason for hiding this comment

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

sidebar_recommendation:

It should be consistent with the other translations. Do not specify a specific parameter (params.sidebar.widgets).

translation: "<b>$HUGE_HOME/content</b> 디렉토리 아래에 게시물(*.md)을 게시하면 여기에 표시됩니다.<br />기본적으로, 가장 많은 글이 있는 하나의 섹션을 기준으로 합니다."

- id: noposts_warning_tip
translation: 'Tip: <a href="https://gohugo.io/functions/where/#mainsections" rel="nofollow noopener" target="_blank">params.mainSections</a> 설정 매개변수를 사용하여, 원하는 만큼의 섹션을 표시할 수 있습니다.'
Copy link
Owner

Choose a reason for hiding this comment

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

noposts_warning_tip:

  1. Use double quotes (escape the double quotes inside).
  2. params.mainSections -> mainSections.
    It should be consistent with the other translations. Do not use a specific parameter (params.mainSections).

translation: "표시할 게시물이 없습니다!"

- id: noposts_warning_description
translation: "<b>$HUGE_HOME/content</b> 디렉토리 아래에 게시물(*.md)을 게시하면 여기에 표시됩니다.<br />기본적으로, 가장 많은 글이 있는 하나의 섹션을 기준으로 합니다."
Copy link
Owner

@Vimux Vimux Jul 15, 2023

Choose a reason for hiding this comment

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

noposts_warning_description: Please try to rephrase closer to the original if possible. If not, get rid of direct references to specific formats and dirs. Reasons:

  1. $HUGE_HOME/content? Hugo supports different content dirs. In the default language, we specify that users should put the post in the content directory. Content directory is not always directly equal to ./content.
  2. (*.md)? Hugo supports multiple content formats. Stating only one specific (*.md) is not correct and causes more confusion.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants