rhbz#728858, Editor navigation should only save if something has chan…

…ged, click 'Save as Fuzzy' immediately save
zanata · Aug 11, 2011 · 86aa7d9 · 86aa7d9
1 parent 0dc0db1
commit 86aa7d9
Showing 1 changed file with 9 additions and 4 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
@@ -81,7 +81,8 @@ public class InlineTargetCellEditor implements CellEditor<TransUnit>
       public void onClick(ClickEvent event)
       {
          cellValue.setStatus(ContentState.NeedReview);
-         // acceptEdit();
+         cellValue.setTarget(textArea.getText());
+         curCallback.onComplete(curCellEditInfo, cellValue);
       }
    };
 
@@ -380,9 +381,13 @@ public void editCell(CellEditInfo cellEditInfo, TransUnit cellValue, Callback<Tr
       // save the content in previous cell before start new editing
       if (this.cellValue != null && curRow != cellEditInfo.getRowIndex())
       {
-         // this.cellValue.setStatus(ContentState.Approved);
-         Log.debug("save content of previous cell");
-         acceptEdit();
+         // if something has changed, save as approved
+         if (!this.cellValue.getTarget().equals(textArea.getText()))
+         {
+            Log.debug("save content of previous cell");
+            this.cellValue.setStatus(ContentState.Approved);
+            acceptEdit();
+         }
       }
 
       Log.debug("starting edit of cell");