Re: [RP] [PATCH] Make focus* commands screen aware.

[Joseph Mingrone]

Pedro Silva <address@hidden> writes:

> focusright, focusleft, focusup, and focusdown previously worked solely
> within the frame's screen, because their underlying handling logic in
> split.c strictly did not consider their screen offsets.

> This patch introduces 4 new functions, frame_*_abs, which instead report
> the frame's left, right, top, and bottom coordinates offset by it's
> screen arrangement. This allows the use of the focus* commands as
> additional screen navigation tools, and has the added benefit of making
> the find_frame_* functions clearer.
> ---
>  doc/ratpoison.texi |  4 +++-
>  src/frame.c        | 27 ++++++++++++++++++++++++
>  src/frame.h        |  4 ++++
>  src/split.c        | 62 
> +++++++++++++++++++++++++++++++++++-------------------
>  4 files changed, 74 insertions(+), 23 deletions(-)

This is a nice idea.  It makes ratpoison behave similar to StumpWM with respect 
to multiple screens, which I find more intuitive.

I've attached an updated patch, which applies against the xrandr branch.

0001-Update-screen-aware-focus-commands.patch
Description: Updated screen-aware focus* patch

signature.asc
Description: PGP signature