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 visual.txt #1121

Merged
merged 6 commits into from
Jul 22, 2022
Merged

update visual.txt #1121

merged 6 commits into from
Jul 22, 2022

Conversation

vampire-yuta
Copy link
Contributor

翻訳しました。ご確認よろしくお願いします 😄

doc/visual.jax Outdated Show resolved Hide resolved
doc/visual.jax Outdated
Comment on lines 497 to 498
ビジュアルモードであるとき、ビジュアルモードを終了するが、選択モードがマッ
プされたキーを実行後に復元するため、選択モードでは終了しない。以下を使用す
Copy link
Member

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.

レビューありがとうございます:smile:
こちらのコメントで対応しました。
a89c7eb

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.

レビューしました。

doc/visual.jax Outdated
@@ -492,6 +492,13 @@ abcdefghijklmnXXXXXXuvwxyz
われる。ビジュアルモードと選択モードで別々にマッピングを定義するときは|:xmap|
や|:smap|を使うこと。

特定のエッジケースである場合は、以下を使用する: >
Copy link
Member

Choose a reason for hiding this comment

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

特殊なケースの一例として、L496の:vnoremap ~を挙げて、ビジュアルモードと選択モードで挙動が異なるという説明をした上で、解決策として L500の:snoremap ~を示している流れだと思うので、少し表現が違うと思います。

Suggested change
定のエッジケースである場合は、以下を使用する: >
殊なエッジケースの一例: >

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/visual.jax Outdated Show resolved Hide resolved
@@ -509,6 +509,13 @@ work both in Visual mode and in Select mode. When these are used in Select
mode Vim automatically switches to Visual mode, so that the same behavior as
in Visual mode is effective. If you don't want this use |:xmap| or |:smap|.

One particular edge case: >
:vnoremap <C-K> <Esc>
Copy link
Member

Choose a reason for hiding this comment

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

空白とタブが混在しているので本家に修正をお願いする必要があります。
が、vim-jp/issues#1387 にpatchを貼っておけば、誰かがまとめてvim_devにPRしてくれます。
今回はこちらで対応しました。vim-jp/issues#1387 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

あ、なるほどですね。ありがとうございます 😄

vampire-yuta and others added 3 commits July 22, 2022 21:12
レビューありがとうございます!
なるほどですね。微妙な翻訳で恐縮です 🙏

Co-authored-by: K.Takata <kentkt@csc.jp>
レビューありがとうございます 🙏
こちらのCommit suggestionを適用しました。精進します。

Co-authored-by: h_east <h.east.727@gmail.com>
@vampire-yuta
Copy link
Contributor Author

Autofmtプラグインを導入し、レビューコメント対応しました。
よろしくお願いします 👏

@k-takata k-takata merged commit eca78e7 into vim-jp:master Jul 22, 2022
@k-takata
Copy link
Member

ありがとうございます。

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