bug#30207: 27.0.50; [PATCH] other-window-for-scrolling returns window on daemon frame

["Basil L. Contovounesios" <contovob@tcd.ie>]

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

Eli Zaretskii <eliz@gnu.org> writes:

>> From: "Basil L. Contovounesios" <contovob@tcd.ie>
>> Date: Wed, 02 May 2018 19:57:59 +0100
>> Cc: Noam Postavsky <npostavs@users.sourceforge.net>
>> 
>> +@kindex C-M-S-v
>> +@findex scroll-other-window-down
>>    The usual scrolling commands (@pxref{Display}) apply to the selected
>> -window only, but there is one command to scroll the next window.
>> +window only, but there are two commands to scroll the next window.
>                     ^^^^^^^^^^^^^^^^^^^^^^
> I'd just say "there are commands".

That's much better, but the sentence still seems a bit abrupt to me.
How about "there are also commands"?

>>  @kbd{C-M-v} (@code{scroll-other-window}) scrolls the window that
>> -@kbd{C-x o} would select.  It takes arguments, positive and negative,
>> -like @kbd{C-v}.  (In the minibuffer, @kbd{C-M-v} scrolls the help
>> -window associated with the minibuffer, if any, rather than the next
>> -window in the standard cyclic order; @pxref{Minibuffer Edit}.)
>> +@kbd{C-x o} would select upward.
>                      ^^^^^^^^^^^^^
> This makes the sentence ambiguous: it is not clear whether "upward"
> goes with "scroll" or with "select".  Either say "scrolls upward" at
> the beginning of the sentence, or maybe explain the "upward" part in a
> separate sentence (and what does "upward" mean here anyway -- does the
> text or the viewport move upward?).

Good point.  Given that (a) forward/upward and backward/downward
scrolling is described in more detail under the indirectly referenced
"(emacs) Scrolling", and (b) half of this paragraph is already
parenthetical, could we get away with just minor disambiguation?
For example:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: other-window.patch --]
[-- Type: text/x-diff, Size: 1407 bytes --]

diff --git a/doc/emacs/windows.texi b/doc/emacs/windows.texi
index 59b3b65f65..17ce4ad04d 100644
--- a/doc/emacs/windows.texi
+++ b/doc/emacs/windows.texi
@@ -183,13 +183,18 @@ Other Window
 
 @kindex C-M-v
 @findex scroll-other-window
+@kindex C-M-S-v
+@findex scroll-other-window-down
   The usual scrolling commands (@pxref{Display}) apply to the selected
-window only, but there is one command to scroll the next window.
+window only, but there are also commands to scroll the next window.
 @kbd{C-M-v} (@code{scroll-other-window}) scrolls the window that
-@kbd{C-x o} would select.  It takes arguments, positive and negative,
-like @kbd{C-v}.  (In the minibuffer, @kbd{C-M-v} scrolls the help
-window associated with the minibuffer, if any, rather than the next
-window in the standard cyclic order; @pxref{Minibuffer Edit}.)
+@kbd{C-x o} would select.  In other respects, the command behaves like
+@kbd{C-v}; both move the buffer text upward relative to the window, and
+take positive and negative arguments.  (In the minibuffer, @kbd{C-M-v}
+scrolls the help window associated with the minibuffer, if any, rather
+than the next window in the standard cyclic order; @pxref{Minibuffer
+Edit}.)  @kbd{C-M-S-v} (@code{scroll-other-window-down}) scrolls the
+next window downward in a similar way.
 
 @vindex mouse-autoselect-window
   If you set @code{mouse-autoselect-window} to a non-@code{nil} value,

[-- Attachment #3: Type: text/plain, Size: 103 bytes --]


[As you can see, I'm not particularly confident of my taste in technical
 prose.]

Thanks,

-- 
Basil