From 35def004ac501f155630128b8d9c1b2f829b81f6 Mon Sep 17 00:00:00 2001
From: David Huynh <dfhuynh@gmail.com>
Date: Fri, 15 Oct 2010 05:17:07 +0000
Subject: [PATCH] Made expression bars inside facets highlight on hover to
 suggest they are editable.

git-svn-id: http://google-refine.googlecode.com/svn/trunk@1556 7d457c2a-affb-35e4-300a-418c747d4874
---
 main/webapp/modules/core/scripts/facets/list-facet.js      | 2 +-
 main/webapp/modules/core/scripts/facets/range-facet.js     | 2 +-
 main/webapp/modules/core/scripts/facets/timerange-facet.js | 2 +-
 main/webapp/modules/core/styles/project/facets.less        | 4 ++++
 4 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/main/webapp/modules/core/scripts/facets/list-facet.js b/main/webapp/modules/core/scripts/facets/list-facet.js
index 41e5b9342..d73aa2646 100644
--- a/main/webapp/modules/core/scripts/facets/list-facet.js
+++ b/main/webapp/modules/core/scripts/facets/list-facet.js
@@ -124,7 +124,7 @@ ListFacet.prototype._initializeUI = function() {
                 '</td>' +
             '</tr></table></div>' +
         '</div>' +
-        '<div class="facet-expression" bind="expressionDiv"></div>' +
+        '<div class="facet-expression" bind="expressionDiv" title="Click to edit expression"></div>' +
         '<div class="facet-controls" bind="controlsDiv" style="display:none;"><div class="grid-layout layout-tightest layout-full">' +
             '<table><tr>' +
                 '<td><a bind="choiceCountContainer" class="action" href="javascript:{}"></a> <span class="facet-controls-sortControls" bind="sortGroup">sorted by ' +
diff --git a/main/webapp/modules/core/scripts/facets/range-facet.js b/main/webapp/modules/core/scripts/facets/range-facet.js
index 419005c1a..c868228a5 100644
--- a/main/webapp/modules/core/scripts/facets/range-facet.js
+++ b/main/webapp/modules/core/scripts/facets/range-facet.js
@@ -107,7 +107,7 @@ RangeFacet.prototype._initializeUI = function() {
                     '</td>' +
                 '</tr></table></div>' +
             '</div>' +
-            '<div class="facet-expression" bind="expressionDiv"></div>' +
+            '<div class="facet-expression" bind="expressionDiv" title="Click to edit expression"></div>' +
             '<div class="facet-range-body">' +
                 '<div class="facet-range-message" bind="messageDiv">Loading...</div>' +
                 '<div class="facet-range-slider" bind="sliderWidgetDiv">' +
diff --git a/main/webapp/modules/core/scripts/facets/timerange-facet.js b/main/webapp/modules/core/scripts/facets/timerange-facet.js
index 4daecd47e..697b86f8c 100644
--- a/main/webapp/modules/core/scripts/facets/timerange-facet.js
+++ b/main/webapp/modules/core/scripts/facets/timerange-facet.js
@@ -108,7 +108,7 @@ TimeRangeFacet.prototype._initializeUI = function() {
                     '</td>' +
                 '</tr></table></div>' +
             '</div>' +
-            '<div class="facet-expression" bind="expressionDiv"></div>' +
+            '<div class="facet-expression" bind="expressionDiv" title="Click to edit expression"></div>' +
             '<div class="facet-range-body">' +
                 '<div class="facet-range-message" bind="messageDiv">Loading...</div>' +
                 '<div class="facet-range-slider" bind="sliderWidgetDiv">' +
diff --git a/main/webapp/modules/core/styles/project/facets.less b/main/webapp/modules/core/styles/project/facets.less
index 725382f91..db3b698d2 100644
--- a/main/webapp/modules/core/styles/project/facets.less
+++ b/main/webapp/modules/core/styles/project/facets.less
@@ -49,6 +49,10 @@ a.facet-title-remove:hover {
     border-bottom: 1px solid #ccc;
 }
 
+.facet-expression:hover {
+  background: #ffffd6;
+}
+
 .facet-status {
     font-size: 90%;
     padding: 2px 5px;