|
From: | Daniel Colascione |
Subject: | Re: [PATCH] About the :distant-foreground face attribute |
Date: | Wed, 15 Jan 2014 13:03:29 -0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
On 01/15/2014 12:40 PM, Eli Zaretskii wrote:
From: Josh <address@hidden> Date: Wed, 15 Jan 2014 12:05:05 -0800 Cc: Stefan Monnier <address@hidden>, Jan Djärv <address@hidden>, Daniel Colascione <address@hidden>, Chong Yidong <address@hidden>, emacs-devel <address@hidden> Though this does seem likely, it might be wise to adopt a less specific name for this attribute now in case other appropriate uses of this adjustment mechanism later come to light. If we were to use a name like :attribute-filters or :attribute-adjusters we'd retain flexibility for the future even though we might only support adjusting foreground color initially (or indeed ever).It has been a long tradition of Emacs not to generalize before there's a clear and present need to.
Sure. The problem with :adjust-colors or some other specific name, though, is what happens to ordering if we introduce other kinds of filters. If we have :adjust-colors and some hypothetical :adjust-foo, the order in which the adjustments are made becomes unclear. If we have a single list of filters, ordering is natural.
Also, can we please include "post-" in the name of the attribute? I want to emphasize that the filtering happens *after* face merging.
[Prev in Thread] | Current Thread | [Next in Thread] |