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

端末をフルスクリーン・最大化するとポップアップメニューの残像が残る #1077

Closed
norio13 opened this Issue Aug 29, 2017 · 8 comments

Comments

Projects
None yet
4 participants
@norio13

norio13 commented Aug 29, 2017

質問・報告の内容

カーソル位置から上方向にポップアップメニューが表示されている状態で、端末を最大化・フルスクリーンにするとポップアップメニューの残像が残りました。
端末の縁をマウスドラッグで拡大した場合は再現しません。

sample.vim:

execute 'normal!' winheight('%') . 'o'
call feedkeys("Gi\<C-x>\<C-v>")

Vim起動:
端末を最大化・フルスクリーンされてない状態にしてからVimを起動します。
ポップアップメニューが表示されたら、手動で端末を最大化またはフルスクリーンにしてください。

$ vim --clean -S sample.vim

xfce4-terminalで再現した時の画像

Vimのバージョン

8.0.1010

OSの種類/ディストリ/バージョン

  • Linux Debian stretch 9.1 64bit

@ichizok ichizok added the kind/bug label Sep 4, 2017

@ichizok

This comment has been minimized.

Show comment
Hide comment
@ichizok
Member

ichizok commented Sep 5, 2017

https://github.com/vim/vim/compare/master...ichizok:fix/popupmenu.diff
単純に update_screen してしまう手。

@norio13

This comment has been minimized.

Show comment
Hide comment
@norio13

norio13 Sep 5, 2017

ありがとうございます。
ichizokさんのpatch適用で残像が消えました。

ichizokさんのpatchを適用するまで気付かなかったのですが、
patch未適用のVimで端末のサイズをフルスクリーン・最大化から元のサイズ(小さいサイズ)に変更すると、ポップアップメニューとカーソルが被ってました。
こちらもpatch適用で修正されているのが確認できました。

xfce4-terminalでポップアップメニューとカーソルが被っている画像:
xfce4-terminalでポップアップメニューとカーソルが被っている画像

norio13 commented Sep 5, 2017

ありがとうございます。
ichizokさんのpatch適用で残像が消えました。

ichizokさんのpatchを適用するまで気付かなかったのですが、
patch未適用のVimで端末のサイズをフルスクリーン・最大化から元のサイズ(小さいサイズ)に変更すると、ポップアップメニューとカーソルが被ってました。
こちらもpatch適用で修正されているのが確認できました。

xfce4-terminalでポップアップメニューとカーソルが被っている画像:
xfce4-terminalでポップアップメニューとカーソルが被っている画像

@ichizok

This comment has been minimized.

Show comment
Hide comment
@ichizok

ichizok Sep 17, 2017

Member

投げました。
vim/vim#2110

Member

ichizok commented Sep 17, 2017

投げました。
vim/vim#2110

@ichizok

This comment has been minimized.

Show comment
Hide comment
Member

ichizok commented Sep 20, 2017

@mattn

This comment has been minimized.

Show comment
Hide comment
@mattn
Member

mattn commented Oct 3, 2017

@k-takata

This comment has been minimized.

Show comment
Hide comment
@ichizok

This comment has been minimized.

Show comment
Hide comment
@ichizok
Member

ichizok commented Dec 27, 2017

8.0.1161
vim/vim@a5e6621

@norio13

This comment has been minimized.

Show comment
Hide comment
@norio13

norio13 Dec 27, 2017

norio13 commented Dec 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment