GNU bug report logs - #69271
30.0.50; Strange interaction between :inverse-video overlay face and calendar today face

Previous Next

Full log

Message #8 received at 69271 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ihor Radchenko <yantar92 <at> posteo.net>
Cc: 69271 <at> debbugs.gnu.org
Subject: Re: bug#69271: 30.0.50;
 Strange interaction between :inverse-video overlay face and calendar
 today face
Date: Tue, 20 Feb 2024 18:08:14 +0200

> From: Ihor Radchenko <yantar92 <at> posteo.net>
> Date: Mon, 19 Feb 2024 12:14:28 +0000
> 
> 1. emacs -Q
> 2. Insert the following code into *scratch* buffer
> 
> 
> (defface yant/test '((t :foreground "red" :weight bold)) "")
> (defface yant/test-inv '((t :foreground "red"  :inverse-video t)) "")
> (require 'calendar)
> 
> ;; (dotimes (_ 5)
> ;;   (let* ((mark-calendar
> ;; 	  (lambda ()
> ;; 	    (calendar-mark-visible-date
> ;; 	     (read (format-time-string "(%m %d %Y)"))
> ;; 	     'yant/test)))
> ;; 	 (calendar-today-visible-hook))
> ;;     (add-hook 'calendar-today-visible-hook mark-calendar)
> ;;     (calendar)
> ;;     (setq cursor-type nil)
> ;;     (let ((ov (make-overlay (1- (point)) (1+ (point)))))
> ;;       (overlay-put ov 'face 'yant/test-inv)))
> ;;   (read-char "Observe overlay obeying inverse-video. Press any key")
> ;;   (kill-buffer calendar-buffer))
> (dotimes (_ 50)
>   (let* ((mark-calendar
> 	  (lambda ()
> 	    (calendar-mark-visible-date
> 	     (read (format-time-string "(%m %d %Y)"))
> 	     (list :foreground "red" :weight 'bold))))
> 	 (calendar-today-visible-hook))
>     (add-hook 'calendar-today-visible-hook mark-calendar)
>     (calendar)
>     (setq cursor-type nil)
>     (let ((ov (make-overlay (1- (point)) (1+ (point)))))
>       (overlay-put ov 'face 'yant/test-inv)))
>   (read-char "Observe overlay sometimes *not* obeying inverse-video. Press any key")
>   (kill-buffer calendar-buffer))
> 
> 3. M-x eval-buffer <RET>
> 4. Observe today date in the calendar fontified inconsistently as the
>    code is executed multiple times.

This code places two overlays on the same text, with each overlay
defining a foreground color.  So which one "wins" is basically random,
unless you give each overlay a priority to make that deterministic.

I tried a simpler recipe:

  emacs -Q
  M-x load-library RET calendar RET
  C-x b foo RET
  C-u 30 a RET
  M-: (defface yant/test-inv '((t :foreground "red"  :inverse-video t)) "") RET
  M-: (setq ov1 (make-overlay 24 26)) RET
  M-: (overlay-put ov1 'face (calendar-make-temp-face (list :foreground "red" :weight 'bold))) RET
  M-: (setq ov2 (make-overlay 24 26)) RET
  M-: (overlay-put ov2 'face 'yant/test-inv) RET

This produced text with red background, according to the yant/test-inv
face.  But if you define the overlays in the opposite order, you get
the red bold foreground instead.  If you now give each overlay a
priority, you can control which face shows: the one whose overlay has
the higher priority.

Maybe I'm missing something, but I see no bug here.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.