Skip to content

Conversation

@h-east
Copy link
Member

@h-east h-east commented Jun 7, 2025

tabpanel関連は先行して直したりしてます。
vim-jp/issues#1433 (comment)

tabpanelopt の "wrap" 項目はpatch 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.

みました。他は良さげです。

>
+------ This width
|
<----+----->
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
Member Author

Choose a reason for hiding this comment

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

いつもの(タブ、スペースのMix)にした方がいいですね。

Comment on lines +441 to +442
The 'tabpanel' and 'showtabpanel' options function similar to
'statusline' or 'tabline'.
Copy link
Member

Choose a reason for hiding this comment

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

ここの部分、'tabpanel' が 'statusline' や 'tabline' と似た機能を持つというところはわかるのですが、'showtabpanel' は別でむしろ対応するのは 'laststatus' や 'showtabline' だと思うので、「'tabpanel' と 'showtabpanel' オプションは 'statusline' や 'tabline' オプションと同じ機能を持つ」と言われるとちょっと変な感じがするのですがどうでしょう?

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#1433 (comment)

typo 報告スレに投げてみました。

@h-east h-east changed the title Update tabpanel.{txt,jax} Update tabpage.{txt,jax} Jun 7, 2025
@h-east
Copy link
Member Author

h-east commented Jun 8, 2025

原文更新しましたが、日本語訳は先行して修正していたので日付のみの更新です 👍

@h-east h-east requested a review from mityu June 8, 2025 00:16
@h-east h-east merged commit 2b1ee49 into vim-jp:master Jun 9, 2025
2 checks passed
@h-east h-east deleted the update-tabpanel branch June 9, 2025 13:52
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