GNU bug report logs - #16574
24.3.50; icon not shown on the splash screen (mac osx)

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 16574 in the body.
You can then email your comments to 16574 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Darren Hoo <darren.hoo <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Tue, 28 Jan 2014 10:49:07 +0800

[Message part 1 (text/plain, inline)]

open /Applications/Emacs.app --args -q

the emacs icon on the fancy splash screen (*GNU Emacs*) is missing, try to
increase the frame size a little bit then press `c-h c-a', the icon is back,
however if decrease the frame size to some size less or equal to the initial
startup size and press `c-h c-a' the icon is lost again.



In GNU Emacs 24.3.50.79 (x86_64-apple-darwin13.0.0, NS apple-appkit-1265.00)
 of 2014-01-28 on darren-rmbp
Repository revision: 116181 rgm <at> gnu.org-20140128014902-1z6jns5bv3qnug09
Windowing system distributor `Apple', version 10.3.1265
Configured using:
 `configure --with-ns'

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Fundamental

Minor modes in effect:
  tooltip-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  buffer-read-only: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
C-h C-a <help-echo> C-h C-a <help-echo> <down-mouse-1>
<mouse-1> C-h C-a <wheel-up> <double-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <help-echo> <wheel-up> <double-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> <triple-wheel-up>
<triple-wheel-up> <triple-wheel-up> M-x r e p o r t
<tab> <return>

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
byte-code: Beginning of buffer [38 times]

Load-path shadows:
None found.

Features:
(shadow sort gnus-util mail-extr emacsbug message format-spec rfc822 mml
easymenu mml-sec mm-decode mm-bodies mm-encode mail-parse rfc2231
mailabbrev gmm-utils mailheader sendmail rfc2047 rfc2045 ietf-drums
mm-util help-fns mail-prsvr mail-utils time-date tooltip electric
uniquify ediff-hook vc-hooks lisp-float-type mwheel ns-win tool-bar dnd
fontset image regexp-opt fringe tabulated-list newcomment lisp-mode
prog-mode register page menu-bar rfn-eshadow timer select scroll-bar
mouse jit-lock font-lock syntax facemenu font-core frame cham georgian
utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao korean
japanese hebrew greek romanian slovak czech european ethiopic indian
cyrillic chinese case-table epa-hook jka-cmpr-hook help simple abbrev
minibuffer nadvice loaddefs button faces cus-face macroexp files
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote make-network-process
cocoa ns multi-tty emacs)

[Message part 2 (text/html, inline)]

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

From: Glenn Morris <rgm <at> gnu.org>
To: Darren Hoo <darren.hoo <at> gmail.com>
Cc: 16574 <at> debbugs.gnu.org
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Mon, 27 Jan 2014 21:54:02 -0500

Darren Hoo wrote:

> open /Applications/Emacs.app --args -q
>
> the emacs icon on the fancy splash screen (*GNU Emacs*) is missing, try to
> increase the frame size a little bit then press `c-h c-a', the icon is back,
> however if decrease the frame size to some size less or equal to the initial
> startup size and press `c-h c-a' the icon is lost again.

I don't think this is a bug, and AFAIK it has been like this for some time.
The image is not shown if the frame seems to be too small.
See use-fancy-splash-screens-p.

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

From: Darren Hoo <darren.hoo <at> gmail.com>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 16574 <at> debbugs.gnu.org
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Tue, 28 Jan 2014 12:54:46 +0800

[Message part 1 (text/plain, inline)]

On Tue, Jan 28, 2014 at 10:54 AM, Glenn Morris <rgm <at> gnu.org> wrote:

> Darren Hoo wrote:
>
> > open /Applications/Emacs.app --args -q
> >
> > the emacs icon on the fancy splash screen (*GNU Emacs*) is missing, try
> to
> > increase the frame size a little bit then press `c-h c-a', the icon is
> back,
> > however if decrease the frame size to some size less or equal to the
> initial
> > startup size and press `c-h c-a' the icon is lost again.
>
> I don't think this is a bug, and AFAIK it has been like this for some time.
> The image is not shown if the frame seems to be too small.
> See use-fancy-splash-screens-p.
>

OK, I see.
Though use-fancy-splash-screens-p is not strictly correct on this.
If  I force it to use splash screen:

(defun use-fancy-splash-screens-p () t)

And examine these values of the splash screen on the initial startup
frame:

(window-pixel-height)
 => 476

(cdr (window-text-pixel-size
      (selected-window) nil nil nil nil t)) ; include mode-line
 => 462

it has plenty room for display.

