editor fix: source editor should also react to contentSet event

[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 0d7b12e..1930939 100644 (file)
--- a/src/editor/modules/sourceEditor/sourceEditor.js
+++ b/src/editor/modules/sourceEditor/sourceEditor.js
@@ -1,10 +1,10 @@
-define(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;
@@ -18,25 +18,22 @@ return function(sandbox) {
             sandbox.publish('documentSet');
             documentIsDirty = false;
         }
-    }
+    };
 
     view.onHide = function() {
         if(documentEditedHere) {
             documentEditedHere = false;
             wlxmlDocument.loadXML(editor.getValue());
         }
-    }
+    };
     
+    /* globals ace */
     var editor = ace.edit(view.find('#rng-sourceEditor-editor')[0]),
         session = editor.getSession();
-    editor.setTheme("ace/theme/chrome");
-    session.setMode("ace/mode/xml")
+    editor.setTheme('ace/theme/chrome');
+    session.setMode('ace/mode/xml');
     session.setUseWrapMode(true);
     
-    $('textarea', view).on('keyup', function() {
-        documentEditedHere = true;
-    });
-    
     editor.getSession().on('change', function() {
         documentEditedHere = true;
     });
@@ -52,6 +49,9 @@ return function(sandbox) {
             wlxmlDocument.on('change', function() {
                 documentIsDirty = true;
             });
+            wlxmlDocument.on('contentSet', function() {
+                documentIsDirty = true;
+            });
         },
         getDocument: function() {
             return editor.getValue();