From 2f291ef3b7b203163c7ed29124881ef1b0a1de66 Mon Sep 17 00:00:00 2001
From: Alex Eng <aeng@redhat.com>
Date: Fri, 27 Sep 2013 07:34:14 +1000
Subject: [PATCH] Implement fix from
 https://bugzilla.redhat.com/show_bug.cgi?id=1006063#c2, item: I, II

---
 .../java/org/zanata/webtrans/client/resources/UiMessages.java  | 3 +++
 .../org/zanata/webtrans/client/ui/ValidationWarningPanel.java  | 2 +-
 .../org/zanata/webtrans/client/view/TargetContentsDisplay.java | 2 +-
 .../java/org/zanata/webtrans/client/view/TransFilterView.java  | 1 +
 .../org/zanata/webtrans/client/view/TransFilterView.ui.xml     | 2 +-
 5 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/resources/UiMessages.java b/zanata-war/src/main/java/org/zanata/webtrans/client/resources/UiMessages.java
index ddeb7530ee..3ed1dfe80c 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/resources/UiMessages.java
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/resources/UiMessages.java
@@ -189,4 +189,7 @@ public interface UiMessages extends Messages
 
    @DefaultMessage("Highlight matches")
    String diffModeAsHighlight();
+
+   @DefaultMessage("Translation that contained validation warning or error.")
+   String invalidTooltip();
 }
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/ui/ValidationWarningPanel.java b/zanata-war/src/main/java/org/zanata/webtrans/client/ui/ValidationWarningPanel.java
index ccc0b6a608..6c4a80160d 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/ui/ValidationWarningPanel.java
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/ui/ValidationWarningPanel.java
@@ -110,7 +110,7 @@ public void onClick(ClickEvent event)
          public void onClick(ClickEvent event)
          {
             hide();
-            listener.setFocus();
+            listener.setSelected(transUnitId);
          }
       });
    }
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
index f9770a3852..9443e83462 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TargetContentsDisplay.java
@@ -108,7 +108,7 @@ interface Listener
 
       void rejectTranslation(TransUnitId id);
 
-      void setFocus();
+      void setSelected(TransUnitId transUnitId);
    }
 
    enum EditingState
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.java b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.java
index d80513d4bb..307d25328b 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.java
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.java
@@ -76,6 +76,7 @@ public TransFilterView(UiMessages messages)
       hintMessage = messages.findSourceOrTargetString();
       searchField.setTextBoxTitle(hintMessage);
       initWidget(uiBinder.createAndBindUi(this));
+      hasErrorChk.setTitle(messages.invalidTooltip());
       getElement().setId("TransFilterView");
    }
 
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.ui.xml b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.ui.xml
index 752d0b9f2d..03fd23a791 100644
--- a/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.ui.xml
+++ b/zanata-war/src/main/java/org/zanata/webtrans/client/view/TransFilterView.ui.xml
@@ -175,7 +175,7 @@
 			</li>
 
 			<li class="{style.drop-down}">
-				<g:CheckBox ui:field="hasErrorChk" styleName="{style.hasError} {style.checkbox}"><ui:msg>Has warning</ui:msg></g:CheckBox>
+				<g:CheckBox ui:field="hasErrorChk" styleName="{style.hasError} {style.checkbox}"><ui:msg>Invalid</ui:msg></g:CheckBox>
 			</li>
 		</ul>
 	</g:HTMLPanel>