[PATCH] lei: Delete message buffer's window along with query window

[PATCH] lei: Delete message buffer's window along with query window

[Kyle Meyer]

Within query buffers, a message for the current line can be displayed
while keeping point in the query buffer.  Calling quit-window in a
query buffer, however, leaves the associated buffer's window in place,
which is probably not what the caller wants.  Add a custom quit-window
variant that handles the message buffer too.

Suggested-by: Xinglu Chen <public@yoctocell.xyz>
Link: https://inbox.kyleam.com/piem/871r96am1q.fsf@yoctocell.xyz
---
 piem-lei.el | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/piem-lei.el b/piem-lei.el
index 6c537339..22814c74 100644
--- a/piem-lei.el
+++ b/piem-lei.el
@@ -375,6 +375,16 @@ (defun piem-lei-query-show-or-scroll-down (arg)
          (arg (- arg))
          (t '-))))
 
+(defun piem-lei-query-quit-window (&optional kill)
+  "Call `quit-window' on the selected window.
+If there is a visible `piem-lei-show-mode' buffer, first call
+`quit-window' on its window.  The prefix argument KILL is passed
+to both underlying `quit-window' calls."
+  (interactive "P")
+  (when-let ((msg-win (piem-lei-query--get-visible-message-window)))
+    (quit-window kill msg-win))
+  (quit-window kill))
+
 (defvar piem-lei-query-mode-map
   (let ((map (make-sparse-keymap)))
     (define-key map (kbd "RET") #'piem-lei-query-show)
@@ -384,6 +394,7 @@ (defvar piem-lei-query-mode-map
     (define-key map "p" #'piem-lei-query-previous-line)
     (define-key map "s" #'piem-lei-q)
     (define-key map "t" #'piem-lei-mid-thread)
+    (define-key map "q" #'piem-lei-query-quit-window)
     map)
   "Keymap for `piem-lei-query-mode'.")
 

base-commit: d7d15a74c933e9a108c47f0515fc7a1b6d3199d6
-- 
2.34.0

Re: [PATCH] lei: Delete message buffer's window along with query window

[Xinglu Chen]

[-- Attachment #1: Type: text/plain, Size: 680 bytes --]

Kyle schrieb am Dienstag der 28. Dezember 2021 um 18:11 -05:

> Within query buffers, a message for the current line can be displayed
> while keeping point in the query buffer.  Calling quit-window in a
> query buffer, however, leaves the associated buffer's window in place,
> which is probably not what the caller wants.  Add a custom quit-window
> variant that handles the message buffer too.
>
> Suggested-by: Xinglu Chen <public@yoctocell.xyz>
> Link: https://inbox.kyleam.com/piem/871r96am1q.fsf@yoctocell.xyz
> ---
>  piem-lei.el | 11 +++++++++++
>  1 file changed, 11 insertions(+)

I saw that you already pushed it, but just wanted to say that it works
as expected.  :-)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 861 bytes --]