[Message part 2 (text/html, inline)]

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Darren Hoo <darren.hoo <at> gmail.com>
Cc: 16574 <at> debbugs.gnu.org, rgm <at> gnu.org
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Tue, 28 Jan 2014 17:50:47 +0200

> Date: Tue, 28 Jan 2014 12:54:46 +0800
> From: Darren Hoo <darren.hoo <at> gmail.com>
> Cc: 16574 <at> debbugs.gnu.org
> 
> > I don't think this is a bug, and AFAIK it has been like this for some time.
> > The image is not shown if the frame seems to be too small.
> > See use-fancy-splash-screens-p.
> >
> 
> OK, I see.
> Though use-fancy-splash-screens-p is not strictly correct on this.
> If  I force it to use splash screen:
> 
> (defun use-fancy-splash-screens-p () t)
> 
> And examine these values of the splash screen on the initial startup
> frame:
> 
> (window-pixel-height)
>  => 476
> 
> (cdr (window-text-pixel-size
>       (selected-window) nil nil nil nil t)) ; include mode-line
>  => 462
> 
> it has plenty room for display.

How do you deduce that this is enough?  splash.xpm needs 217 pixels on
my system, and use-fancy-splash-screens-p reserves 19 more lines for
text, which on my system need 19 x 16 = 304 more pixels.  The sum is
greater than either 462 or 476, so it sounds to me like
use-fancy-splash-screens-p is right.  (On my system, the window is 528
pixel high, which is indeed greater than 217 + 304.)

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

From: Darren Hoo <darren.hoo <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 16574 <at> debbugs.gnu.org, Glenn Morris <rgm <at> gnu.org>
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Wed, 29 Jan 2014 07:52:11 +0800

[Message part 1 (text/plain, inline)]

On Tue, Jan 28, 2014 at 11:50 PM, Eli Zaretskii <eliz <at> gnu.org> wrote:

> > Date: Tue, 28 Jan 2014 12:54:46 +0800
> > From: Darren Hoo <darren.hoo <at> gmail.com>
> > Cc: 16574 <at> debbugs.gnu.org
> >
> > > I don't think this is a bug, and AFAIK it has been like this for some
> time.
> > > The image is not shown if the frame seems to be too small.
> > > See use-fancy-splash-screens-p.
> > >
> >
> > OK, I see.
> > Though use-fancy-splash-screens-p is not strictly correct on this.
> > If  I force it to use splash screen:
> >
> > (defun use-fancy-splash-screens-p () t)
> >
> > And examine these values of the splash screen on the initial startup
> > frame:
> >
> > (window-pixel-height)
> >  => 476
> >
> > (cdr (window-text-pixel-size
> >       (selected-window) nil nil nil nil t)) ; include mode-line
> >  => 462
> >
> > it has plenty room for display.
>
> How do you deduce that this is enough?


I can see that on my system the splash screen fits the frame quite well
in one page.

window-text-pixel-size (462) above returns the height of the text
displayed,
that counts the height of the splash image and the window-pixel-height
is 476 > 462, both inludes the height of the modeline.


> splash.xpm needs 217 pixels on
> my system, and use-fancy-splash-screens-p reserves 19 more lines for
> text, which on my system need 19 x 16 = 304 more pixels.  The sum is
> greater than either 462 or 476, so it sounds to me like


Which  splash file on your system is actually used for display?

The problem is that use-fancy-splash-screens-p uses splash.xpm to
determine the image height which  is 217 pixel high, while on my system
splash.png (188 pixel high) is actually  used (since it's more fancier
than the xpm one).

I think it's better for  use-fancy-splash-screens-p and fancy-splash-head
to use  the same file:

=== modified file 'lisp/startup.el'

--- lisp/startup.el 2014-01-01 07:43:34 +0000

+++ lisp/startup.el 2014-01-28 23:09:37 +0000

@@ -1590,24 +1590,28 @@



 (declare-function image-size "image.c" (spec &optional pixels frame))



+

