Skip to content

Update popup.{txt,jax}#2627

Open
h-east wants to merge 1 commit into
vim-jp:masterfrom
h-east:update-popup
Open

Update popup.{txt,jax}#2627
h-east wants to merge 1 commit into
vim-jp:masterfrom
h-east:update-popup

Conversation

@h-east
Copy link
Copy Markdown
Member

@h-east h-east commented May 12, 2026

No description provided.

Copy link
Copy Markdown
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.

LGTM

Copy link
Copy Markdown
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.

ちょっと細かい指摘も混ざってしまった気がするんですが、いくつか気づいた点をコメントしました。

Comment thread doc/popup.jax
Comment on lines +730 to +731
と、ポップアップは非表示になる。デフォルトは FALSE で
ある。
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
と、ポップアップは非表示になる。デフォルトは FALSE で
ある
と、ポップアップは非表示になる。
デフォルトは FALSE である

細かいようですが、原文ではここに陽に改行が入っていそうだったので揃えました。

Comment thread doc/popup.jax
Comment on lines +963 to +966
("host window") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定する
と、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ出し
たポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全に超
えてスクロールすると、ポップアップは非表示になる。
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
("host window") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定する
と、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ出し
ポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全に超
てスクロールすると、ポップアップは非表示になる。
("ホストウィンドウ") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定
ると、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ
したポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全
超えてスクロールすると、ポップアップは非表示になる。

この付近で "host window" と検索しても、ここの部分と章タイトルの部分でしか引っ掛からなかったので、ここの原文の (the "host window") は、章タイトルに出てくる "HOST WINDOW" の部分の用語補足として置かれているようだなと。それを考慮するとここの訳文中の ("host window") は、翻訳版の章タイトルに合わせて「ホストウィンドウ」としておくほうが良いのではと思いました。

Comment thread doc/popup.jax
Comment on lines +979 to +980
"posinvert" をデフォルト値 (TRUE) のままにしておくと、テキストプロパティにス
ペースがない場合、ポップアップが反対側に反転される可能性がある。指定された側を
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
"posinvert" をデフォルト値 (TRUE) のままにしておくと、テキストプロパティにス
ースがない場合、ポップアップが反対側に反転される可能性がある。指定された側を
"posinvert" をデフォルト値 (TRUE) のままにしておくと、スペースがない場合にポッ
アップがテキストプロパティの反対側に反転される可能性がある。指定された側を

「スペースがない場合」というのは、「(ポップアップを表示する十分な)スペースがない場合」だと思うので訳文はこうかなあと。

また、現状では "flipped" を「反転される」と訳していますが、これは実際の挙動などを考えると「移動される」としても良いかなあと思いました。
なお少し意訳っぽくなってしまうので、とりあえず↑の suggestion は「反転される」のままにしています。


「"posinvert" をデフォルト値〜」の挙動を眺めるための簡単なスクリプトを手元で用意したので、一応共有しておきます。

call prop_type_add('sample', {'highlight': 'Constant'})
let id = prop_add(1, 1, {'length': 0, 'type': 'sample'})
call popup_create(range(1, 10)->map('string(v:val)'), #{
	\ textprop: 'sample',
	\ textpropid: id,
	\ pos: 'topleft',
	\ line: -1, col: 0,
	\ clipwindow: v:true,
	\ })
  • 上のスクリプトを popup.vim として保存
  • vim --clean -S popup.vim
  • <C-w>s でウインドウを分割
    • ここで、分割後に下のウインドウ内にポップアップが表示されている状態にしておくのが大事です。
    • 分割した時点でそうなると思うのですが、もしならなかったら <C-w>r あたりでウインドウを入れ替えてやってください。
  • ステータスバーを上下に動かして、ポップアップの描画位置が変わるのを観測する

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.

3 participants