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

Update map.{txt,jax} #1460

Merged
merged 1 commit into from
Mar 19, 2024
Merged

Update map.{txt,jax} #1460

merged 1 commit into from
Mar 19, 2024

Conversation

h-east
Copy link
Member

@h-east h-east commented Mar 14, 2024

No description provided.

Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

ちょっとコメントしましたが、あまり重要というほどじゃないかな

1.12 superキーまたはcommandキーのマップ |:map-super-keys|
1.13 modifyOtherKeys モードのマップ |modifyOtherKeys|
1.14 Kitty キーボードプロトコルを使ったマップ |kitty-keyboard-protocol|
1.15 オペレータの作成 |:map-operator|
Copy link
Contributor

Choose a reason for hiding this comment

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

ここですが、mappingのニュアンスを追加する感じに(再訳)はしなくていいですかね?
流れで分るのはありますが。

Copy link
Member Author

Choose a reason for hiding this comment

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

影響箇所が多い(1, 1.3, 1.4, 1.5, 1.9 ~ 1.15)ので今回はこのままで。
Issueを立てていただけると助かります。

Copy link
Contributor

Choose a reason for hiding this comment

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

作成 #1464

@@ -586,10 +587,12 @@ While mappings are being listed, it is not possible to add or clear mappings,
e.g. from a timer callback. *E1309*

*:map-verbose*
When 'verbose' is non-zero, listing a key map will also display where it was
When 'verbose' is non-zero, the detected and used 'keyprotocol' value will be
Copy link
Contributor

Choose a reason for hiding this comment

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

いってもしょうがないところですけど、non-zeroだと負数もありえそうな記述ですね(原文のみ)

だからというのはあまりないですが...(セットする人もそうそういなさそう?)

Copy link
Member Author

Choose a reason for hiding this comment

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

:vimgrep "verbose.*non-zero" *.txt したら 13か所あったので、今回はそのままで。
本家に修正提案するのはアリだと思います。

Copy link
Contributor

Choose a reason for hiding this comment

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

作成 #1465
他も含めて調査しないとだと思ったので


1.12 MAPPING IN modifyOtherKeys mode *modifyOtherKeys*
The Super modifier is available in GUI mode (when |gui_running| is 1) for
GVim on Linux and MacVim on Mac OS. If you're on a Mac, this represents the
Copy link
Member

Choose a reason for hiding this comment

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

原文の話になりますが、文頭ではないので、"GVim" ではなく "gVim" 表記の方が良さそうですね。参考:
https://github.com/vim/vim/blob/f6272551bdae3f265b6948a4155b079c37fe110f/runtime/doc/intro.txt#L46-L47

Copy link
Member

Choose a reason for hiding this comment

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

typo 報告スレの方に報告しました。vim-jp/issues#1387 (comment)

もし仮にこれを本家 help の typo だとするとして、この場で jax の方の訳文をどうするかはお任せします。

Copy link
Member Author

Choose a reason for hiding this comment

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

本家修正後で 👍

@h-east
Copy link
Member Author

h-east commented Mar 18, 2024

私の返信で特に問題がなければmergeしてしまおうかと思います。

Copy link
Member

@mityu mityu left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@h-east h-east merged commit 8497eb2 into master Mar 19, 2024
3 checks passed
@h-east h-east deleted the hh-update-map branch March 19, 2024 12:36
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.

None yet

3 participants