-
Notifications
You must be signed in to change notification settings - Fork 65
Update options.{txt,jax} #1930
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
Update options.{txt,jax} #1930
Conversation
mityu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ぼちぼち復帰できそうです。
2箇所だけコメントしました。他は良さげです。
doc/options.jax
Outdated
| 'scrollbind' がオンのときには大抵便利である。 | ||
|
|
||
| foldcolumn:{n} 差分モードの開始時にオプション 'foldcolumn' を | ||
| {n}に設定する。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
原文の "Without this 2 is used." の訳が抜けていそうです。...ただこの文が何を言ってるのかがよくわからない...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
よくわかりませんね... 2だけ設定できない?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ、「これが設定されていなければ、デフォルト値として 2 が使用される」という意味ですかね?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
しっくりきました。
doc/options.jax
Outdated
| 行の2 バッファ差分ハンク、またはそれぞれ 20 行 | ||
| の 3 バッファ差分ハンクの揃えが可能になる。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| 行の2 バッファ差分ハンク、またはそれぞれ 20 行 | |
| の 3 バッファ差分ハンクの揃えが可能になる。 | |
| 行の 2 バッファ差分ハンク、またはそれぞれ 20 | |
| 行の 3 バッファ差分ハンクの揃えが可能になる。 |
空白抜けです
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L2997の " の前に空白がないのも発見したのでローカルで修正しました。
tsuyoshicho
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
気付けたところはすべて指摘しました。
Co-authored-by: Tsuyoshi CHO <[email protected]>
Co-authored-by: Tsuyoshi CHO <[email protected]>
Co-authored-by: Tsuyoshi CHO <[email protected]>
Co-authored-by: Tsuyoshi CHO <[email protected]>
Co-authored-by: Tsuyoshi CHO <[email protected]>
tsuyoshicho
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
LGTM 👍 |
'diffopt' のところはアルファベット順にソートされましたが、"linematch:" が追加されただけです。