Bug#742329: use softer colours for architecture qualification page

2014-03-22 Thread Andreas Barth 
* Thijs Kinkhorst (th...@debian.org) [140322 16:51]:
> On Sat, March 22, 2014 16:28, Julien Cristau wrote:
> > looks like that if col=="red" is now broken?
> 
> Indeed, see fixed patch attached.

>  print '%s' % (col,contents)

I'm asking myself if we shouldn't just rather translate colors here,
i.e. (with an appropriate colormap)
>  print '%s' % 
> (colormap.get(col,col),contents)



Andi


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Bug#742329: use softer colours for architecture qualification page

2014-03-22 Thread Thijs Kinkhorst 
On Sat, March 22, 2014 16:28, Julien Cristau wrote:
> looks like that if col=="red" is now broken?

Indeed, see fixed patch attached.

Thijs
>From 8f84a1be4a9c49782ea8f736ef315508591e1608 Mon Sep 17 00:00:00 2001
From: Thijs Kinkhorst 
Date: Sat, 22 Mar 2014 16:47:16 +0100
Subject: [PATCH] Use less hard colours to reduce eyebleedage.

---
 www/jessie/arch_qualify.py |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/www/jessie/arch_qualify.py b/www/jessie/arch_qualify.py
index 0e56ead..f872f3c 100644
--- a/www/jessie/arch_qualify.py
+++ b/www/jessie/arch_qualify.py
@@ -18,9 +18,9 @@ from collections import OrderedDict
 
 ### formatting helpers
 
-def FAIL(value): return ("red",value)
-def WARN(value): return ("yellow",value)
-def PASS(value): return ("lime",value)
+def FAIL(value): return ("#e87272",value)
+def WARN(value): return ("#ccff66",value)
+def PASS(value): return ("#60e760",value)
 
 def c_truth(value):
 if value == True or value == "yes":
@@ -152,10 +152,10 @@ def dump_table(info,waivers):
 
 w = waivers.get(arch,{}).get(c,None)
 if w:
-col="cyan"
+col="#00"
 contents += ' (w)' % (w)
 
-if col=="red":
+if col=="#e87272":
 candidacy_at_risk[arch]=True
 
 print '%s' % (col,contents)
-- 
1.7.10.4

Bug#742329: use softer colours for architecture qualification page

2014-03-22 Thread Julien Cristau 
On Sat, Mar 22, 2014 at 15:11:54 +0100, Thijs Kinkhorst wrote:

> diff --git a/www/jessie/arch_qualify.py b/www/jessie/arch_qualify.py
> index 0e56ead..9ffa0ee 100644
> --- a/www/jessie/arch_qualify.py
> +++ b/www/jessie/arch_qualify.py
> @@ -18,9 +18,9 @@ from collections import OrderedDict
>  
>  ### formatting 
> helpers
>  
> -def FAIL(value): return ("red",value)
> -def WARN(value): return ("yellow",value)
> -def PASS(value): return ("lime",value)
> +def FAIL(value): return ("#e87272",value)
> +def WARN(value): return ("#ccff66",value)
> +def PASS(value): return ("#60e760",value)
>  
>  def c_truth(value):
>  if value == True or value == "yes":
> @@ -152,7 +152,7 @@ def dump_table(info,waivers):
>  
>  w = waivers.get(arch,{}).get(c,None)
>  if w:
> -col="cyan"
> +col="#00"
>  contents += ' (w)' % (w)
>  
>  if col=="red":

looks like that if col=="red" is now broken?

Cheers,
Julien


signature.asc
Description: Digital signature

Bug#742329: use softer colours for architecture qualification page

2014-03-22 Thread Thijs Kinkhorst 
Package: release.debian.org
Severity: minor
Tags: patch

Attached patch uses softer colours which are easier on the eye for
the architecture qualification page.
>From 3932bb06d69557a5d05efbf50459d9b7b9b5cccf Mon Sep 17 00:00:00 2001
From: Thijs Kinkhorst 
Date: Sat, 22 Mar 2014 14:39:18 +0100
Subject: [PATCH] Use less hard colours to reduce eyebleedage.

---
 www/jessie/arch_qualify.py |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/www/jessie/arch_qualify.py b/www/jessie/arch_qualify.py
index 0e56ead..9ffa0ee 100644
--- a/www/jessie/arch_qualify.py
+++ b/www/jessie/arch_qualify.py
@@ -18,9 +18,9 @@ from collections import OrderedDict
 
 ### formatting helpers
 
-def FAIL(value): return ("red",value)
-def WARN(value): return ("yellow",value)
-def PASS(value): return ("lime",value)
+def FAIL(value): return ("#e87272",value)
+def WARN(value): return ("#ccff66",value)
+def PASS(value): return ("#60e760",value)
 
 def c_truth(value):
 if value == True or value == "yes":
@@ -152,7 +152,7 @@ def dump_table(info,waivers):
 
 w = waivers.get(arch,{}).get(c,None)
 if w:
-col="cyan"
+col="#00"
 contents += ' (w)' % (w)
 
 if col=="red":
-- 
1.7.10.4