+(defun fancy-splash-file ()

+  (cond ((stringp fancy-splash-image)

+ fancy-splash-image)

+ ((display-color-p)

+ (cond ((<= (display-planes) 8)

+ (if (image-type-available-p 'xpm)

+     "splash.xpm"

+   "splash.pbm"))

+       ((or (image-type-available-p 'svg)

+     (image-type-available-p 'imagemagick))

+ "splash.svg")

+       ((image-type-available-p 'png)

+ "splash.png")

+       ((image-type-available-p 'xpm)

+ "splash.xpm")

+       (t "splash.pbm")))

+ (t "splash.pbm")))

+

 (defun fancy-splash-head ()

   "Insert the head part of the splash screen into the current buffer."

-  (let* ((image-file (cond ((stringp fancy-splash-image)

-     fancy-splash-image)

-   ((display-color-p)

-     (cond ((<= (display-planes) 8)

-   (if (image-type-available-p 'xpm)

-       "splash.xpm"

-     "splash.pbm"))

-   ((or (image-type-available-p 'svg)

-       (image-type-available-p 'imagemagick))

-   "splash.svg")

-   ((image-type-available-p 'png)

-   "splash.png")

-   ((image-type-available-p 'xpm)

-   "splash.xpm")

-   (t "splash.pbm")))

-   (t "splash.pbm")))

+  (let* ((image-file (fancy-splash-file))

  (img (create-image image-file))

  (image-width (and img (car (image-size img))))

  (window-width (window-width)))

@@ -1805,22 +1809,16 @@



 (defun use-fancy-splash-screens-p ()

   "Return t if fancy splash screens should be used."

-  (when (and (display-graphic-p)

-             (or (and (display-color-p)

-       (image-type-available-p 'xpm))

-                 (image-type-available-p 'pbm)))

-    (let ((frame (fancy-splash-frame)))

-      (when frame

- (let* ((img (create-image (or fancy-splash-image

-       (if (and (display-color-p)

-       (image-type-available-p 'xpm))

-   "splash.xpm" "splash.pbm"))))

-       (image-height (and img (cdr (image-size img nil frame))))

-       ;; We test frame-height so that, if the frame is split

-       ;; by displaying a warning, that doesn't cause the normal

-       ;; splash screen to be used.

-       (frame-height (1- (frame-height frame))))

-   (> frame-height (+ image-height 19)))))))

+  (and (display-graphic-p)

+       (let ((frame (fancy-splash-frame)))

+ (when frame

+   (let* ((img (create-image (fancy-splash-file)))

+   (image-height (and img (cdr (image-size img nil frame))))

+   ;; We test frame-height so that, if the frame is split

+   ;; by displaying a warning, that doesn't cause the normal

+   ;; splash screen to be used.

+   (frame-height (1- (frame-height frame))))

+     (> frame-height (+ image-height 19)))))))





 (defun normal-splash-screen (&optional startup concise)

[Message part 2 (text/html, inline)]

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Darren Hoo <darren.hoo <at> gmail.com>
Cc: 16574 <at> debbugs.gnu.org, rgm <at> gnu.org
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Wed, 29 Jan 2014 05:45:00 +0200

> Date: Wed, 29 Jan 2014 07:52:11 +0800
> From: Darren Hoo <darren.hoo <at> gmail.com>
> Cc: Glenn Morris <rgm <at> gnu.org>, 16574 <at> debbugs.gnu.org
> 
> The problem is that use-fancy-splash-screens-p uses splash.xpm to
> determine the image height which  is 217 pixel high, while on my system
> splash.png (188 pixel high) is actually  used (since it's more fancier
> than the xpm one).

Ah, that was important information.  Yes, I guess
use-fancy-splash-screens-p should do the same as fancy-splash-head to
determine which image will be displayed.

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

From: Glenn Morris <rgm <at> gnu.org>
To: Darren Hoo <darren.hoo <at> gmail.com>
Cc: 16574 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Thu, 30 Jan 2014 21:01:47 -0500

Thank, applied.

The whitespace in your patch was severely messed up, to the extent of
extra blank lines all over the place. Please try to figure out why that
happened to avoid it in future.


This raises the question of why some of the versions of the splash image
have slightly different appearance / image dimensions.

png and svg are (almost) the same, as are pbm and xpm.

I did "convert -background transparent splash.png splash2.xpm", and the
result looked better to my untrained eye than the current splash.xpm
does, apart from some edge effects.

Perhaps someone would like to try re-making the xpm and pbm versions
so that they are closer to the other versions.

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

From: Darren Hoo <darren.hoo <at> gmail.com>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 16574 <at> debbugs.gnu.org
Subject: Re: bug#16574: 24.3.50; icon not shown on the splash screen (mac osx)
Date: Fri, 31 Jan 2014 11:25:30 +0800

Glenn Morris <rgm <at> gnu.org> writes:

>
> The whitespace in your patch was severely messed up, to the extent of
> extra blank lines all over the place. Please try to figure out why that
> happened to avoid it in future.
>

Sorry for that. Most of the time I just use Gnus, I used gmail web
interface once and it just screwed me up.

Previous Next

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