quran#341 spelling mistake + slight refactor.

thabti · Jul 2, 2016 · d4046dd · d4046dd
1 parent 9763333
commit d4046dd
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/src/components/Share/index.js b/src/components/Share/index.js
@@ -1,24 +1,26 @@
 import React, { Component } from 'react'
-import Style from './style.scss'
-
+let Style = require('./style.scss');
 export default class Share extends Component {
 
+  onClickPopup(url, title)  {
+    window.open(url, title, `width=670,height=540,scrollbars=no,toolbar=0`);
+  }
+
   render() {
+
     let {surahId, name} = this.props.surah;
-    let url = encodeURIComponent(`http://quran.com/${surahId}`);
 
-    let facebookUl = `https://www.facebook.com/sharer/sharer.php?u=${url}`;
-    let twitterUrl = `https://twitter.com/intent/tweet?url=${url}&text=Surat ${name.simple}`;
+    let surahUrl = encodeURIComponent(`http://quran.com/${surahId}`);
 
     return (
       <div className={Style.share}>
 
-        <i onClick={()=> this.onClickPopup(facebookUl, "Facebook")}
+        <i onClick={()=> this.onClickPopup(`https://www.facebook.com/sharer/sharer.php?u=${surahUrl}`, "Facebook")}
            className={Style.facebook}
            data-metrics-event-name="Share:Facebook"
            title="Share on Facebook"></i>
 
-        <i onClick={()=> this.onClickPopup(twitterUrl, "Twitter")}
+        <i onClick={()=> this.onClickPopup(`https://twitter.com/intent/tweet?url=${surahUrl}&text=Surat ${name.simple}`, "Twitter")}
            className={Style.twitter}
            data-metrics-event-name="Share:Twitter"
            title="Share on Twitter"></i>
@@ -27,9 +29,6 @@ export default class Share extends Component {
     )
   }
 
-  onClickPopup(url, title)  {
-    window.open(url, title, `width=670,height=540,scrollbars=no,toolbar=0`);
-  }
 
-}
 
+}