Hi Christine,

Am Dienstag, den 15.12.2009, 20:14 -0500 schrieb A. Christine Spang:
> It just so happens that I was trying unicode-screensaver
> today and decided that I would much prefer white text on a
> black background as opposed to the opposite, though more
> from an aesthetic point of view than particularly worrying
> about screen burn-in.
> 
> I decided to try my hand at hacking up a solution myself.
> Please find the attached output of git format-patch, which
> can be applied to the current unicode-screensaver trunk via
> git am. It changes the default colours to a black background
> with a white foreground, but still allows those who prefer
> other colorschemes to specify them via the -background and
> -foreground options. This is my first time working with xlib
> or xscreensaver, so I hope the patch isn't too bad.
> 
> One thing I didn't do was hack the xscreensaver config file
> that would allow the user to configure the colours via the
> xscreensaver-demo gui, but I doubt that doing so would be
> too hard.

nice, four minutes before I asked for such a patch!

It looks good a at a first glance. I’ll have a deeper look tomorrow or
so, add the required GUI options and change the default back to my
choice :-)

Did you test what happens if you pass an illegal string to the color
option?

Greetings,
Joachim


-- 
Joachim "nomeata" Breitner
Debian Developer
  nome...@debian.org | ICQ# 74513189 | GPG-Keyid: 4743206C
  JID: nome...@joachim-breitner.de | http://people.debian.org/~nomeata

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil

Reply via email to