A couple of things, Didier; in future, could you include the explanation of
your introduced properties--in this case, this:
+ foreback The foreground color of the face's
background pixmap,
+ when the pixmap is a bitmap.
+ Only used by faces on X and MS Windows devices.
+ For valid instantiators, see `make-color-specifier'.
+
in the commit message, as well as in the diff? “New "foreback" face
property.” is certainly succinct, but it’s difficult to get the gist of what
you mean.
Ar an seachtú lá is fiche de mí na Nollaig, scríobh Didier Verna:
NOTE: This patch has been committed.
... and here's another new face property that will most likely never
be used, but still fixes one particular problem. See the following blog
entry for an explanation:
http://www.didierverna.com/sciblog/index.php?post/2011/12/27/XEmacs-now-h...
As usual, the property name is open for suggestions ;-)
I would prefer if there were a sense that one is *forcing* the foreground of the
background to be something other than what the pixmap file says in the
name. But it’s difficult to come up with a reasonable formulation.
More important than that is your explanation that
#'set-face-background-pixmap doesn’t in general set the background *pixmap*,
it sets a *bitmap*, using the foreground and background. I haven’t seen this
documented anywhere in XEmacs, and it does need documenting.
--
‘Iodine deficiency was endemic in parts of the UK until, through what has been
described as “an unplanned and accidental public health triumph”, iodine was
added to cattle feed to improve milk production in the 1930s.’
(EN Pearce, Lancet, June 2011)
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches