use labels instead of spans in filter checkbox area

git-svn-id: svn+ssh://svn.debian.org/svn/secure-testing@28965 e39458fd-73e7-0310-bf30-c45bca0a0e42

3 files changed, 12 insertions, 8 deletions

diff --git a/bin/tracker_service.py b/bin/tracker_service.py
index cd903db681..9f2c51c488 100644
--- a/bin/tracker_service.py
+++ b/bin/tracker_service.py
@@ -54,10 +54,10 @@ class BugFilter:
         l.append(INPUT(type='hidden', name='filter', value='1'))
         for (prop, desc, field) in self.action_list:
             if prop in self.filters:
-                l.append(SPAN(INPUT(desc, type='checkbox', name='filter', value=prop, checked='checked'), rel=field))
+                l.append(LABEL(INPUT(desc, type='checkbox', name='filter', value=prop, checked='checked'), rel=field))
                 self.params[prop]=1
             else:
-                l.append(SPAN(INPUT(desc, type='checkbox', name='filter', value=prop), rel=field))
+                l.append(LABEL(INPUT(desc, type='checkbox', name='filter', value=prop), rel=field))
 
         return FORM(tag("span",l, id="filters"),
                     INPUT(type='submit', value='Apply'),
diff --git a/lib/python/web_support.py b/lib/python/web_support.py
index 82bb4c56ee..3306883841 100644
--- a/lib/python/web_support.py
+++ b/lib/python/web_support.py
@@ -419,6 +419,8 @@ def TD(*contents):
     return tag('td', contents)
 def FORM(*__contents, **__attribs):
     return Tag('form', __contents, __attribs)
+def LABEL(*__contents, **__attribs):
+    return Tag('label', __contents, __attribs)
 def INPUT(*__contents, **__attribs):
     return Tag('input', __contents, __attribs)
 def UL(contents):
diff --git a/static/style.css b/static/style.css
index d07030d220..649bc95911 100644
--- a/static/style.css
+++ b/static/style.css
@@ -137,6 +137,10 @@ form {
 	border-bottom: 1px solid #ddd;
 }
 
+label {
+	cursor: pointer;
+}
+
 input[type="text"] {
 	background: #fff;
 	border: 1px solid #ddd;
@@ -167,30 +171,28 @@ input[type="checkbox"] {
 	vertical-align: middle;
 }
 
-span[rel="urgency"] {
+label[rel="urgency"] {
 	padding: 0.5em 0;
 	color: #fff;
 	/*background: #729fcf;*/
 	background: #ad7fa8;
 }
 
-span[rel="scope"] {
+label[rel="scope"] {
 	padding: 0.5em 0;
 	color: #fff;
 	/*background: #3465a4;*/
 	background: #75507b;
 }
 
-span[rel="extra"] {
+label[rel="extra"] {
 	padding: 0.5em 0;
 	color: #fff;
 	/*background: #204a87;*/
 	background: #5c3566;
 }
 
-span[rel="urgency"]:last-child,
-span[rel="scope"]:last-child,
-span[rel="extra"]:last-child {
+label[rel="extra"]:last-child {
 	margin-right: 0.5em;
 	padding-right: 0.5em;
 }