greg(a)alphatech.com (Greg Klanderman) writes:
Are we agreed that with f-f-m true, C-x 5 o (other-frame) is the
*only* command which should warp focus.
Yes. But this is still a major behavioral change which you should
document. For instance, it may be worth noting (in the manual) that
if you want `select-frame' to actually select the WM frame, you must
bind focus-follows-mouse to nil.
Other than the `C-x 5 o' problem, I must once again state that your
patch does The Right Thing. For instance,
(progn (select-frame (other-frame)) (insert "foo")) inserts "foo" in
the other buffer's frame, which is correct, and then reselects this
frame for focus. Kudos to you. :-)
Is it ever called non-interactively? (eg, should it check and only
actually change focus when run interactively?)
Good question. I think it should always and definitely change focus,
as per documentation:
C-x 5 o runs `other-frame'
Documentation:
Select the ARG'th different visible frame, and raise it.
All frames are arranged in a cyclic order.
This command selects the frame ARG steps away in that order.
A negative ARG moves in the opposite order.
--
Hrvoje Niksic <hniksic(a)srce.hr> | Student at FER Zagreb, Croatia
--------------------------------+--------------------------------
* Q: What is an experienced Emacs user?
* A: A person who wishes that the terminal had pedals.