From 27e47a741dd88c71ae0eb4313521fa37aeb9f0a8 Mon Sep 17 00:00:00 2001
From: Ayush Sharma <ayush.aceit@gmail.com>
Date: Sun, 10 Feb 2019 09:02:07 +0100
Subject: [PATCH] refactor: adds mail link to author component

---
 src/webui/components/Author/index.js     | 16 ++++++++++++++--
 src/webui/components/Developers/index.js |  4 ++--
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/webui/components/Author/index.js b/src/webui/components/Author/index.js
index effcda4ab..2fecf3320 100644
--- a/src/webui/components/Author/index.js
+++ b/src/webui/components/Author/index.js
@@ -21,17 +21,29 @@ class Authors extends Component<any, any> {
     );
   }
 
+  renderLinkForMail(email, avatarComponent) {
+    if (!email) {
+      return avatarComponent;
+    }
+    return (
+      <a href={`mailto:${email}`} target={'_top'}>
+        {avatarComponent}
+      </a>
+    );
+  }
+
   renderAuthor = ({ packageMeta }) => {
     const { author } = packageMeta.latest;
 
     if (!author) {
       return null;
     }
-
+    console.log(author);
+    const avatarComponent = <Avatar alt={author.name} src={author.avatar} />;
     return (
       <List subheader={<Heading variant={'subheading'}>{'Author'}</Heading>}>
         <AuthorListItem>
-          <Avatar alt={author.name} src={author.avatar} />
+          {this.renderLinkForMail(author.email, avatarComponent)}
           <ListItemText primary={author.name} />
         </AuthorListItem>
       </List>
diff --git a/src/webui/components/Developers/index.js b/src/webui/components/Developers/index.js
index 49c454e6f..001c002f3 100644
--- a/src/webui/components/Developers/index.js
+++ b/src/webui/components/Developers/index.js
@@ -52,7 +52,7 @@ class Developers extends Component<Props, any> {
     );
   }
 
-  renderLinkForClick(email, avatar) {
+  renderLinkForMail(email, avatar) {
     if(!email) {
       return avatar;
     }
@@ -67,7 +67,7 @@ class Developers extends Component<Props, any> {
     const avatarComponent = <Avatar aria-label={name} src={avatar} />;
     return (
       <Tooltip title={name}>
-        {this.renderLinkForClick(email, avatarComponent)}
+        {this.renderLinkForMail(email, avatarComponent)}
       </Tooltip>
     );
   }