Harmonize spaces after keywords

Always use a space after a keyword preceding a ( or {, except for
anonymous functions.

Author: fphilipe

Gruntfile.js

@@ -7,7 +7,7 @@ var folderMount = function folderMount(connect, point) {
   return connect.static(path.resolve(point));
 };
 
-module.exports = function (grunt) {
+module.exports = function(grunt) {
 
   // load all grunt tasks
   require('matchdep').filterDev('grunt-*').forEach(grunt.loadNpmTasks);
@@ -50,7 +50,7 @@ module.exports = function (grunt) {
         options: {
           port: 9001,
           hostname: '0.0.0.0',
-          middleware: function (connect, options) {
+          middleware: function(connect, options) {
             return [
               folderMount(connect, options.base)
             ];

src/behavior.js

@@ -55,7 +55,7 @@ var behavior = (function() {
       var br = document.createElement('br');
       cursor.insertBefore(br);
 
-      if(atEnd) {
+      if (atEnd) {
         log('at the end');
 
         var noWidthSpace = document.createTextNode('\u200B');
@@ -76,9 +76,9 @@ var behavior = (function() {
       log('Default insert ' + direction + ' behavior');
       var parent = element.parentNode;
       var newElement = element.cloneNode(false);
-      if(newElement.id) newElement.removeAttribute('id');
+      if (newElement.id) newElement.removeAttribute('id');
 
-      switch(direction) {
+      switch (direction) {
       case 'before':
         parent.insertBefore(newElement, element);
         element.focus();
@@ -104,7 +104,7 @@ var behavior = (function() {
       log('Default merge ' + direction + ' behavior');
       var container, merger, fragment, chunks, i, newChild, range;
 
-      switch(direction) {
+      switch (direction) {
       case 'before':
         container = block.previous(element);
         merger = element;
@@ -115,18 +115,18 @@ var behavior = (function() {
         break;
       }
 
-      if(!(container && merger))
+      if (!(container && merger))
         return;
 
-      if(container.childNodes.length > 0)
+      if (container.childNodes.length > 0)
         cursor.moveAtTextEnd(container);
       else
         cursor.moveAtBeginning(container);
       cursor.setSelection();
 
       fragment = document.createDocumentFragment();
       chunks = merger.childNodes;
-      for(i = 0; i < chunks.length; i++) {
+      for (i = 0; i < chunks.length; i++) {
         fragment.appendChild(chunks[i].cloneNode(true));
       }
       newChild = container.appendChild(fragment);
@@ -149,7 +149,7 @@ var behavior = (function() {
 
       var next, previous;
 
-      switch(direction) {
+      switch (direction) {
       case 'before':
         previous = block.previous(element);
         if (previous) {
@@ -175,11 +175,11 @@ var behavior = (function() {
       log('Default clipboard behavior');
       var pasteHolder, sel;
 
-      if(action !== 'paste') return;
+      if (action !== 'paste') return;
 
       element.setAttribute(config.pastingAttribute, true);
 
-      if(cursor.isSelection) {
+      if (cursor.isSelection) {
         cursor = cursor.deleteContent();
       }
 

src/block.js

@@ -2,13 +2,13 @@ var block = (function() {
   return {
     next: function(element) {
       var next = element.nextElementSibling;
-      if(next && next.getAttribute('contenteditable')) return next;
+      if (next && next.getAttribute('contenteditable')) return next;
       return null;
     },
 
     previous: function(element) {
       var previous = element.previousElementSibling;
-      if(previous && previous.getAttribute('contenteditable')) return previous;
+      if (previous && previous.getAttribute('contenteditable')) return previous;
       return null;
     }
   };

src/content.js

@@ -21,18 +21,18 @@ var content = (function() {
 
       for (i = 0; i < element.childNodes.length; i++) {
         node = element.childNodes[i];
-        if(!node) continue;
+        if (!node) continue;
 
         // skip empty tags, so they'll get removed
-        if(node.nodeName !== 'BR' && !node.textContent) continue;
+        if (node.nodeName !== 'BR' && !node.textContent) continue;
 
-        if(node.nodeType === 1 && node.nodeName !== 'BR') {
+        if (node.nodeType === 1 && node.nodeName !== 'BR') {
           sibling = node;
-          while((sibling = sibling.nextSibling) !== null) {
-            if(!parser.isSameNode(sibling, node))
+          while ((sibling = sibling.nextSibling) !== null) {
+            if (!parser.isSameNode(sibling, node))
               break;
 
-            for(j = 0; j < sibling.childNodes.length; j++) {
+            for (j = 0; j < sibling.childNodes.length; j++) {
               node.appendChild(sibling.childNodes[j].cloneNode(true));
             }
 
@@ -71,9 +71,9 @@ var content = (function() {
     normalizeSpaces: function(element) {
       var nonBreakingSpace = '\u00A0';
 
-      if(!element) return;
+      if (!element) return;
 
-      if(element.nodeType === 3) {
+      if (element.nodeType === 3) {
         element.nodeValue = element.nodeValue.replace(/^(\s)/, nonBreakingSpace).replace(/(\s)$/, nonBreakingSpace);
       }
       else {
@@ -207,7 +207,7 @@ var content = (function() {
         $(elem)[0] :
         elem;
 
-      if(this.isWrappable(range)) {
+      if (this.isWrappable(range)) {
         var a = range.surroundContents(elem);
       } else {
         console.log('content.wrap(): can not surround range');
@@ -285,7 +285,7 @@ var content = (function() {
             return node.nodeValue.search(charRegexp) >= 0;
           });
 
-          for(var i = 0; i < textNodes.length; i++) {
+          for (var i = 0; i < textNodes.length; i++) {
             var node = textNodes[i];
             node.nodeValue = node.nodeValue.replace(charRegexp, '');
           }

src/cursor.js

@@ -174,13 +174,13 @@ var Cursor = (function() {
       },
 
       equals: function(cursor) {
-        if(!cursor) return false;
+        if (!cursor) return false;
 
-        if(!cursor.host) return false;
-        if(!cursor.host.isEqualNode(this.host)) return false;
+        if (!cursor.host) return false;
+        if (!cursor.host.isEqualNode(this.host)) return false;
 
-        if(!cursor.range) return false;
-        if(!cursor.range.equals(this.range)) return false;
+        if (!cursor.range) return false;
+        if (!cursor.range.equals(this.range)) return false;
 
         return true;
       }

src/dispatcher.js

@@ -24,10 +24,10 @@ var dispatcher = (function() {
    */
   var setupElementEvents = function($document, notifier) {
     $document.on('focus.editable', editableSelector, function(event) {
-      if(this.getAttribute(config.pastingAttribute)) return;
+      if (this.getAttribute(config.pastingAttribute)) return;
       notifier('focus', this);
     }).on('blur.editable', editableSelector, function(event) {
-      if(this.getAttribute(config.pastingAttribute)) return;
+      if (this.getAttribute(config.pastingAttribute)) return;
       notifier('blur', this);
     }).on('copy.editable', editableSelector, function(event) {
       log('Copy');
@@ -111,7 +111,7 @@ var dispatcher = (function() {
       var range = selectionWatcher.getFreshRange();
       if (range.isCursor) {
         var cursor = range.getCursor();
-        if(cursor.isAtTextEnd()) {
+        if (cursor.isAtTextEnd()) {
           event.preventDefault();
           event.stopPropagation();
           notifier('merge', this, 'after', cursor);
@@ -127,7 +127,7 @@ var dispatcher = (function() {
 
       if (cursor.isAtTextEnd()) {
         notifier('insert', this, 'after', cursor);
-      } else if(cursor.isAtBeginning()) {
+      } else if (cursor.isAtBeginning()) {
         notifier('insert', this, 'before', cursor);
       } else {
         notifier('split', this, cursor.before(), cursor.after(), cursor);
@@ -241,7 +241,7 @@ var dispatcher = (function() {
       if (eventListeners === undefined) return;
 
       for (var i=0, len=eventListeners.length; i < len; i++) {
-        if(eventListeners[i].apply(
+        if (eventListeners[i].apply(
             Editable,
             Array.prototype.slice.call(arguments).splice(1)
         ) === false)
@@ -263,7 +263,7 @@ var dispatcher = (function() {
       listeners = {};
       // TODO check the config.event object to prevent
       // registering invalid handlers
-      for(eventType in config.event) {
+      for (eventType in config.event) {
         this.addListener(eventType, config.event[eventType]);
       }
 

src/keyboard.js

@@ -32,7 +32,7 @@ var keyboard = (function() {
     if (eventListeners === undefined) return;
 
     for (var i=0, len=eventListeners.length; i < len; i++) {
-      if(eventListeners[i].apply(
+      if (eventListeners[i].apply(
           context,
           Array.prototype.slice.call(arguments).splice(2)
       ) === false)

src/parser.js

@@ -169,22 +169,22 @@ var parser = (function() {
       }
     },
 
-    isStartOffset: function (container, offset) {
+    isStartOffset: function(container, offset) {
       if (container.nodeType === 3) {
         return offset === 0;
       } else {
-        if(container.childNodes.length === 0)
+        if (container.childNodes.length === 0)
           return true;
         else
           return container.childNodes[offset] === container.firstChild;
       }
     },
 
-    isEndOffset: function (container, offset) {
+    isEndOffset: function(container, offset) {
       if (container.nodeType === 3) {
         return offset === container.length;
       } else {
-        if(container.childNodes.length === 0)
+        if (container.childNodes.length === 0)
           return true;
         else if (offset > 0)
           return container.childNodes[offset - 1] === container.lastChild;
@@ -210,7 +210,7 @@ var parser = (function() {
       }
     },
 
-    isTextEndOffset: function (container, offset) {
+    isTextEndOffset: function(container, offset) {
       if (container.nodeType === 3) {
         var text = string.trimRight(container.nodeValue);
         return offset >= text.length;
@@ -225,15 +225,15 @@ var parser = (function() {
     isSameNode: function(target, source) {
       var i, len, attr;
 
-      if(target.nodeType !== source.nodeType)
+      if (target.nodeType !== source.nodeType)
         return false;
 
-      if(target.nodeName !== source.nodeName)
+      if (target.nodeName !== source.nodeName)
         return false;
 
-      for(i = 0, len = target.attributes.length; i < len; i++) {
+      for (i = 0, len = target.attributes.length; i < len; i++) {
         attr = target.attributes[i];
-        if(source.getAttribute(attr.name) !== attr.value)
+        if (source.getAttribute(attr.name) !== attr.value)
           return false;
       }
 
@@ -248,7 +248,7 @@ var parser = (function() {
      * @return {HTMLElement}           THe deepest last child in the container.
      */
     latestChild: function(container) {
-      if(container.lastChild)
+      if (container.lastChild)
         return this.latestChild(container.lastChild);
       else
         return container;