TAB scrolls _not_ the quail choices if used in the minibuffer

TAB scrolls _not_ the quail choices if used in the minibuffer

[Dan Jacobson]

I have 2 different windows on my screen.  I do
C-x C-f C-\ g u <tab> <tab> <tab>
And instead of scrolling the quail choices, it scrolls the wrong buffer.
-- 
http://jidanni.org/ Taiwan(04)25854780

Re: TAB scrolls _not_ the quail choices if used in the minibuffer

[Dan Jacobson]

>>>>> "RMS" == Richard Stallman <rms@gnu.org> writes:

>     I have 2 different windows on my screen.  I do
>     C-x C-f C-\ g u <tab> <tab> <tab>
>     And instead of scrolling the quail choices, it scrolls the wrong buffer.

RMS> This is not a complete test case.  Would you please send a complete one?

$ emacs [using http://jidanni.org/comp/emacs.txt as .emacs]
C-x C-f C-\ b <tab> <tab> #works nicely
C-x 2 C-x C-f C-\ b <tab> <tab> <tab> <tab> <tab> #does not scroll
quail choices properly...  do also test with a buffer longer than *scratch*.
-- 
http://jidanni.org/ Taiwan(04)25854780

Re: TAB scrolls _not_ the quail choices if used in the minibuffer

[Kenichi Handa]

In article <87n0qd7c62.fsf@jidanni.org>, Dan Jacobson <jidanni@dman.ddts.net> writes:
> $ emacs [using http://jidanni.org/comp/emacs.txt as .emacs]
> C-x C-f C-\ b <tab> <tab> #works nicely
> C-x 2 C-x C-f C-\ b <tab> <tab> <tab> <tab> <tab> #does not scroll
> quail choices properly...  do also test with a buffer longer than *scratch*.

Thank you for the report.  I've just installed the
attached fix.

---
handa@m17n.org

2002-09-23  Kenichi Handa  <handa@etl.go.jp>

	* international/quail.el (quail-completion): Be sure to scroll
	quail-completion-buf.

*** quail.el.~1.120.~	Thu Jun 27 22:22:01 2002
--- quail.el	Mon Sep 23 11:35:29 2002
***************
*** 2124,2130 ****
  	  ;; shown.  We just scroll it appropriately.
  	  (if (pos-visible-in-window-p (point-max) win)
  	      (set-window-start win (point-min))
! 	    (let ((other-window-scroll-buffer quail-completion-buf))
  	      (scroll-other-window)))
  	(setq quail-current-key key)
  	(erase-buffer)
--- 2124,2133 ----
  	  ;; shown.  We just scroll it appropriately.
  	  (if (pos-visible-in-window-p (point-max) win)
  	      (set-window-start win (point-min))
! 	    (let ((other-window-scroll-buffer quail-completion-buf)
! 		  ;; This nil binding is necessary to surely scroll
! 		  ;; quail-completion-buf.
! 		  (minibuffer-scroll-window nil))
  	      (scroll-other-window)))
  	(setq quail-current-key key)
  	(erase-buffer)