unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Appt Msg Window problem
@ 2004-05-10 21:07 Tak Ota
  2004-05-12  0:35 ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Tak Ota @ 2004-05-10 21:07 UTC (permalink / raw)


It seems appointment message window doesn't display correctly.  Try
evaluating (appt-disp-window "5" "3" "Test") and see the window
splits overly.  Following patch fixes this problem.

BTW, `split-window' doc string does not explain about its return
value.

-Tak

*** ../../../../pub/emacs/pure/emacs-21.3.50/lisp/calendar/appt.el	Wed Mar 24 10:36:20 2004
--- appt.el	Mon May 10 13:51:21 2004
***************
*** 449,456 ****
  		  (same-window-p (buffer-name appt-disp-buf)))
  	;; By default, split the bottom window and use the lower part.
  	(appt-select-lowest-window)
! 	(split-window))
!       (pop-to-buffer appt-disp-buf))
      (setq mode-line-format
  	  (concat "-------------------- Appointment in "
  		  min-to-app " minutes. " new-time " %-"))
--- 449,456 ----
  		  (same-window-p (buffer-name appt-disp-buf)))
  	;; By default, split the bottom window and use the lower part.
  	(appt-select-lowest-window)
! 	(select-window (split-window)))
!       (switch-to-buffer appt-disp-buf))
      (setq mode-line-format
  	  (concat "-------------------- Appointment in "
  		  min-to-app " minutes. " new-time " %-"))

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Appt Msg Window problem
  2004-05-10 21:07 Appt Msg Window problem Tak Ota
@ 2004-05-12  0:35 ` Glenn Morris
  2004-05-12  1:12   ` Tak Ota
  0 siblings, 1 reply; 4+ messages in thread
From: Glenn Morris @ 2004-05-12  0:35 UTC (permalink / raw)
  Cc: emacs-devel

Tak Ota wrote:

> It seems appointment message window doesn't display correctly.  Try
> evaluating (appt-disp-window "5" "3" "Test") and see the window
> splits overly. 

I evaluated that expression and the results looks fine to me. What
should I be seeing?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Appt Msg Window problem
  2004-05-12  0:35 ` Glenn Morris
@ 2004-05-12  1:12   ` Tak Ota
  2004-05-13  0:36     ` Glenn Morris
  0 siblings, 1 reply; 4+ messages in thread
From: Tak Ota @ 2004-05-12  1:12 UTC (permalink / raw)
  Cc: emacs-devel

Wed, 12 May 2004 01:35:44 +0100: Glenn Morris <gmorris+emacs@ast.cam.ac.uk> wrote:

> Tak Ota wrote:
> 
> > It seems appointment message window doesn't display correctly.  Try
> > evaluating (appt-disp-window "5" "3" "Test") and see the window
> > splits overly. 
> 
> I evaluated that expression and the results looks fine to me. What
> should I be seeing?

Please try that again after (setrq split-height-threshold 8).  The doc
string of split-height-threshold reads mysterious.

-Tak

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Appt Msg Window problem
  2004-05-12  1:12   ` Tak Ota
@ 2004-05-13  0:36     ` Glenn Morris
  0 siblings, 0 replies; 4+ messages in thread
From: Glenn Morris @ 2004-05-13  0:36 UTC (permalink / raw)
  Cc: emacs-devel

Tak Ota wrote:

> Please try that again after (setrq split-height-threshold 8). The
> doc string of split-height-threshold reads mysterious.

Ah, I see now. I installed your change - thanks. I also tried to
improve the docs for display-buffer, split-window, and
split-height-threshold; though others who know more about their
workings might like to correct me.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2004-05-13  0:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-05-10 21:07 Appt Msg Window problem Tak Ota
2004-05-12  0:35 ` Glenn Morris
2004-05-12  1:12   ` Tak Ota
2004-05-13  0:36     ` Glenn Morris

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).