fix(codepen): escape ampersand in  

Closes #2827. Fixes #2826. Without escaping the ampersand the   is translated into the incorrect character.
angular · May 12, 2015 · 0a5603f · 0a5603f
1 parent cae22e2
commit 0a5603f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/docs/app/js/codepen.js b/docs/app/js/codepen.js
@@ -142,6 +142,7 @@
     function htmlEscapeAmpersand(html) {
       return html
         .replace(/&gt;/g, "&amp;gt;")
+        .replace(/&nbsp;/g, "&amp;nbsp;")
         .replace(/&lt;/g, "&amp;lt;");
     }
 

diff --git a/docs/spec/codepen.spec.js b/docs/spec/codepen.spec.js
@@ -118,6 +118,15 @@ describe('CodepenDataAdapter', function() {
 
     });
 
+    describe('when the html example includes &nbsp;', function() {
+
+      it('escapes the ampersand, so that the codepen does not translate to an invalid character', function() {
+        demo.files.index.contents = '<div>&nbsp;&nbsp;</div>';
+        data = codepenDataAdapter.translate(demo, externalScripts);
+        expect(angular.element(data.html).html()).toBe('&amp;nbsp;&amp;nbsp;');
+      });
+    });
+
     describe('when the module definition in the js file is formatted in different ways', function() {
 
       it('handles second argument on a new line', function() {