Fixed mailto links. Bump to version v0.2.14

theonion · Aug 11, 2014 · 9c72e69 · 9c72e69
1 parent c1effe5
commit 9c72e69
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 6 deletions.
diff --git a/bower.json b/bower.json
@@ -1,6 +1,6 @@
 {
   "name": "onion-editor",
-  "version": "0.2.13",
+  "version": "0.2.14",
   "homepage": "https://github.com/theonion/editor",
   "authors": [
     "Mike Wnuk <mwnuk@theonion.com>"

diff --git a/build/onion-editor.js b/build/onion-editor.js
@@ -10622,7 +10622,7 @@ define('link-formatter',[
         if (
             url.substr(0, 7) !== "http://" &&
             url.substr(0, 8) !== "https://" &&
-            url.substr(0, 6) !== "mailto:" &&
+            url.substr(0, 7) !== "mailto:" &&
             url.substr(0, 1) !== "/" 
             ) {
             // check for email, but default to http

diff --git a/build/onion-editor.min.js b/build/onion-editor.min.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "onion-editor",
-  "version": "0.2.12",
+  "version": "0.2.14",
   "author": {
     "name": "Mike Wnuk",
     "email": "mwnuk@theonion.com"

diff --git a/src/js/formatters/link-formatter.js b/src/js/formatters/link-formatter.js
@@ -46,7 +46,7 @@ define([
         if (
             url.substr(0, 7) !== "http://" &&
             url.substr(0, 8) !== "https://" &&
-            url.substr(0, 6) !== "mailto:" &&
+            url.substr(0, 7) !== "mailto:" &&
             url.substr(0, 1) !== "/" 
             ) {
             // check for email, but default to http

diff --git a/test/formatters/link-formatter-spec.js b/test/formatters/link-formatter-spec.js
@@ -36,6 +36,12 @@ define(['scribe', 'formatters/link-formatter'], function(Scribe, LinkFormatter)
       var body = '<p><a>Some heading or something</a></p>';
       expect(scribe._htmlFormatterFactory.format(body)).toEqual(body);
     });
+
+    it('handles mailto: links', function () {
+      scribe.use(new LinkFormatter({domain: 'example.com'}));
+      var body = '<p><a href="mailto:webtech@theonion.com">Send us an email</a></p>';
+      expect(scribe._htmlFormatterFactory.format(body)).toEqual(body);
+    });
   });
 
 });