GNU bug report logs - #40990
Improve message-mode and isearch icons

Previous Next

Full log

View this message in rfc822 format

From: Dmitry Gutov <dgutov <at> yandex.ru>
To: 40990 <at> debbugs.gnu.org
Subject: bug#40990: Improve message-mode and isearch icons
Date: Fri, 1 May 2020 02:50:04 +0300

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

As discussed previously in the "Why is emacs so square?" thread, here's
a couple of patches that improve the toolbar icons in GTK builds.

Attached are several screenshots from my machine: isearch toolbar with
the x-gtk-stock-map patch and without (the difference is one icon), and
message-mode toolbar without the patch, with the patch, and also with
the extra patch that makes message-tool-bar-retro use the standrd
"mail/send" icon instead of the Gnus-specific one. Fixing Emacs not to
use message-tool-bar-retro is left as a future exercise.

I'd like to present a few arguments for the inclusion of the
x-gtk-stock-map patch in Emacs 27 (on which the current answer is no,
but I'll try to elaborate):

1. With it, the answer to the question "why does everybody tell the new
users to turn off toolbars" becomes less obvious, at least in GTK based
environments.

2. This only affects GTK. And also, if a theme doesn't contain the "new"
replacement image, the previous one will be shown instead.

3. Since "everybody" turns off the toolbars now, the risk of complaints
is pretty low. By the same token, we could de-prioritize said complaints
even if they appear, given that for most people the improvement is most
likely obvious (feedback welcome).

The second patch is a tiny bit more risky, but honestly, since it uses
an icon apparently borrowed from some old GTK-related icon set, using a
new version of the same icon shouldn't bother most people. But this
patch *can* affect users on different toolkits and different platforms
(again, feedback welcome).

[message-tool-bar-retro.diff (text/x-patch, attachment)]

[x-gtk-stock-map.diff (text/x-patch, attachment)]

[isearch-icons-new.png (image/png, attachment)]

[isearch-icons-old.png (image/png, attachment)]

[message-mode-icons-new.png (image/png, attachment)]

[message-mode-icons-new-with-extra-patch.png (image/png, attachment)]

[message-mode-icons-old.png (image/png, attachment)]

Previous Next

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