From 4047ec14f31c157e48bcefdf87ea94e797cb3baa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20P=C3=A9ters?= <fpeters@entrouvert.com>
Date: Sun, 6 Apr 2014 14:50:37 +0200
Subject: [PATCH] backoffice: use status colour in status <select> box (#4533)

---
 wcs/backoffice/root.py  | 16 ++++++++++++----
 wcs/forms/backoffice.py | 10 +---------
 wcs/qommon/misc.py      | 14 ++++++++++++++
 3 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/wcs/backoffice/root.py b/wcs/backoffice/root.py
index 517b72f..ca08d1d 100644
--- a/wcs/backoffice/root.py
+++ b/wcs/backoffice/root.py
@@ -381,15 +381,23 @@ class FormPage(Directory):
         if waitpoint_status:
             r += htmltext('<h3>%s</h3>') % _('Status to display')
             r += htmltext('<select name="filter" onchange="this.form.submit()">')
-            filters = [('all', _('All')), ('pending', _('Pending')), ('done', _('Done'))]
+            filters = [('all', _('All'), None),
+                    ('pending', _('Pending'), None),
+                    ('done', _('Done'), None)]
             for status in waitpoint_status:
-                filters.append((status.id, status.name))
-            for filter_id, filter_label in filters:
+                filters.append((status.id, status.name, status.colour))
+            for filter_id, filter_label, filter_colour in filters:
                 if filter_id == selected_filter:
                     selected = ' selected="selected"'
                 else:
                     selected = ''
-                r += htmltext('<option value="%s"%s/>%s</option>') % (filter_id, selected, filter_label)
+                style = ''
+                if filter_colour and filter_colour != 'FFFFFF':
+                    fg_colour = misc.get_foreground_colour(filter_colour)
+                    style = 'style="background: #%s; color: %s;"' % (
+                            filter_colour, fg_colour)
+                r += htmltext('<option value="%s"%s %s>' % (filter_id, selected, style))
+                r += htmltext('%s</option>') % filter_label
             r += htmltext('</select>')
             r += htmltext('<noscript><input type="submit" value="%s"/></noscript>') % _('Refresh')
 
diff --git a/wcs/forms/backoffice.py b/wcs/forms/backoffice.py
index 6a76fa0..55c5a0a 100644
--- a/wcs/forms/backoffice.py
+++ b/wcs/forms/backoffice.py
@@ -55,15 +55,7 @@ class FormDefUI:
             colours = []
             for status in self.formdef.workflow.possible_status:
                 if status.colour and status.colour != 'FFFFFF':
-                    # luminance coefficients taken from section C-9 fro
-                    # http://www.faqs.org/faqs/graphics/colorspace-faq/
-                    brightess = int(status.colour[0:2], 16) * 0.212671 + \
-                            int(status.colour[2:4], 16) * 0.715160 + \
-                            int(status.colour[4:6], 16) * 0.072169
-                    if brightess > 128:
-                        fg_colour = 'black'
-                    else:
-                        fg_colour = 'white'
+                    fg_colour = misc.get_foreground_colour(status.colour)
                     colours.append((status.id, status.colour, fg_colour))
             if colours:
                 r += htmltext('<style>')
diff --git a/wcs/qommon/misc.py b/wcs/qommon/misc.py
index fb02693..140ea37 100644
--- a/wcs/qommon/misc.py
+++ b/wcs/qommon/misc.py
@@ -494,3 +494,17 @@ def get_variadic_url(url, variables, encode_query=True):
                 query = '&'.join('%s=%s' % (k,v) for (k,v) in query)
     return urlparse.urlunsplit((scheme, netloc, path, query, fragment))
 
+
+def get_foreground_colour(background_colour):
+    """Calculates the luminance of the given colour (six hexadecimal digits)
+       and returns an appropriate foreground colour."""
+    # luminance coefficients taken from section C-9 from
+    # http://www.faqs.org/faqs/graphics/colorspace-faq/
+    brightess = int(background_colour[0:2], 16) * 0.212671 + \
+            int(background_colour[2:4], 16) * 0.715160 + \
+            int(background_colour[4:6], 16) * 0.072169
+    if brightess > 128:
+        fg_colour = 'black'
+    else:
+        fg_colour = 'white'
+    return fg_colour
-- 
1.9.1