lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: New function css-color (accompanying x11-color) (issue 570690043 by


From: git
Subject: Re: New function css-color (accompanying x11-color) (issue 570690043 by address@hidden)
Date: Wed, 05 Jun 2019 12:32:47 -0700

Reviewers: thomasmorley651,

Message:
Please do *not* review this patch anymore, it has (for some reason that
is surely my fault) been replaced with
https://codereview.appspot.com/580780043/ so please review *that*...

Description:
New function css-color (accompanying x11-color)

I was asked if LilyPond could support choosing colors by their CSS name,
which I found a reasonable wish.

This commit makes make-x11-color-handler generic and specifies
a new function css-color. From this it would be easy to also add further
namespaces and color lists.

CSS colors are case insensitive, so this commit also provides support
for
this. Incidentally, a warning is now issued when the requested color is
not defined (instead of *silently* falling back to black).

Only warn about missing color when necessary


Merge pull request #6 from rettinghaus/dev/urs/css-fonts

add named colors from CSS Color Module Level 4

Rename x11-color.scm to color.scm

With the previous commit adding `css-color` the file `x11-color.scm`
should be
rename to the generic `color.scm`.

add named colors from CSS Color Module Level 4


Define function css-color (accompany x11-color)

It should be possible to specify colors using the CSS names in addition
to
the X11 names.

This commit makes `make-x11-color-handler` generic and adds a function
`css-color`.
What remains to be done is (of course documenting and) filling in the
remaining
~140 colors.

Please review this at https://codereview.appspot.com/570690043/

Affected files (+165, -718 lines):
  A + scm/color.scm
  M scm/lily.scm
  D scm/x11-color.scm





reply via email to

[Prev in Thread] Current Thread [Next in Thread]