rhbz844820 - add transUnitDetail panel in

zanata-war/src/main/java/org/zanata/webtrans/client/editor/table/SourceContentsView.java

@@ -25,6 +25,7 @@
 
 import org.zanata.webtrans.client.ui.HasSelectableSource;
 import org.zanata.webtrans.client.ui.SourcePanel;
+import org.zanata.webtrans.client.ui.TransUnitDetailsPanel;
 import org.zanata.webtrans.shared.model.TransUnit;
 
 import com.google.common.base.Objects;
@@ -34,7 +35,11 @@
 import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.FlowPanel;
 import com.google.gwt.user.client.ui.Grid;
+import com.google.gwt.user.client.ui.HasVerticalAlignment;
+import com.google.gwt.user.client.ui.VerticalPanel;
 import com.google.gwt.user.client.ui.Widget;
+import com.google.inject.Inject;
+import com.google.inject.Provider;
 
 public class SourceContentsView extends Composite implements SourceContentsDisplay
 {
@@ -43,19 +48,27 @@ public class SourceContentsView extends Composite implements SourceContentsDispl
    public static final int DEFAULT_ROWS = 1;
    private final Grid sourcePanelContainer;
    private List<HasSelectableSource> sourcePanelList;
+   private final TransUnitDetailsPanel transUnitDetailsPanel;
 
-   public SourceContentsView()
+   @Inject
+   public SourceContentsView(Provider<TransUnitDetailsPanel> transUnitDetailsPanelProvider)
    {
       sourcePanelList = new ArrayList<HasSelectableSource>();
+      VerticalPanel root = new VerticalPanel();
+      root.setWidth("100%");
       FlowPanel container = new FlowPanel();
       container.setSize("100%", "100%");
 
-      initWidget(container);
-
       sourcePanelContainer = new Grid(DEFAULT_ROWS, COLUMNS);
       sourcePanelContainer.addStyleName("sourceTable");
 
       container.add(sourcePanelContainer);
+      root.add(container);
+      root.setVerticalAlignment(HasVerticalAlignment.ALIGN_TOP);
+      transUnitDetailsPanel = transUnitDetailsPanelProvider.get();
+      root.add(transUnitDetailsPanel);
+
+      initWidget(root);
    }
 
    @Override
@@ -73,6 +86,7 @@ public void setValue(TransUnit value)
    @Override
    public void setValue(TransUnit value, boolean fireEvents)
    {
+      transUnitDetailsPanel.setDetails(value);
       sourcePanelContainer.resizeRows(value.getSources().size());
       sourcePanelList.clear();
 

zanata-war/src/main/java/org/zanata/webtrans/client/editor/table/TableEditorTableDefinition.java

@@ -260,7 +260,7 @@ public void setRowValueOverride(int row, TransUnit targetCell)
          }
       };
       this.targetCellEditor = new InlineTargetCellEditor(cancelCallBack, transValueCallBack, isReadOnly, targetContentsPresenter);
-      this.transUnitDetailsContent = new TransUnitDetailsPanel(messages.transUnitDetailsHeading());
+//      this.transUnitDetailsContent = new TransUnitDetailsPanel(messages.transUnitDetailsHeading());
       targetColumnDefinition.setCellEditor(targetCellEditor);
 
       addColumnDefinition(sourceColumnDefinition);

zanata-war/src/main/java/org/zanata/webtrans/client/resources/NavigationMessages.java

@@ -115,9 +115,6 @@ public interface NavigationMessages extends Messages
    @DefaultMessage("Click here for more info")
    String clickHereForMoreInfo();
 
-   @DefaultMessage("Translation Unit Details")
-   String transUnitDetailsHeading();
-
    @DefaultMessage("No content")
    String noContent();
 }

zanata-war/src/main/java/org/zanata/webtrans/client/resources/TableEditorMessages.java

@@ -1,5 +1,6 @@
 package org.zanata.webtrans.client.resources;
 
+import org.zanata.webtrans.shared.model.TransUnitId;
 import com.google.gwt.i18n.client.LocalizableResource.DefaultLocale;
 import com.google.gwt.i18n.client.LocalizableResource.Generate;
 import com.google.gwt.i18n.client.Messages;
@@ -75,4 +76,10 @@ String validationWarningsHeading(@PluralCount
 
    @DefaultMessage("Close editor")
    String closeEditor();
+
+   @DefaultMessage("Translation Unit Details")
+   String transUnitDetailsHeading();
+
+   @DefaultMessage("Row {0}, Id {1} Translation Unit Details {2}")
+   String transUnitDetailsHeadingWithInfo(int rowIndex, String transUnitId, String info);
 }

