all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Dan Espen <despen@verizon.net>
To: help-gnu-emacs@gnu.org
Subject: custom-themes BAD?
Date: Tue, 25 Feb 2014 12:08:50 -0500	[thread overview]
Message-ID: <ic1tyrdsrh.fsf@home.home> (raw)


Custom-themes look to me like they are BAD (Broken As Designed).

I'm running Emacs 24.3.1 here and this is what I see in the themes
directory:

  5089 Jan  1  2013 adwaita-theme.el
  6434 Jan  1  2013 deeper-blue-theme.el
  6048 Jan  1  2013 dichromacy-theme.el
  2885 Jan  1  2013 light-blue-theme.el
 40948 Jan  1  2013 manoj-dark-theme.el
  5167 Jan  1  2013 misterioso-theme.el
  7720 Jan  1  2013 tango-dark-theme.el
  7080 Jan  1  2013 tango-theme.el
  3550 Jan  1  2013 tsdh-dark-theme.el
  3357 Jan  1  2013 tsdh-light-theme.el
  3680 Jan  1  2013 wheatgrass-theme.el
  5165 Jan  1  2013 whiteboard-theme.el
  5287 Jan  1  2013 wombat-theme.el

Notice that one theme, (manoj-dark) is way bigger than
any of the other themes.

Looking in these files, I see that each theme designer
decides which Emacs packages he's going to support.
The more packages supported, the bigger the theme file.

I think the fault lies with the base package for fonts
Emacs defines.  The basic faces were the font-lock faces,
builtin, constant, preprocessor, etc.

When we get to packages like gnus, gnus goes ahead and
defines it's own faces:

gnus-cite-attribution-face
gnus-cite-face-1
gnus-cite-face-2
gnus-cite-face-3
gnus-cite-face-4
gnus-cite-face-5
gnus-cite-face-6
gnus-cite-face-7
gnus-cite-face-8
gnus-cite-face-9
gnus-cite-face-10
gnus-cite-face-11
gnus-emphasis-bold
gnus-emphasis-bold-italic

That's the problem, there should be font-lock faces like:

font-lock-bold,
font-lock-level-1 thru 12,
font-lock-large-1,,,
font-lock-blue...
font-lock-reverse

and so on.

All the packages should be using font-lock-* faces as far as
possible.  Then the themes can all set the same set of faces
much more easily.


-- 
Dan Espen


             reply	other threads:[~2014-02-25 17:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-25 17:08 Dan Espen [this message]
2014-02-25 17:22 ` custom-themes BAD? Joost Kremers
2014-02-25 17:38   ` E Sabof
2014-02-25 17:54   ` Dan Espen
     [not found]   ` <mailman.16024.1393350344.10748.help-gnu-emacs@gnu.org>
2014-02-25 18:06     ` Dan Espen
2014-02-25 17:35 ` Drew Adams
     [not found] ` <mailman.16021.1393349758.10748.help-gnu-emacs@gnu.org>
2014-02-25 18:03   ` Dan Espen
2014-02-25 19:33     ` Rainer M Krug
     [not found]     ` <mailman.16038.1393356841.10748.help-gnu-emacs@gnu.org>
2014-02-25 19:48       ` Dan Espen
2014-02-25 20:04         ` Rainer M Krug
2014-02-25 21:05           ` W. Greenhouse
2014-02-25 22:19             ` Drew Adams
     [not found] <<ic1tyrdsrh.fsf@home.home>
     [not found] ` <<slrnlgpkad.3j2.joost.m.kremers@j.kremers4.news.arnhem.chello.nl>
2014-02-25 17:45   ` Drew Adams

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=ic1tyrdsrh.fsf@home.home \
    --to=despen@verizon.net \
    --cc=help-gnu-emacs@gnu.org \
    /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.