Re: patch: push window to frame

I think I understand, now, that the man page and the info doc are
maintained separately. The attached patch adds documentation about the new
pushwindow command.


Jeff Abrahamson
+33 6 24 40 01 57
+44 7920 594 255    <-- only if I'm in the UK

http://jeff.purple.com/
http://blog.purple.com/jeff/


On 6 September 2014 22:25, Jeff Abrahamson <[email protected]> wrote:

> I've implemented the command pushwindow, which moves the current window to
> another frame, leaving the focus in the current frame. This is what I
> originally called "send to frame" in this bug
> <https://savannah.nongnu.org/bugs/?42223> and when I proposed to
> implement this.
>
> I've bound the command to "h", which is the last letter of "push". I'd
> like to implement "pullwindow" the same way, and I'd like to bind it to "l"
> (lower case "L"). This is currently bound, along with C-L, to redisplay, so
> maybe that's a bad idea (changing the binding). I think the bindings for
> push and pull should be thought through together.
>
> Feedback on the code and on the naming/binding most welcome, of course.
>
> I don't see how to make one change to the documentation so that both the
> man page and the texinfo documentation both change. So maybe patch 2 is
> wrong.
>
> Jeff Abrahamson
> +33 6 24 40 01 57
> +44 7920 594 255    <-- only if I'm in the UK
>
> http://jeff.purple.com/
> http://blog.purple.com/jeff/
>

_______________________________________________
Ratpoison-devel mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/ratpoison-devel

From 083a9a7c6c9cd59212acd3857dc0ef28e3545905 Mon Sep 17 00:00:00 2001
From: Jeff Abrahamson <[email protected]>
Date: Sun, 7 Sep 2014 19:53:57 +0200
Subject: [PATCH 3/3] Add documentation for command pushwindow.

---
 doc/ratpoison.texi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/ratpoison.texi b/doc/ratpoison.texi
index d12501f..9af7509 100644
--- a/doc/ratpoison.texi
+++ b/doc/ratpoison.texi
@@ -270,6 +270,12 @@ This toggles between the current window and the last window. By
 default, this is bound to @kbd{C-t C-t}.
 @end deffn
 
+@deffn Command pushwindow
+Move the current window to a different frame, leaving the focus
+on the current frame. Prompts for the destination frame.
+By default, this is bound to @kbd{C-t h}.
+@end deffn
+
 @deffn Command prev
 This jumps you to the previous window in the window list. By default,
 this is bound to @kbd{C-t p}.
-- 
1.9.1