Treat nbsp as a tag

Author: Bill Long

angular-rich-text-diff.js

@@ -17,6 +17,11 @@ var AngularRichTextDiff;
             });
             this.tagMap = {};
             this.mapLength = 0;
+            // Go ahead and map nbsp;
+            var unicodeCharacter = String.fromCharCode(this.unicodeRangeStart + this.mapLength);
+            this.tagMap[' '] = unicodeCharacter;
+            this.tagMap[unicodeCharacter] = ' ';
+            this.mapLength++;
             this.dmp = new diff_match_patch();
             this.doDiff();
         }
@@ -33,6 +38,14 @@ var AngularRichTextDiff;
             this.$scope.diffOutput = this.$sce.trustAsHtml(diffOutput);
         };
         RichTextDiffController.prototype.insertTagsForOperation = function (diffableString, operation) {
+            // Don't insert anything if these are all tags
+            var n = -1;
+            do {
+                n++;
+            } while (diffableString.charCodeAt(n) >= this.unicodeRangeStart);
+            if (n + 1 >= diffableString.length) {
+                return diffableString;
+            }
             var openTag = '';
             var closeTag = '';
             if (operation === 1) {
@@ -71,6 +84,7 @@ var AngularRichTextDiff;
             return outputString;
         };
         RichTextDiffController.prototype.convertHtmlToDiffableString = function (htmlString) {
+            htmlString = htmlString.replace(/ /g, this.tagMap[' ']);
             var diffableString = '';
             var offset = 0;
             while (offset < htmlString.length) {

angular-rich-text-diff.ts

@@ -25,10 +25,17 @@ module AngularRichTextDiff {
         dmp: diff_match_patch;
 
         constructor(public $scope: IRichTextDiffScope, public $sce: ng.ISCEService) {
-            $scope.$watch('left', () => { this.doDiff(); });
-            $scope.$watch('right', () => { this.doDiff(); });
+            $scope.$watch('left',() => { this.doDiff(); });
+            $scope.$watch('right',() => { this.doDiff(); });
             this.tagMap = {};
             this.mapLength = 0;
+            
+            // Go ahead and map nbsp;
+            var unicodeCharacter = String.fromCharCode(this.unicodeRangeStart + this.mapLength);
+            this.tagMap[' '] = unicodeCharacter;
+            this.tagMap[unicodeCharacter] = ' ';
+            this.mapLength++;
+
             this.dmp = new diff_match_patch();
             this.doDiff();
         }
@@ -48,6 +55,17 @@ module AngularRichTextDiff {
         }
 
         insertTagsForOperation(diffableString: string, operation: number): string {
+        
+            // Don't insert anything if these are all tags
+            var n = -1;
+            do {
+                n++;
+            }
+            while (diffableString.charCodeAt(n) >= this.unicodeRangeStart);
+            if (n + 1 >= diffableString.length) {
+                return diffableString;
+            }
+
             var openTag = '';
             var closeTag = '';
             if (operation === 1) {
@@ -88,6 +106,7 @@ module AngularRichTextDiff {
         }
 
         convertHtmlToDiffableString(htmlString: string): string {
+            htmlString = htmlString.replace(/ /g, this.tagMap[' ']);
             var diffableString = '';
 
             var offset = 0;

bower.json

@@ -1,7 +1,7 @@
 {
   "name": "angular-rich-text-diff",
   "main": "angular-rich-text-diff.js",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "authors": [
     "Bill Long <bill@blackout.us>"
   ],