From b495ecf54f064bfafbc08fd71b367a219227d38f Mon Sep 17 00:00:00 2001
From: David Huynh <dfhuynh@gmail.com>
Date: Fri, 7 May 2010 21:25:00 +0000
Subject: [PATCH] Put a line through deactivated options in the checkbox button
 set in the range facet.

git-svn-id: http://google-refine.googlecode.com/svn/trunk@619 7d457c2a-affb-35e4-300a-418c747d4874
---
 src/main/webapp/scripts/facets/range-facet.js | 30 +++++++++----------
 src/main/webapp/styles/project/browsing.css   |  6 +++-
 2 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/src/main/webapp/scripts/facets/range-facet.js b/src/main/webapp/scripts/facets/range-facet.js
index b97bc8104..0f631b445 100644
--- a/src/main/webapp/scripts/facets/range-facet.js
+++ b/src/main/webapp/scripts/facets/range-facet.js
@@ -175,21 +175,6 @@ RangeFacet.prototype._renderOtherChoices = function() {
     $('<br>').appendTo(numericLabel);
     $('<span>').text(this._numericCount).addClass("facet-choice-count").appendTo(numericLabel);
 
-    // ----------------- blank -----------------
-    
-    var blankCheck = $('<input type="checkbox" />').attr("id",facet_id + "-blank").appendTo(choices).change(function() {
-        self._selectBlank = !self._selectBlank;
-        self._updateRest();
-    });
-    if (this._selectBlank) blankCheck.attr("checked","checked");
-
-    var blankLabel = $('<label>').attr("for", facet_id + "-blank").appendTo(choices);    
-    $('<span>').text("Blank ").addClass("facet-choice-label").appendTo(blankLabel);
-    $('<br>').appendTo(blankLabel);
-    $('<span>').text(this._blankCount).addClass("facet-choice-count").appendTo(blankLabel);
-
-    if (this._baseBlankCount === 0) blankCheck.removeAttr("checked");
-    
     // ----------------- non-numeric -----------------
     
     var nonNumericCheck = $('<input type="checkbox" />').attr("id",facet_id + "-non-numeric").appendTo(choices).change(function() {
@@ -205,6 +190,21 @@ RangeFacet.prototype._renderOtherChoices = function() {
 
     if (this._baseNonNumericCount === 0) nonNumericCheck.removeAttr("checked");
     
+    // ----------------- blank -----------------
+    
+    var blankCheck = $('<input type="checkbox" />').attr("id",facet_id + "-blank").appendTo(choices).change(function() {
+        self._selectBlank = !self._selectBlank;
+        self._updateRest();
+    });
+    if (this._selectBlank) blankCheck.attr("checked","checked");
+
+    var blankLabel = $('<label>').attr("for", facet_id + "-blank").appendTo(choices);    
+    $('<span>').text("Blank ").addClass("facet-choice-label").appendTo(blankLabel);
+    $('<br>').appendTo(blankLabel);
+    $('<span>').text(this._blankCount).addClass("facet-choice-count").appendTo(blankLabel);
+
+    if (this._baseBlankCount === 0) blankCheck.removeAttr("checked");
+    
     // ----------------- error -----------------
 
     var errorCheck = $('<input type="checkbox" />').attr("id",facet_id + "-error").appendTo(choices).change(function() {
diff --git a/src/main/webapp/styles/project/browsing.css b/src/main/webapp/styles/project/browsing.css
index afb35b3f0..c350f47de 100644
--- a/src/main/webapp/styles/project/browsing.css
+++ b/src/main/webapp/styles/project/browsing.css
@@ -197,9 +197,13 @@ img.facet-choice-link {
     margin-bottom: 0.8em;
     text-align: center;
 }
-.facet-range-choices .ui-button-text {
+.facet-range-choices .ui-button .ui-button-text {
     line-height: 1;
     padding: 0.1em 0.4em;   
+    text-decoration: line-through;
+}
+.facet-range-choices .ui-button.ui-state-active .ui-button-text {
+    text-decoration: none;
 }
 .facet-text-body {
     padding: 5px;