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 th...@debian.org
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 += ' a href=%s(w)/a' % (w)
 
 if col==red:
-- 
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 += ' a href=%s(w)/a' % (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 
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 th...@debian.org
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 += ' a href=%s(w)/a' % (w)
 
-if col==red:
+if col==#e87272:
 candidacy_at_risk[arch]=True
 
 print 'td style=background-color:%s%s/td' % (col,contents)
-- 
1.7.10.4

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 'td style=background-color:%s%s/td' % (col,contents)

I'm asking myself if we shouldn't just rather translate colors here,
i.e. (with an appropriate colormap)
  print 'td style=background-color:%s%s/td' % 
 (colormap.get(col,col),contents)



Andi


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: https://lists.debian.org/20140322155321.gc16...@mails.so.argh.org