a lot of fix (outlook link)

dist/mailgo.js

@@ -233,7 +233,7 @@ var mailgoRender = function mailgoRender(mailgo) {
   showMailgo(); // listener keyDown
 
   mailgo.addEventListener("keydown", function () {
-    return mailgoKeydown(mail, url, mailtoHref, encEmail, copyButton);
+    return mailgoKeydown(mail, cc, bcc, subject, bodyMail, url, mailtoHref, encEmail, copyButton);
   }, false);
 }; // actions
 
@@ -245,8 +245,8 @@ var actions = {
   },
   openOutlook: function openOutlook(mail, subject, bodyMail) {
     var outlookUrl = "https://outlook.live.com/owa/?path=/mail/action/compose&to=" + encodeURIComponent(mail);
-    if (subject != "") outlookUrl = outlookUrl.concat("&subject=" + subject);
+    if (subject) outlookUrl = outlookUrl.concat("&subject=" + subject);
-    if (bodyMail != "") outlookUrl = outlookUrl.concat("&body=" + bodyMail);
+    if (bodyMail) outlookUrl = outlookUrl.concat("&body=" + bodyMail);
     window.open(outlookUrl, "_blank");
   },
   openDefault: function openDefault(encEmail) {
@@ -265,7 +265,7 @@ var isMailgo = function isMailgo(element) {
   return (// first case: it is an <a> element with "mailto:..." in href and no no-mailgo in classList
     element.href && element.href.toLowerCase().startsWith(MAILTO) && !element.classList.contains("no-mailgo") || // second case: the href=#mailgo
     element.href && element.getAttribute("href").toLowerCase() === "#mailgo" || // third case: the classList contains mailgo
-    element.classList.contains("mailgo")
+    element.classList && element.classList.contains("mailgo")
   );
 };
 /**
@@ -279,16 +279,17 @@ var isMailgo = function isMailgo(element) {
 
 var mailgoCheckRender = function mailgoCheckRender(event) {
   // check if the id=mailgo exists in the body
-  if (!document.contains(getE("mailgo"))) return; // go in the event.path to find if the user has clicked on a mailgo element
+  if (!document.contains(getE("mailgo"))) return;
-
+  event.path.forEach(function (element) {
-  if (event.path.some(isMailgo)) {
+    // go in the event.path to find if the user has clicked on a mailgo element
-    // stop the normal execution of the element click
+    if (isMailgo(element)) {
-    event.preventDefault(); // render mailgo
+      // stop the normal execution of the element click
-
+      event.preventDefault(); // render mailgo
-    mailgoRender(e);
-    return;
-  }
 
+      mailgoRender(element);
+      return;
+    }
+  });
   return;
 };
 /**
@@ -297,15 +298,7 @@ var mailgoCheckRender = function mailgoCheckRender(event) {
  */
 
 
-var mailgoKeydown = function mailgoKeydown(mail) {
+var mailgoKeydown = function mailgoKeydown(mail, cc, bcc, subject, bodyMail, url, mailtoHref, encEmail, copyButton) {
-  var cc = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : "";
-  var bcc = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : "";
-  var bodyMail = arguments.length > 3 && arguments[3] !== undefined ? arguments[3] : "";
-  var subject = arguments.length > 4 && arguments[4] !== undefined ? arguments[4] : "";
-  var url = arguments.length > 5 ? arguments[5] : undefined;
-  var mailtoHref = arguments.length > 6 ? arguments[6] : undefined;
-  var encEmail = arguments.length > 7 ? arguments[7] : undefined;
-  var copyButton = arguments.length > 8 ? arguments[8] : undefined;
   // if mailgo is not showing do nothing
   if (!mailgoIsShowing()) return;
 

src/mailgo.js

@@ -291,7 +291,18 @@ const mailgoRender = mailgo => {
   // listener keyDown
   mailgo.addEventListener(
     "keydown",
-    () => mailgoKeydown(mail, url, mailtoHref, encEmail, copyButton),
+    () =>
+      mailgoKeydown(
+        mail,
+        cc,
+        bcc,
+        subject,
+        bodyMail,
+        url,
+        mailtoHref,
+        encEmail,
+        copyButton
+      ),
     false
   );
 };
@@ -310,8 +321,8 @@ const actions = {
     let outlookUrl =
       "https://outlook.live.com/owa/?path=/mail/action/compose&to=" +
       encodeURIComponent(mail);
-    if (subject != "") outlookUrl = outlookUrl.concat("&subject=" + subject);
+    if (subject) outlookUrl = outlookUrl.concat("&subject=" + subject);
-    if (bodyMail != "") outlookUrl = outlookUrl.concat("&body=" + bodyMail);
+    if (bodyMail) outlookUrl = outlookUrl.concat("&body=" + bodyMail);
 
     window.open(outlookUrl, "_blank");
   },
@@ -336,7 +347,7 @@ const isMailgo = element =>
   // second case: the href=#mailgo
   (element.href && element.getAttribute("href").toLowerCase() === "#mailgo") ||
   // third case: the classList contains mailgo
-  element.classList.contains("mailgo");
+  (element.classList && element.classList.contains("mailgo"));
 
 /**
  * mailgoCheckRender
@@ -349,16 +360,18 @@ const mailgoCheckRender = event => {
   // check if the id=mailgo exists in the body
   if (!document.contains(getE("mailgo"))) return;
 
-  // go in the event.path to find if the user has clicked on a mailgo element
+  event.path.forEach(element => {
-  if (event.path.some(isMailgo)) {
+    // go in the event.path to find if the user has clicked on a mailgo element
-    // stop the normal execution of the element click
+    if (isMailgo(element)) {
-    event.preventDefault();
+      // stop the normal execution of the element click
+      event.preventDefault();
 
-    // render mailgo
+      // render mailgo
-    mailgoRender(e);
+      mailgoRender(element);
 
-    return;
+      return;
-  }
+    }
+  });
 
   return;
 };
@@ -369,10 +382,10 @@ const mailgoCheckRender = event => {
  */
 const mailgoKeydown = (
   mail,
-  cc = "",
+  cc,
-  bcc = "",
+  bcc,
-  bodyMail = "",
+  subject,
-  subject = "",
+  bodyMail,
   url,
   mailtoHref,
   encEmail,