Switch to CSS interpretation of rgb() and rgba() colors

CSS3 defines a somewhat odd syntax for rgba() colors - the rgb values
are integers from 0 to 255 or percentages and the a value is a
float from 0 to 1. To avoid increasing the total amount of confusion
in the world, make gdk_rgb_to_string() and gdk_rgb_parse() follow
this syntax rather than using floats for r, g, and b.

https://bugzilla.gnome.org/show_bug.cgi?id=633762
This commit is contained in:
Owen W. Taylor
2010-11-02 15:30:44 -04:00
parent fbdcf193ae
commit 5daab52661
2 changed files with 85 additions and 35 deletions

View File

@ -14,23 +14,23 @@ test_color_parse (void)
res = gdk_rgba_parse ("", &color);
g_assert (!res);
expected.red = 100/255.;
expected.green = 90/255.;
expected.blue = 80/255.;
expected.alpha = 0.1;
res = gdk_rgba_parse ("rgba(100,90,80,0.1)", &color);
g_assert (res);
g_assert (gdk_rgba_equal (&color, &expected));
expected.red = 0.4;
expected.green = 0.3;
expected.blue = 0.2;
expected.alpha = 0.1;
res = gdk_rgba_parse ("rgba(0.4,0.3,0.2,0.1)", &color);
res = gdk_rgba_parse ("rgba(40%,30%,20%,0.1)", &color);
g_assert (res);
g_assert (gdk_rgba_equal (&color, &expected));
res = gdk_rgba_parse ("rgba ( 0.4 , 0.3 , 0.2 , 0.1 )", &color);
g_assert (res);
g_assert (gdk_rgba_equal (&color, &expected));
expected.red = 0.4;
expected.green = 0.3;
expected.blue = 0.2;
expected.alpha = 1.0;
res = gdk_rgba_parse ("rgb(0.4,0.3,0.2)", &color);
res = gdk_rgba_parse ("rgba( 40 % , 30 % , 20 % , 0.1 )", &color);
g_assert (res);
g_assert (gdk_rgba_equal (&color, &expected));
@ -61,10 +61,13 @@ test_color_to_string (void)
gchar *res_en;
gchar *orig;
/* Using /255. values for the r, g, b components should
* make sure they round-trip exactly without rounding
* from the double => integer => double conversions */
rgba.red = 1.0;
rgba.green = 0.5;
rgba.blue = 0.1;
rgba.alpha = 1.0;
rgba.green = 128/255.;
rgba.blue = 64/255.;
rgba.alpha = 0.5;
orig = g_strdup (setlocale (LC_ALL, NULL));
res = gdk_rgba_to_string (&rgba);