Bug#862056: screen-message: please invert the default colours

[Chris Lamb]

Source: screen-message
Version: 0.24-1
Severity: wishlist
Tags: patch

Hi,

Please invert the default colours for screen-message; a large white
background is not very "hacker friendly" and almost physically painful
to experience very late into the early hours…

Patch attached.


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      la...@debian.org / chris-lamb.co.uk
       `-

diff --git a/sm.6 b/sm.6
index 6a344b1..92140dd 100644
--- a/sm.6
+++ b/sm.6
@@ -28,7 +28,7 @@ sm \- Displays a short text fullscreen
 .SH DESCRIPTION
 .BR Screen\ Message 
 will display a given multi-line message as large as
-possible, fullscreen and black on white. You can specify the text either
+possible, fullscreen and white on black. You can specify the text either
 when launching sm, or edit it while the program is running.
 .PP
 After a short timeout, the text entry and the quit button will disappear, 
leaving
@@ -48,13 +48,13 @@ it will read the text to display from the standard input, 
see \fBREMOTE CONTROLL
 .TP
 .BI \-f,\ \-\-foreground= colordesc
 Define a different color to use for the foreground of the text to
-display than black. The text string can be in any of the forms accepted
+display than white. The text string can be in any of the forms accepted
 by XParseColor; these include name for a color from rgb.txt, such as
 DarkSlateGray, or a hex specification such as #3050b2 or #35b.
 .TP
 .BI \-b,\ \-\-background= colordesc
 Define a different color to use for the background of the text to
-display than white. For possible values, see above.
+display than black. For possible values, see above.
 .TP
 .BI \-n,\ \-\-font= fontspec
 Define a different font to use than the default sans-serif font of your
diff --git a/sm.c b/sm.c
index 0adc28f..0471720 100644
--- a/sm.c
+++ b/sm.c
@@ -97,7 +97,7 @@ static char *get_text() {
 static void redraw(GtkWidget *draw, cairo_t *cr, gpointer data) {
        int q;
 
-       gdk_cairo_set_source_rgba(cr, &white);
+       gdk_cairo_set_source_rgba(cr, &black);
        cairo_paint(cr);
 
        const char *text = get_text();
@@ -333,14 +333,14 @@ int main(int argc, char **argv) {
        g_signal_connect(window, "destroy", G_CALLBACK(gtk_main_quit), NULL);
 
        if (foreground != NULL) {
-               my_gdk_rgba_parse(&black, foreground);
+               my_gdk_rgba_parse(&white, foreground);
        } else {
-               my_gdk_rgba_parse(&black, "black");
+               my_gdk_rgba_parse(&white, "white");
        }
        if (background != NULL) {
-               my_gdk_rgba_parse(&white, background);
+               my_gdk_rgba_parse(&black, background);
        } else {
-               my_gdk_rgba_parse(&white, "white");
+               my_gdk_rgba_parse(&black, "black");
        }
 
        draw = gtk_drawing_area_new();
diff --git a/webapp.html b/webapp.html
index 27a6b8c..f4fa968 100644
--- a/webapp.html
+++ b/webapp.html
@@ -31,6 +31,8 @@
        border:0px;
        text-align:center;
        overflow:hidden;
+       background-color:black;
+       color:white;
 }
 
 #test {