Skip to content

Update autocmd.{txt,jax} #2051

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

h-east
Copy link
Member

@h-east h-east commented Apr 22, 2025

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.

コメントしました

Comment on lines +651 to +654
|CmdlineLeave| の前に発生する。この情報は
|CmdlineLeave| が呼び出される前にクリアされる
ため、|cmdcomplete_info()| で補完情報を取得す
るのに便利である。マッピングで ":" を非対話的
Copy link
Contributor

Choose a reason for hiding this comment

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

英語のほうの this information は、前段にある completion inlfo を受けてのもののはず。

日本語で先にこの情報としてしまうと意味が不明確になるので

Suggested change
|CmdlineLeave| の前に発生する。この情報は
|CmdlineLeave| が呼び出される前にクリアされる
ため、|cmdcomplete_info()| で補完情報を取得す
るのに便利である。マッピングで ":" を非対話的
|CmdlineLeave| の前に発生する。補完情報は
|CmdlineLeave| が呼び出される前にクリアされる
ため、|cmdcomplete_info()| はこの情報を取得す
るのに便利である。マッピングで ":" を非対話的

のように入れ替えてはどうでしょう?

あと、てにおはをちょっとだけ変更しました。

@@ -404,6 +404,7 @@ Vimは以下のイベントを認識する。イベント名が大文字か小
|CmdlineChanged| コマンドラインのテキストに変更が加えられた後
|CmdlineEnter| カーソルがコマンドラインに移動した後
|CmdlineLeave| カーソルがコマンドラインを離れる前
|CmdlineLeavePre| コマンドラインを離れる前
Copy link
Contributor

Choose a reason for hiding this comment

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

preparing が抜けてしまっている気がするので

Suggested change
|CmdlineLeavePre| コマンドラインを離れる前
|CmdlineLeavePre| コマンドラインを離れる準備の前

とかはどうでしょうか?

|InsertLeave|		挿入モードを抜けるとき
|InsertLeavePre|	挿入モードを抜ける直前

にならったり、合せてもいいかもしれませんが... CmdlineLeave も調整しないとですね

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.

2 participants