From d4969e02eee9f0148ed7e00bde6793a0d295c692 Mon Sep 17 00:00:00 2001
From: David Huynh <dfhuynh@gmail.com>
Date: Mon, 25 Oct 2010 17:27:38 +0000
Subject: [PATCH] Fixed: Issue 167 in google-refine: 'No syntax error' is
 alarming

git-svn-id: http://google-refine.googlecode.com/svn/trunk@1659 7d457c2a-affb-35e4-300a-418c747d4874
---
 .../core/scripts/dialogs/expression-preview-dialog.html      | 2 +-
 .../core/scripts/dialogs/expression-preview-dialog.js        | 4 ++--
 .../core/scripts/views/data-table/text-transform-dialog.html | 2 +-
 .../core/styles/dialogs/expression-preview-dialog.less       | 5 ++++-
 4 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.html b/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.html
index c2915093d..9638ee102 100644
--- a/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.html
+++ b/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.html
@@ -6,7 +6,7 @@
   </tr>
   <tr>
     <td colspan="3"><div class="input-container"><textarea class="expression-preview-code" bind="expressionPreviewTextarea" /></div></td>
-    <td class="expression-preview-error-container" bind="expressionPreviewErrorContainer" width="150" style="vertical-align: top;"></td>
+    <td class="expression-preview-parsing-status" bind="expressionPreviewParsingStatus" width="150" style="vertical-align: top;"></td>
   </tr>
   <tr>
     <td colspan="4">
diff --git a/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.js b/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.js
index 7ef43d435..d4a26afde 100644
--- a/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.js
+++ b/main/webapp/modules/core/scripts/dialogs/expression-preview-dialog.js
@@ -354,10 +354,10 @@ ExpressionPreviewDialog.Widget.prototype._renderPreview = function(expression, d
     };
     
     if (this._results !== null) {
-        this._elmts.expressionPreviewErrorContainer.empty().text("No syntax error.");
+        this._elmts.expressionPreviewParsingStatus.empty().removeClass("error").text("No syntax error.");
     } else {
         var message = (data.type == "parser") ? data.message : "Internal error";
-        this._elmts.expressionPreviewErrorContainer.empty().text(message);
+        this._elmts.expressionPreviewParsingStatus.empty().addClass("error").text(message);
     }
     
     for (var i = 0; i < this._values.length; i++) {
diff --git a/main/webapp/modules/core/scripts/views/data-table/text-transform-dialog.html b/main/webapp/modules/core/scripts/views/data-table/text-transform-dialog.html
index 1c926c3e8..23a2443a8 100644
--- a/main/webapp/modules/core/scripts/views/data-table/text-transform-dialog.html
+++ b/main/webapp/modules/core/scripts/views/data-table/text-transform-dialog.html
@@ -8,7 +8,7 @@
               <td width="1%">On error</td>
               <td><input type="radio" name="text-transform-dialog-onerror-choice" value="set-to-blank" checked /> set to blank<br/><input type="radio" name="text-transform-dialog-onerror-choice" value="store-error" /> store error<br/><input type="radio" name="text-transform-dialog-onerror-choice" value="keep-original" /> keep original</td>
               <td width="1%"><input type="checkbox" bind="repeatCheckbox" /></td>
-              <td>Re-transform until no change<br/>up to <input bind="repeatCountInput" value="10" size="3" /> times</td>
+              <td>Re-transform up to <input bind="repeatCountInput" value="10" size="2" /> times until no change</td>
           </tr>
       </table>
     </div>
diff --git a/main/webapp/modules/core/styles/dialogs/expression-preview-dialog.less b/main/webapp/modules/core/styles/dialogs/expression-preview-dialog.less
index 887c41a38..078bf316b 100644
--- a/main/webapp/modules/core/styles/dialogs/expression-preview-dialog.less
+++ b/main/webapp/modules/core/styles/dialogs/expression-preview-dialog.less
@@ -43,7 +43,10 @@ textarea.expression-preview-code {
     vertical-align: top;
 }
 
-.expression-preview-error-container {
+.expression-preview-parsing-status {
+    color: @light_grey;
+}
+.expression-preview-parsing-status.error {
     color: red;
 }