all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Yuan Fu <casouri@gmail.com>
To: Po Lu <luangruo@yahoo.com>
Cc: "Ergus via Emacs development discussions." <emacs-devel@gnu.org>,
	Stefan Kangas <stefankangas@gmail.com>
Subject: Re: Add Tango and Adwaita icons for the toolbar
Date: Sat, 4 May 2024 09:42:01 -0700	[thread overview]
Message-ID: <7C9CBDCA-F058-46D4-9E40-63A18ABE34A7@gmail.com> (raw)
In-Reply-To: <87plu19ac7.fsf@yahoo.com>



> On May 4, 2024, at 2:48 AM, Po Lu <luangruo@yahoo.com> wrote:
> 
> Yuan Fu <casouri@gmail.com> writes:
> 
>> Hey guys,
>> 
>> I followed the original tool-bar icon thread in 2022. At the time I
>> didn’t have much time and I personally prefer colorful icons for the
>> tool bar [1] (IMO mono-colored symbolic icons are more suitable for
>> inline, small icons), so I didn’t do anything except for observing the
>> discussion.
>> 
>> [1] I especially adore the life ring icon for help! Whatever icon set
>> we use for the tool-bar, the help icon should always be a cute life
>> ring :-)
>> 
>> The recent Emacs app icon thread reminded me of that thread, and I do
>> have time now. So, I went ahead and searched for the Tango and Adwaita
>> icons (basically the high-res version of the existing tool-bar icon
>> that we use), then converted them to be suitable for Emacs tool-bar.
>> 
>> About 1/3 of the icons can be directly taken from Tango or Adwaita,
>> the rest I have to modify or kind of take apart and
>> re-assemble. Anyway, in the end I managed to get most of the basic
>> icons covered. For details see etc/images/scalable/README.
>> 
>> You can see the attached screenshots for the current built-in icons
>> and the new icons. I also attached the patch for it. You want to
>> rebuild or byte-compile tool-bar.el for the new icons to take effect.
>> 
>> I hope these new icons won’t be controversial. As you can see, they
>> are basically straight high-res upgrades for the existing icons. There
>> are many imperfections that’ll be atrocious in the eyes of a graphic
>> designer—I did the conversion work in an afternoon and I’m no where
>> near a graphic designer; but it’s definitely an upgrade and we can
>> always incrementally improve after.
> 
> They will not be controversial if it is possible to opt out of them, and
> enough of the icons are replaced that none of us encounter tool-bars
> bearing an unsightly mixture of old and new.  That is to say:
> 
> Please generate XPM and monochrome PBM versions of the new icons
> introduced, these being the only two image formats implemented without
> dependencies in image.c, and enable SVG only on systems where the
> scalable versions produce a visible improvement to image quality.
> Finally, please see that _all_ major modes are covered by this new icon
> set, including but not limited to message-mode, Doc View,
> xwidget-webkit, and anywhere else a custom tool-bar-map is defined, and
> that there is means of reverting to the old set of icons, for users who
> are so inclined.

I don’t think it’s necessary to generate XPM and BPM versions, because if you compare the new icons with the old, you’ll find that they are essentially the same icon (same shape, motif, etc). That is to say, we already have XPM and BPM version of the icons: the current ones.

Also the icons are in png format, the source is svg, but I exported the svg into png. Because svg isn’t rendered very well, presumably because theses icons are relatively complicated? (More complicated than the symbolic icons for sure.)

As for opting-out, there isn’t a mechanism to explicitly choose what version of icons to use in tool-bar.el right now. So it’ll need to be added. I believe the branch mentioned in the original discussion, created by Stefan K, has some mechanism for an “icon theme”. Maybe we can merge that, plus the new monotone icons in that branch, but left out changing the tool-bar icons?

It’ll be a big project to find counterparts for all the current icons, plus I’m not a designer so I can’t conjure up icons by myself :-) I think we could make it opt-in first, and make it default when we have full coverage.

> The PBM icons should not be unmodified ImageMagick output, they should
> be modified for legibility on actual monochrome displays.  The other day
> there was a post to help-gnu-emacs from an Android user with a
> monochrome display, so these configurations are by no means irrelevant.
> 
>> If no one objects, I’ll happily push this to master.
> 
> Nobody objects to these icons in principle, but do not push such changes
> until they are complete and have been tested by an adequately wide
> audience.  Let it not be forgotten that we are still contending with the
> fallout from another enormously premature change.

Yuan


  reply	other threads:[~2024-05-04 16:42 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <EDDF7D6A-E818-483A-AA98-CB2D093BCDAC@gmail.com>
2024-05-04  9:48 ` Add Tango and Adwaita icons for the toolbar Po Lu
2024-05-04 16:42   ` Yuan Fu [this message]
2024-05-05  0:22     ` Po Lu
2024-05-05  3:04       ` Yuan Fu
2024-05-05  4:14         ` Po Lu
2024-05-05  6:59           ` Yuan Fu
2024-05-05  6:01         ` Eli Zaretskii
2024-05-05  6:39           ` Po Lu
2024-05-05  5:55       ` Eli Zaretskii
2024-05-04 12:27 ` Eli Zaretskii
2024-05-04 21:27   ` Yuan Fu
2024-05-05  0:08     ` Po Lu
2024-05-05 15:42       ` Dmitry Gutov
2024-05-06  3:01         ` Yuan Fu
2024-05-05  5:33     ` Eli Zaretskii
2024-05-05  6:41       ` Po Lu
2024-05-05  7:03         ` Yuan Fu
2024-05-05  8:04           ` Po Lu
2024-05-05  9:10           ` Eli Zaretskii
2024-05-06  2:58             ` Yuan Fu
2024-05-06 11:00               ` Eli Zaretskii
2024-05-06 11:24                 ` Po Lu
2024-05-08  0:25                 ` Yuan Fu
2024-05-08  1:38                   ` Dmitry Gutov
2024-05-08  3:55                     ` Po Lu
2024-05-08 13:44                       ` Dmitry Gutov
2024-05-05 17:06         ` Max Nikulin
2024-05-05 15:25 ` Dmitry Gutov
2024-05-05 23:01 ` Rudolf Adamkovič
2024-05-05  4:07 Elijah G
     [not found] ` <15F31342-2F2E-4F1E-9FE2-D8F1A62BCC5A@gmail.com>
2024-05-05  7:06   ` Yuan Fu
2024-05-05  8:07   ` Po Lu
2024-05-06  0:10     ` Yuan Fu
2024-05-06  5:10   ` Elijah G
2024-05-06  5:32     ` Yuan Fu
2024-05-07  4:18       ` Elijah G
2024-05-08  0:22         ` Yuan Fu
2024-05-08  3:19           ` Elijah G
2024-05-09  3:52           ` Elijah G
2024-05-09  4:50             ` Yuan Fu
2024-05-10  2:04               ` Elijah G.
2024-05-10  7:06                 ` Eli Zaretskii
2024-05-10 15:43                   ` Elijah G.
2024-05-16  5:54                     ` Yuan Fu
2024-06-28  2:24 ` Elijah G.
2024-06-28  4:36   ` Yuan Fu
2024-06-28  9:22     ` Po Lu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7C9CBDCA-F058-46D4-9E40-63A18ABE34A7@gmail.com \
    --to=casouri@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=luangruo@yahoo.com \
    --cc=stefankangas@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.