Update popup.{txt,jax}#2627
Open
h-east wants to merge 1 commit into
Open
Conversation
mityu
reviewed
May 17, 2026
Member
mityu
left a comment
There was a problem hiding this comment.
ちょっと細かい指摘も混ざってしまった気がするんですが、いくつか気づいた点をコメントしました。
Comment on lines
+730
to
+731
| と、ポップアップは非表示になる。デフォルトは FALSE で | ||
| ある。 |
Member
There was a problem hiding this comment.
Suggested change
| と、ポップアップは非表示になる。デフォルトは FALSE で | |
| ある。 | |
| と、ポップアップは非表示になる。 | |
| デフォルトは FALSE である。 |
細かいようですが、原文ではここに陽に改行が入っていそうだったので揃えました。
Comment on lines
+963
to
+966
| ("host window") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定する | ||
| と、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ出し | ||
| たポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全に超 | ||
| えてスクロールすると、ポップアップは非表示になる。 |
Member
There was a problem hiding this comment.
Suggested change
| ("host window") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定する | |
| と、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ出し | |
| たポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全に超 | |
| えてスクロールすると、ポップアップは非表示になる。 | |
| ("ホストウィンドウ") の端を超えて広がる場合がある。"clipwindow" を TRUE に設定 | |
| すると、ポップアップはウィンドウのコンテンツ領域内に収まる。ウィンドウからはみ | |
| 出したポップアップの部分は切り取られ、テキストプロパティがウィンドウの端を完全 | |
| に超えてスクロールすると、ポップアップは非表示になる。 |
この付近で "host window" と検索しても、ここの部分と章タイトルの部分でしか引っ掛からなかったので、ここの原文の (the "host window") は、章タイトルに出てくる "HOST WINDOW" の部分の用語補足として置かれているようだなと。それを考慮するとここの訳文中の ("host window") は、翻訳版の章タイトルに合わせて「ホストウィンドウ」としておくほうが良いのではと思いました。
Comment on lines
+979
to
+980
| "posinvert" をデフォルト値 (TRUE) のままにしておくと、テキストプロパティにス | ||
| ペースがない場合、ポップアップが反対側に反転される可能性がある。指定された側を |
Member
There was a problem hiding this comment.
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あたりでウインドウを入れ替えてやってください。
- ステータスバーを上下に動かして、ポップアップの描画位置が変わるのを観測する
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.