krafthaus/bauhaus

Function handleContentEditableSelection has 266 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function handleContentEditableSelection() {
        var dom = editor.dom, selection = editor.selection, caretContainerId = 'mce_noneditablecaret', invisibleChar = '\uFEFF';

        // Get caret container parent for the specified node
        function getParentCaretContainer(node) {

Function handleKey has 124 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function handleKey(e) {
            var keyCode = e.keyCode, nonEditableParent, caretContainer, startElement, endElement;

            function getNonEmptyTextNodeSibling(node, prev) {
                while ((node = node[prev ? 'previousSibling' : 'nextSibling'])) {

File plugin.js has 369 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/**
 * plugin.js
 *
 * Copyright, Moxiecode Systems AB
 * Released under LGPL License.

Function moveSelection has 57 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        function moveSelection() {
            var nonEditableStart, nonEditableEnd, isCollapsed, rng, element;

            // Checks if there is any contents to the left/right side of caret returns the noneditable element or
            // any editable element if it finds one inside

Function canDelete has 52 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            function canDelete(backspace) {
                var rng, container, offset, nonEditableParent;

                function removeNodeIfNotParent(node) {
                    var parent = container;

Function hasSideContent has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

            function hasSideContent(element, left) {
                var container, offset, walker, node, len;

                container = rng.startContainer;
                offset = rng.startOffset;

Avoid deeply nested control flow statements.

Open

                            if (nonEditableParent && getContentEditable(nonEditableParent) === "false") {
                                e.preventDefault();

                                if (keyCode == VK.RIGHT) {
                                    positionCaretOnElement(nonEditableParent, false);

Avoid deeply nested control flow statements.

Open

                            if (nonEditableParent && getContentEditable(nonEditableParent) === "false") {
                                e.preventDefault();

                                if (keyCode == VK.LEFT) {
                                    positionCaretOnElement(nonEditableParent, true);

Avoid too many return statements within this function.

Open

                return element;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            if (nonEditableParent && getContentEditable(nonEditableParent) === "false") {
                                e.preventDefault();

                                if (keyCode == VK.RIGHT) {
                                    positionCaretOnElement(nonEditableParent, false);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                            if (nonEditableParent && getContentEditable(nonEditableParent) === "false") {
                                e.preventDefault();

                                if (keyCode == VK.LEFT) {
                                    positionCaretOnElement(nonEditableParent, true);

• Read up
  Read up