editor: fixing, refactoring, improving styling in choice exercises

[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..dadc08e 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/ace/ace', 'libs/text!./template.html'], function($, ace, template) {

 
 'use strict';
 
 return function(sandbox) {
 
 
 'use strict';
 
 return function(sandbox) {
 

-    var view = $(sandbox.getTemplate('main')()),
+    var view = $(template),

         documentIsDirty = true,
         documentEditedHere = false,
         wlxmlDocument;
         documentIsDirty = true,
         documentEditedHere = false,
         wlxmlDocument;


@@ -15,28 +15,26 @@ return function(sandbox) {
             editor.gotoLine(0);
             documentEditedHere = false;
 
             editor.gotoLine(0);
             documentEditedHere = false;
 

-            sandbox.publish('documentSet');

             documentIsDirty = false;
         }
             documentIsDirty = false;
         }

-    }
+    };

 
     view.onHide = function() {
         if(documentEditedHere) {
 
     view.onHide = function() {
         if(documentEditedHere) {

-            documentEditedHere = false;
-            wlxmlDocument.loadXML(editor.getValue());
+            commitDocument();

         }
         }

-    }
-    
+    };
+
+    var commitDocument = function() {
+        documentEditedHere = false;
+        wlxmlDocument.loadXML(editor.getValue());
+    };
+

     var editor = ace.edit(view.find('#rng-sourceEditor-editor')[0]),
         session = editor.getSession();
     var editor = ace.edit(view.find('#rng-sourceEditor-editor')[0]),
         session = editor.getSession();

-    editor.setTheme("ace/theme/chrome");
-    session.setMode("ace/mode/xml")
+    session.setMode('ace/mode/xml');

     session.setUseWrapMode(true);
     
     session.setUseWrapMode(true);
     

-    $('textarea', view).on('keyup', function() {
-        documentEditedHere = true;
-    });
-    

     editor.getSession().on('change', function() {
         documentEditedHere = true;
     });
     editor.getSession().on('change', function() {
         documentEditedHere = true;
     });


@@ -52,7 +50,14 @@ return function(sandbox) {
             wlxmlDocument.on('change', function() {
                 documentIsDirty = true;
             });
             wlxmlDocument.on('change', function() {
                 documentIsDirty = true;
             });

+            wlxmlDocument.on('contentSet', function() {
+                documentIsDirty = true;
+            });
+        },
+        changesCommited: function() {
+            return !documentEditedHere;

         },
         },

+        commitChanges: commitDocument,

         getDocument: function() {
             return editor.getValue();
         }
         getDocument: function() {
             return editor.getValue();
         }