[Debian-handbook-translators] review request of selectable color scheme (gray-scale/full-color) for lulu.com POD

[AYANOKOUZI, Ryuunosuke]

Dear Raphael,

Thanks very much for your comments.

At Wed, 2 Mar 2016 12:59:09 +0100,
Raphael Hertzog wrote:
>
> On Sun, 28 Feb 2016, AYANOKOUZI, Ryuunosuke wrote:
> > I would like to push a commit of adding a feature for selectable color
> > scheme[1].  The purpose of this is generating suitable (gray-scale) PDF
> > for lulu.com POD service.
> > Since I'm satisfied with gray-scale conversion by lulu.com,
> > automatic conversion of images are not considered.
> > You can pass the pdf.color.scheme variable from command line [2],
> > and colored parts such as URLs, mail-addresses, and clickable-linked words turn to be black.
> >
> > I would appreciate it if you review, modify and commit/push it as needed.
>
> I have a problem with this patch. It doesn't only change the color of
> links, it changes the color of any text. Even text that was changed to
> grey on purpose.
>
> If you look in the book when you have a page reference, the "page XX"
> strings are in grey. With your patch they are in black.

That is an intended effect.
Because gray text on shaded part (such as sidebar) is even more bad for read for me
compared to gray text on white background.

See following sidebars
* COMMUNITY For or against the non-free section?
* VOCABULARY Sub-project and derivative distribution
* BACK TO BASICS A catch-up course in the appendices

You can search for examples by following command.
$ xmlstarlet sel -t -c '//xref/parent::para/parent::sidebar' en-US/01_the-debian-project.xml

Do you think such "page XX" string should also be kept in gray?

Sincerely yours,
Ryuunosuke Ayanokouzi
--
AYANOKOUZI, Ryuunosuke <i38w7i3 at yahoo.co.jp>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP Digital Signature
URL: <http://lists.alioth.debian.org/pipermail/debian-handbook-translators/attachments/20160302/481e09ca/attachment.sig>