zanata-war/src/main/java/org/zanata/webtrans/client/service/TransUnitNavigationService.java

@@ -41,7 +41,7 @@
 
 @Singleton
 //TODO after retiring TableEditorPresenter remove unused methods and maybe rename it to something else (NavigationStateHolder?) and get the NavigationController to be named after Service.
-class TransUnitNavigationService
+public class TransUnitNavigationService
 {
    private Map<Long, ContentState> idAndStateMap;
    private ArrayList<Long> idIndexList;

zanata-war/src/main/java/org/zanata/webtrans/client/ui/TransUnitDetailsPanel.java

@@ -1,9 +1,12 @@
 package org.zanata.webtrans.client.ui;
 
+import org.zanata.webtrans.client.resources.TableEditorMessages;
 import org.zanata.webtrans.shared.model.TransUnit;
 
+import com.google.common.base.Strings;
 import com.google.gwt.core.client.GWT;
-import com.google.gwt.event.dom.client.ClickEvent;
+import com.google.gwt.event.dom.client.MouseOutEvent;
+import com.google.gwt.event.dom.client.MouseOverEvent;
 import com.google.gwt.uibinder.client.UiBinder;
 import com.google.gwt.uibinder.client.UiField;
 import com.google.gwt.uibinder.client.UiHandler;
@@ -12,6 +15,7 @@
 import com.google.gwt.user.client.ui.Label;
 import com.google.gwt.user.client.ui.LayoutPanel;
 import com.google.gwt.user.client.ui.Widget;
+import com.google.inject.Inject;
 
 public class TransUnitDetailsPanel extends Composite
 {
@@ -25,6 +29,7 @@ public class TransUnitDetailsPanel extends Composite
    HorizontalPanel msgContextPanel;
 
    private static TransUnitDetailsPanelUiBinder uiBinder = GWT.create(TransUnitDetailsPanelUiBinder.class);
+   private final TableEditorMessages messages;
 
    interface TransUnitDetailsPanelUiBinder extends UiBinder<Widget, TransUnitDetailsPanel>
    {
@@ -33,10 +38,12 @@ interface TransUnitDetailsPanelUiBinder extends UiBinder<Widget, TransUnitDetail
    @UiField
    Label resIdLabel, resId, sourceCommentLabel, msgContextLabel, msgContext, sourceComment, lastModifiedByLabel, lastModifiedBy, lastModifiedTimeLabel, lastModifiedTime;
 
-   public TransUnitDetailsPanel(String header)
+   @Inject
+   public TransUnitDetailsPanel(TableEditorMessages messages)
    {
+      this.messages = messages;
       initWidget(uiBinder.createAndBindUi(this));
-      headerLabel.setText(header);
+      headerLabel.setText(messages.transUnitDetailsHeading());
    }
 
    public void setDetails(TransUnit transUnit)
@@ -77,18 +84,45 @@ public void setDetails(TransUnit transUnit)
          lastModifiedTime.setText("");
       }
 
-      expand();
+      if (!Strings.isNullOrEmpty(context) || !Strings.isNullOrEmpty(transUnit.getSourceComment()))
+      {
+         headerLabel.setText(messages.transUnitDetailsHeadingWithInfo(transUnit.getRowIndex(), transUnit.getId().toString(), "(I)"));
+      }
+      else
+      {
+         headerLabel.setText(messages.transUnitDetailsHeadingWithInfo(transUnit.getRowIndex(), transUnit.getId().toString(), ""));
+      }
+//      expand();
+      collapse();
    }
 
+//   @UiHandler("headerLabel")
+//   public void onHeaderLabelClick(ClickEvent event)
+//   {
+//
+//      if (!contentPanel.isVisible())
+//      {
+//         expand();
+//      }
+//      else if (contentPanel.isVisible())
+//      {
+//         collapse();
+//      }
+//   }
+
    @UiHandler("headerLabel")
-   public void onHeaderLabelClick(ClickEvent event)
+   public void onMouseHover(MouseOverEvent event)
    {
-
       if (!contentPanel.isVisible())
       {
          expand();
       }
-      else if (contentPanel.isVisible())
+   }
+
+   @UiHandler("headerLabel")
+   public void onMouseOut(MouseOutEvent event)
+   {
+      if (contentPanel.isVisible())
       {
          collapse();
       }