refactoring: moving create(wlxmlNode) to constructor, caching element instance

[fnpeditor.git] / src / editor / modules / sourceEditor / sourceEditor.js

diff --git a/src/editor/modules/sourceEditor/sourceEditor.js b/src/editor/modules/sourceEditor/sourceEditor.js
index a2ec127..12536ac 100644 (file)
--- a/src/editor/modules/sourceEditor/sourceEditor.js
+++ b/src/editor/modules/sourceEditor/sourceEditor.js
@@ -1,10 +1,10 @@
-define(['libs/jquery'], function($) {
+define(['libs/jquery', 'libs/text!./template.html'], function($, template) {
 
 'use strict';
 
 return function(sandbox) {
 
-    var view = $(sandbox.getTemplate('main')()),
+    var view = $(template),
         documentIsDirty = true,
         documentEditedHere = false,
         wlxmlDocument;
@@ -15,17 +15,20 @@ return function(sandbox) {
             editor.gotoLine(0);
             documentEditedHere = false;
 
-            sandbox.publish('documentSet');
             documentIsDirty = false;
         }
     };
 
     view.onHide = function() {
         if(documentEditedHere) {
-            documentEditedHere = false;
-            wlxmlDocument.loadXML(editor.getValue());
+            commitDocument();
         }
     };
+
+    var commitDocument = function() {
+        documentEditedHere = false;
+        wlxmlDocument.loadXML(editor.getValue());
+    };
     
     /* globals ace */
     var editor = ace.edit(view.find('#rng-sourceEditor-editor')[0]),
@@ -34,10 +37,6 @@ return function(sandbox) {
     session.setMode('ace/mode/xml');
     session.setUseWrapMode(true);
     
-    $('textarea', view).on('keyup', function() {
-        documentEditedHere = true;
-    });
-    
     editor.getSession().on('change', function() {
         documentEditedHere = true;
     });
@@ -53,7 +52,14 @@ return function(sandbox) {
             wlxmlDocument.on('change', function() {
                 documentIsDirty = true;
             });
+            wlxmlDocument.on('contentSet', function() {
+                documentIsDirty = true;
+            });
+        },
+        changesCommited: function() {
+            return !documentEditedHere;
         },
+        commitChanges: commitDocument,
         getDocument: function() {
             return editor.getValue();
         }