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

fix: .jax subsection and others. #1508

Merged
merged 6 commits into from
May 28, 2024
Merged

Conversation

tsuyoshicho
Copy link
Contributor

@tsuyoshicho tsuyoshicho commented May 5, 2024

GW Work

小見出し (☆ を先頭に付けることで対応できる) の修正を全体的に実施しました。
また、あわせて気付いた箇所の修正をしています。

なお、その修正内容的に、英語原文と比較しながら確認を実施する必要はあります...

基本的には全部見れたと思いますが、まだ残ってる可能性はあります。
修正には是非はあると思いますので、指摘していただければ直す/外すなど対応します。

修正内容については各ファイルにコメントなどしていきます。

修正内容的に、急ぎではありません。
(コンフリクトがあったときに適時追従はします、先に気付いたら連絡をpingしていただけると助かります。)

@tsuyoshicho tsuyoshicho self-assigned this May 5, 2024
doc/cmdline.jax Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

各小見出し修正にあわせ、強調のために「強 調」のようになっている記述を修正

Copy link
Contributor Author

@tsuyoshicho tsuyoshicho May 5, 2024

Choose a reason for hiding this comment

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

VIM IS... VI COMPATIBLE のような記述を翻訳

☆Vimは……Vi互換である のように修正していますので

  • スペース空けたほうがいい
  • … はやめて... のままがいい

などあれば

doc/indent.jax Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

最新の英語版とフォーマット/スタイルで差分があったので、修正しています。

@@ -553,7 +553,7 @@ Note: 表示と入力は独立している。入力したい言語のインプ
り見た目が良くない。より良いフォントを選ぶには下記の説明を参照。


X論理フォント名 (X LOGICAL FONT DESCRIPTION (XLFD))
X論理フォント名 (X LOGICAL FONT DESCRIPTION (XLFD))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ここは意味ある感じがします

doc/quickfix.jax Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

☆quickfix または location リスト

に整理

doc/remote.jax Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

一部 ~ で強調している箇所を修正(原文と比較して検知)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

  • ~ の対応
  • 翻訳されてなかった箇所対応
  • 配置修正

などしています。

doc/uganda.jax Outdated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ライセンス文内なので VIM ライセンス は対応していません

@tsuyoshicho tsuyoshicho requested review from h-east, k-takata and mityu May 5, 2024 09:54
@tsuyoshicho tsuyoshicho requested a review from h-east May 11, 2024 00:01
Copy link
Member

@h-east h-east left a comment

Choose a reason for hiding this comment

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

cmdline, develop, diff, editing, fold, ft_ps1, gui, helphelp, insert, mbyte, motion
をレビューしました。

Copy link
Member

@h-east h-east left a comment

Choose a reason for hiding this comment

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

ひと通りレビューしました。

@tsuyoshicho
Copy link
Contributor Author

ありがとうございます。
指摘反映します。

@tsuyoshicho tsuyoshicho mentioned this pull request May 17, 2024
@tsuyoshicho tsuyoshicho requested a review from h-east May 17, 2024 12:48
Copy link
Member

@h-east h-east left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tsuyoshicho
Copy link
Contributor Author

ping @k-takata san, @mityu san.

@h-east
Copy link
Member

h-east commented May 27, 2024

@k-takata, @mityu : 7時間後くらいにmergeしようと思います。レビュー中、またはレビュー予定がある場合は、その旨お伝えください。

@k-takata
Copy link
Member

この行は変更なしでしょうか?

X-WindowでUTF-8を使う (USING UTF-8 IN X-Windows) *utf-8-in-xwindows*

@tsuyoshicho
Copy link
Contributor Author

To: @k-takata cc: @h-east

この行は変更なしでしょうか?

X-WindowでUTF-8を使う (USING UTF-8 IN X-Windows) *utf-8-in-xwindows*

指摘ありがとうございます

他にもちょっとあったけど、UPPER CASEになりきってないために対象行じゃない状態になっていたので、処置していません

  • 原文を直すdiff出す
  • 今回で☆つけたほうがいい
  • 今回で括弧も削除はしたほうがいい

について、対応のご意見いただきたいです。

現状、差分は別途やったほうがいいかなと(ぼんやり頭の中で)考えてました
(そして原文変更待ちとして、今回修正としてはしてない状態です)

けどなにも書いてなかったのはだめでしたね...すみません

@k-takata
Copy link
Member

☆は原文と合わせるべきかと思いますが、括弧は削除してしまっていいのではないかと思いました。

@tsuyoshicho
Copy link
Contributor Author

@k-takata
上記に対応しました。
類似を確認しましたが、当面はなさそうです

小見出しになってない部分の英語版対応PRおよびmbyteで普通の章タイトルの括弧の対応をIssueにしておきます。

@h-east h-east merged commit c87b2a6 into vim-jp:master May 28, 2024
2 checks passed
@h-east
Copy link
Member

h-east commented May 28, 2024

Thanks 👍

@tsuyoshicho tsuyoshicho deleted the fix/subsection branch May 28, 2024 11:21
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.

3 participants