Skip to content

Update syntax.{txt,jax} #1483

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

Merged
merged 12 commits into from
May 3, 2024
Merged

Update syntax.{txt,jax} #1483

merged 12 commits into from
May 3, 2024

Conversation

h-east
Copy link
Member

@h-east h-east commented Apr 10, 2024

No description provided.

@h-east
Copy link
Member Author

h-east commented Apr 10, 2024

ft-ocaml-syntax の部分が最新と異なっていたので、あわせて修正しました。

@tsuyoshicho tsuyoshicho self-requested a review April 11, 2024 13:36
doc/syntax.jax Outdated
Comment on lines 5213 to 5218
これにより、端末で使用する代替フォント番号が得られる。使用可能なフォン
トは端末によって異なる。端末が代替フォント用に設定されていない場合は、
単に何もしない。{font-nr} の範囲は 0-10 で、0 はフォントをデフォルト
フォントにリセットし、1-9 は 9 つの代替フォントの 1 つを選択し、10 は
Fraktur フォントを選択する。詳細については、端末による SGR パラメータ
10-20 の処理を参照。 |t_CF|
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
これにより、端末で使用する代替フォント番号が得られる。使用可能なフォン
トは端末によって異なる。端末が代替フォント用に設定されていない場合は、
単に何もしない。{font-nr} の範囲は 0-10 で、0 はフォントをデフォルト
フォントにリセットし、1-9 は 9 つの代替フォントの 1 つを選択し、10 は
Fraktur フォントを選択する。詳細については、端末による SGR パラメータ
10-20 の処理を参照。 |t_CF|
端末で使用する代替フォント番号を指定する。使用可能なフォントは端末に
よって異なる。端末が代替フォント用に設定されていない場合は、単に何もし
ない。{font-nr} の範囲は 0-10 で、0 はフォントをデフォルトフォントにリ
セットし、1-9 は 9 つの代替フォントの 1 つを選択し、10 は Fraktur フォ
ントを選択する。詳細については、端末による SGR パラメータ10-20 の処理
を参照。 |t_CF|

s/これにより、端末で使用する代替フォント番号が得られる。/端末で使用する代替フォント番号を指定する。/

ここは ctermfont の性質上こうなると思います。

Copy link
Member Author

Choose a reason for hiding this comment

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

「~を与える」としました。

Copy link
Member

Choose a reason for hiding this comment

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

「〜を与える」で訳は全く問題はないのですが、ctermfg や guifg あたりの原文が似たような "These give ~" となっていて、それの訳が「〜を指定する」となっているので、「〜を指定する」の方が周りとの兼ね合いは良くなるのかなーと思います。

Copy link
Member Author

Choose a reason for hiding this comment

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

修正しましたが、過去のあまり良くない訳と合わせる必要はないんじゃないかなって思いました。

Copy link
Member

Choose a reason for hiding this comment

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

過去のあまり良くない訳と合わせる必要はないんじゃない

それはそうですね。私の中では今回の部分は「与える」でも「指定する」でも訳の出来にそんな差はないだろうという感覚だったので、じゃあ周りと合わせた方が綺麗かな?という感じだったのですが、「与える」の訳の方が良さそうならこの近辺丸ごと訳を変えても良いかもしれませんね。

@mityu
Copy link
Member

mityu commented Apr 21, 2024

一通り見て、自分の気になった点などについてコメントしました。

@tsuyoshicho tsuyoshicho requested a review from k-takata April 27, 2024 00:28
@h-east h-east requested a review from mityu April 29, 2024 12:34
@h-east h-east merged commit d253be8 into master May 3, 2024
3 checks passed
@h-east h-east deleted the hh-update-syntax branch May 3, 2024 18:15
@h-east
Copy link
Member Author

h-east commented May 3, 2024

Thanks for reviewing.

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.

4 participants