Skip to content
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

Fix duplication of side windows #117

Merged
merged 2 commits into from
Dec 28, 2014
Merged

Conversation

papaeye
Copy link
Contributor

@papaeye papaeye commented Dec 27, 2014

side window を開いている状態で popwin でバッファを開くと同名の非 side window が生成されます。

screen shot 1

のような状態です。下の *popwin-test* ウィンドウが最初に作成しておいた side window で、popwin でヘルプを開くと上の *popwin-test* ウィンドウが生成されます。

再現手順は次の通りです。

  1. emacs -Q で Emacs を起動後、以下を評価する

    (load "/path/to/popwin.el")
    (popwin-mode)
    
    (display-buffer-in-side-window (get-buffer-create "*popwin-test*") nil)
  2. C-h f popwin-mode で popwin でヘルプを見ようとする

    • 非 side window な *popwin-test* ウィンドウが生成され、それに重なって popwin ウィンドウでヘルプが見える
  3. C-g で popwin ウィンドウを閉じる

    • *popwin-test* ウィンドウが 2 つ残る

環境は OS X Mavericks | Emacs 24.4 | popwin HEAD です。

delete-other-windows で side window が削除されないのが要因のように思ったので popwin:window-config-tree-1 で side window を無視するようにしました。ただ

(display-buffer-in-side-window
 (get-buffer-create "*popwin-test*") '((side . left)))

とした場合には次のようになります。

screen shot 2

@m2ym
Copy link
Contributor

m2ym commented Dec 27, 2014

ありがとうございます。 side window なんてあったんですね、知りませんでした。一つお願いがあるのですが、 test/popwin-test.el にテストケースを追加していただけませんか。おそらく side window を開いた状態で

  • ポップアップできるか
  • ポップアップを閉じられるか
  • side window が残るか

を確認すれば十分だと思います。テストの実行は make test でできます。面倒そうでしたらこちらでやります。

@papaeye
Copy link
Contributor Author

papaeye commented Dec 27, 2014

テストケース追加してみました。ご確認お願いします。

atomic window というのもあるようです。side window は GitHub を検索すると sabof/project-explorer というのが使っていました。

m2ym pushed a commit that referenced this pull request Dec 28, 2014
Fix duplication of side windows
@m2ym m2ym merged commit 97cf1fa into emacsorphanage:master Dec 28, 2014
@m2ym
Copy link
Contributor

m2ym commented Dec 28, 2014

マージしました。ありがとうございます。

@papaeye papaeye deleted the fix-side-window branch December 28, 2014 10:23
@papaeye
Copy link
Contributor Author

papaeye commented Dec 28, 2014

ありがとうございました!

@m2ym
Copy link
Contributor

m2ym commented Dec 28, 2014

Emacs 24 から入った機能のようですね。将来的には popwin も要らなくなるのかもしれません。

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

Successfully merging this pull request may close these issues.

None yet

2 participants