Remove extract-domain dependency (#105)

do the same using native `URL` and strip www
microlinkhq · May 9, 2018 · 03a28a5 · 03a28a5
1 parent 7e44352
commit 03a28a5
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/packages/react-microlink/package.json b/packages/react-microlink/package.json
@@ -33,7 +33,6 @@
     "previsualization"
   ],
   "dependencies": {
-    "extract-domain": "~2.0.3",
     "nanoclamp": "~1.2.6"
   },
   "devDependencies": {

diff --git a/packages/react-microlink/src/components/Card/CardContent.js b/packages/react-microlink/src/components/Card/CardContent.js
@@ -1,10 +1,18 @@
+/* global URL */
+
 import React from 'react'
 import styled, { css } from 'styled-components'
-import extractDomain from 'extract-domain'
 import CardText from './CardText'
 
 import { media } from '../../utils'
 
+const REGEX_STRIP_WWW = /^www\./
+
+const getHostname = href => {
+  const { hostname } = new URL(href)
+  return hostname.replace(REGEX_STRIP_WWW, '')
+}
+
 const isLarge = cardSize => cardSize === 'large'
 
 const largeContentStyle = css`
@@ -68,7 +76,7 @@ export default ({ title, description, url, cardSize, className }) => (
       <CardText lines={2}>{description}</CardText>
     </Description>
     <Url className='microlink_card__content_url'>
-      <CardText lines={1}>{extractDomain(url)}</CardText>
+      <CardText lines={1}>{getHostname(url)}</CardText>
     </Url>
   </Content>
 )
diff --git a/packages/react-microlink/stories/data.js b/packages/react-microlink/stories/data.js
@@ -9,7 +9,9 @@ export const urls = [
   'http://es.engadget.com/2017/10/23/meizu-m6-note-analisis-review-fotos',
   'https://www.nytimes.com/2017/09/19/learning/whats-going-on-in-this-graph-sept-19-2017.html',
   'https://gizmodo.com/drone-video-of-border-wall-prototypes-accidentally-show-1819710328',
-  'https://www.bbc.com/news/technology-40762328'
+  'https://www.bbc.com/news/technology-40762328',
+  'https://browserless.js.org',
+  'https://xn--vi8hl0c.ws/'
 ]
 
 export const urlsVideo = [