fix copy source button issue

zanata · Aug 1, 2011 · c739cf3 · c739cf3
1 parent e3cba59
commit c739cf3
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 12 deletions.
diff --git a/...ata-war/src/main/java/org/zanata/webtrans/client/editor/table/InlineTargetCellEditor.java b/...ata-war/src/main/java/org/zanata/webtrans/client/editor/table/InlineTargetCellEditor.java
@@ -400,9 +400,9 @@ public void editCell(CellEditInfo cellEditInfo, TransUnit cellValue, Callback<Tr
          }
          restoreView();
       }
-      
+
       // save the content in previous cell before start new editing
-      if (this.cellValue != null && curRow != cellEditInfo.getRowIndex() && !textArea.getText().equals(this.cellValue.getTarget()))
+      if (this.cellValue != null && curRow != cellEditInfo.getRowIndex())
       {
          Log.debug("save content of previous cell");
          acceptEdit();

diff --git a/...anata-war/src/main/java/org/zanata/webtrans/client/editor/table/TableEditorPresenter.java b/...anata-war/src/main/java/org/zanata/webtrans/client/editor/table/TableEditorPresenter.java
@@ -490,19 +490,22 @@ public void onCopySource(CopySourceEvent event)
             int row = display.getCurrentPage() * display.getPageSize() + rowOffset;
             // display.getTableModel().setRowValueOverride(row,
             // event.getTransUnit());
-            if (display.getTargetCellEditor().isEditing())
-            {
-               int curRow = display.getTargetCellEditor().getCurrentRow();
-               if (curRow == row)
-               {
-                  display.getTargetCellEditor().setText(event.getTransUnit().getSource());
-                  display.getTargetCellEditor().setTextAreaSize();
-               }
-            }
-            else
+            // if (display.getTargetCellEditor().isEditing())
+            // {
+               // int curRow = display.getTargetCellEditor().getCurrentRow();
+               // if (curRow == row)
+               // {
+            // display.getTargetCellEditor().setText(event.getTransUnit().getSource());
+            // display.getTargetCellEditor().setTextAreaSize();
+               // }
+            // }
+            if (!display.getTargetCellEditor().isEditing())
             {
                tableModelHandler.gotoRow(row);
+
             }
+            display.getTargetCellEditor().setText(event.getTransUnit().getSource());
+            display.getTargetCellEditor().setTextAreaSize();
          }
 
       }));