X-Git-Url: https://git.mdrn.pl/fnpeditor.git/blobdiff_plain/ec10eee3168f37beedfb5dea757a4bdec2f2f36e..2f9850da2c6ad0258f87c3d8972689da6aa8011d:/src/editor/plugins/core/core.js diff --git a/src/editor/plugins/core/core.js b/src/editor/plugins/core/core.js index 425b161..31ac09d 100644 --- a/src/editor/plugins/core/core.js +++ b/src/editor/plugins/core/core.js @@ -8,12 +8,33 @@ var _ = require('libs/underscore'), footnote = require('plugins/core/footnote'), switchTo = require('plugins/core/switch'), lists = require('plugins/core/lists'), - plugin = {name: 'core', actions: [], canvas: {}, documentExtension: {textNode: {}, elementNode: {}}}, + plugin = {name: 'core', actions: [], canvas: {}, documentExtension: {textNode: {}, documentNode: {}}}, Dialog = require('views/dialog/dialog'), canvasElements = require('plugins/core/canvasElements'), - metadataEditor = require('plugins/core/metadataEditor/metadataEditor'); - - + metadataEditor = require('plugins/core/metadataEditor/metadataEditor'), + edumed = require('plugins/core/edumed/edumed'); + + +var exerciseFix = function(newNodes) { + var list, exercise, max, addedItem, answerValues; + if(newNodes.created.is('item')) { + list = newNodes.created.parent(); + exercise = list.parent(); + if(exercise && exercise.is('exercise')) { + if(exercise.is('exercise.order')) { + answerValues = exercise.object.getItems() + .map(function(item) { + if(!addedItem && item.node.sameNode(newNodes.created)) { + addedItem = item; + } + return item.getAnswer(); + }); + max = Math.max.apply(Math.max, answerValues); + addedItem.setAnswer(max + 1); + } + } + } +}; plugin.documentExtension.textNode.transformations = { breakContent: { @@ -38,6 +59,21 @@ plugin.documentExtension.textNode.transformations = { return true; // break } }); + + /* + /* + This makes sure that adding a new item to the list in some of the edumed exercises + sets an answer attribute that makes sense (and not just copies it which would create + a duplicate value). + + This won't be neccessary when/if we introduce canvas element own key event handlers. + + Alternatively, WLXML elements could implement their own item split methods that we + would delegate to. + */ + exerciseFix(newNodes); + /* */ + parentDescribingNodes.forEach(function(node) { newNodes.first.append(node); }); @@ -124,7 +160,7 @@ plugin.documentExtension.textNode.transformations = { } }; -plugin.documentExtension.elementNode.transformations = { +plugin.documentExtension.documentNode.transformations = { moveUp: function() { var toMerge = this, prev = toMerge.prev(); @@ -138,11 +174,16 @@ plugin.documentExtension.elementNode.transformations = { } ret = to.append(node); - if(idx === 0) { + if(idx === 0 && ret.nodeType === Node.TEXT_NODE) { toret = { node: ret, offset: ret.getText().length - len }; + } else if(!toret) { + toret = { + node: ret.getFirstTextNode(), + offset: 0 + }; } }); from.detach(); @@ -152,10 +193,35 @@ plugin.documentExtension.elementNode.transformations = { var strategies = [ { applies: function() { - return toMerge.is('p'); + return toMerge.nodeType === Node.TEXT_NODE && prev.is({tagName: 'span'}); }, run: function() { - if(prev && prev.is('p') || prev.is({tagName: 'header'})) { + var textNode = prev.getLastTextNode(), + txt, prevText, prevTextLen; + if(textNode) { + txt = textNode.getText(); + if(txt.length > 1) { + textNode.setText(txt.substr(0, txt.length-1)); + return {node: toMerge, offset: 0}; + } else { + if((prevText = prev.prev()) && prevText.nodeType === Node.TEXT_NODE) { + prevTextLen = prevText.getText().length; + } + prev.detach(); + return { + node: prevText ? prevText : toMerge, + offset : prevText ? prevTextLen : 0 + }; + } + } + } + }, + { + applies: function() { + return toMerge.is({tagName: 'div', 'klass': 'p'}) || (toMerge.is({tagName: 'div'}) && toMerge.getClass() === ''); + }, + run: function() { + if(prev && (prev.is('p') || prev.is({tagName: 'header'}))) { return merge(toMerge, prev); } if(prev && prev.is('list')) { @@ -164,6 +230,52 @@ plugin.documentExtension.elementNode.transformations = { } } }, + { + applies: function() { + return toMerge.is({tagName: 'span'}); + }, + run: function() { + /* globals Node */ + var toret = {node: toMerge.contents()[0] , offset: 0}, + txt, txtNode, parent; + if(!prev) { + toMerge.parents().some(function(p) { + if(p.is({tagName: 'span'})) { + parent = prev = p; + } else { + if(!parent) { + parent = p; + } + prev = prev && prev.prev(); + return true; + } + }); + } + if(!prev) { + return parent.moveUp(); + } + else if(prev.nodeType === Node.TEXT_NODE && (txt = prev.getText())) { + prev.setText(txt.substr(0, txt.length-1)); + return toret; + } else if(prev.is({tagName: 'span'})) { + if((txtNode = prev.getLastTextNode())) { + txt = txtNode.getText(); + if(txt.length > 1) { + txtNode.setText(txt.substr(0, txt.length-1)); + } else { + if(txtNode.parent().contents().length === 1) { + txtNode.parent().detach(); + } else { + txtNode.detach(); + } + + } + return toret; + } + } + + } + }, { applies: function() { return toMerge.is({tagName: 'header'}); @@ -523,8 +635,7 @@ plugin.actions = [ createWrapTextAction({name: 'cite', klass: 'cite', wrapDescription: gettext('Mark as citation'), unwrapDescription: gettext('Remove citation')}), linkAction, metadataEditor.action(metadataParams) -].concat(plugin.actions, templates.actions, footnote.actions, switchTo.actions, lists.actions); - +].concat(plugin.actions, templates.actions, footnote.actions, switchTo.actions, lists.actions, edumed.actions); plugin.config = function(config) { @@ -541,7 +652,7 @@ plugin.config = function(config) { }); }; -plugin.canvasElements = canvasElements; +plugin.canvasElements = canvasElements.concat(edumed.canvasElements); return plugin;