From 27fd18bea080d0cfb00f9bb55b5f654a7e3c8ce5 Mon Sep 17 00:00:00 2001
From: Uku Taht <Uku.taht@gmail.com>
Date: Fri, 9 Oct 2020 15:23:08 +0300
Subject: [PATCH] Linkify top pages (#350)

* Linkify top pages

* Add changelog entry

* Make automated build push :dev tag to Dockerhub
---
 CHANGELOG.md                                           | 7 ++++++-
 assets/js/dashboard/stats/modals/referrer-drilldown.js | 4 ++--
 assets/js/dashboard/stats/pages.js                     | 7 ++++++-
 assets/js/dashboard/stats/sources/referrer-list.js     | 4 ++--
 docker_push                                            | 4 ++--
 5 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index a3ed73945..dc275c0ca 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,7 +1,12 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
-## [1.0.0] - Unreleased
+## [1.1.0] - Unreleased
+
+### Added
+- Linkify top pages [plausible/analytics#91](https://github.com/plausible/analytics/issues/91)
+
+## [1.0.0] - 2020-10-06
 
 ### Added
 - Collect and present link tags (`utm_medium`, `utm_source`, `utm_campaign`) in the dashboard
diff --git a/assets/js/dashboard/stats/modals/referrer-drilldown.js b/assets/js/dashboard/stats/modals/referrer-drilldown.js
index f68b83d1f..49b2591e5 100644
--- a/assets/js/dashboard/stats/modals/referrer-drilldown.js
+++ b/assets/js/dashboard/stats/modals/referrer-drilldown.js
@@ -51,7 +51,7 @@ class ReferrerDrilldownModal extends React.Component {
   renderExternalLink(name) {
     if (name !== 'Direct / None') {
       return (
-        <a target="_blank" href={'//' + name}>
+        <a target="_blank" href={'//' + name} className="hidden group-hover:block">
           <svg className="inline h-4 w-4 ml-1 -mt-1 text-gray-600" fill="currentColor" viewBox="0 0 20 20"><path d="M11 3a1 1 0 100 2h2.586l-6.293 6.293a1 1 0 101.414 1.414L15 6.414V9a1 1 0 102 0V4a1 1 0 00-1-1h-5z"></path><path d="M5 5a2 2 0 00-2 2v8a2 2 0 002 2h8a2 2 0 002-2v-3a1 1 0 10-2 0v3H5V7h3a1 1 0 000-2H5z"></path></svg>
         </a>
       )
@@ -63,7 +63,7 @@ class ReferrerDrilldownModal extends React.Component {
     query.set('referrer', name)
 
     return (
-      <span className="flex">
+      <span className="flex group">
         <Link className="block truncate hover:underline" to={{search: query.toString(), pathname: '/' + this.props.site.domain}} title={name}>
           {name}
         </Link>
diff --git a/assets/js/dashboard/stats/pages.js b/assets/js/dashboard/stats/pages.js
index cb831a58b..c504d9659 100644
--- a/assets/js/dashboard/stats/pages.js
+++ b/assets/js/dashboard/stats/pages.js
@@ -46,7 +46,12 @@ export default class Pages extends React.Component {
       <div className="flex items-center justify-between my-1 text-sm" key={page.name}>
         <div className="w-full h-8 truncate" style={{maxWidth: 'calc(100% - 4rem)'}}>
           <Bar count={page.count} all={this.state.pages} bg="bg-orange-50" />
-          <Link to={{search: query.toString()}} className="block px-2 hover:underline" style={{marginTop: '-26px'}}>{page.name}</Link>
+          <span className="flex px-2 group" style={{marginTop: '-26px'}} >
+            <Link to={{search: query.toString()}} className="block hover:underline">{page.name}</Link>
+            <a target="_blank" href={'http://' + this.props.site.domain + page.name} className="hidden group-hover:block">
+              <svg className="inline h-4 w-4 ml-1 -mt-1 text-gray-600" fill="currentColor" viewBox="0 0 20 20"><path d="M11 3a1 1 0 100 2h2.586l-6.293 6.293a1 1 0 101.414 1.414L15 6.414V9a1 1 0 102 0V4a1 1 0 00-1-1h-5z"></path><path d="M5 5a2 2 0 00-2 2v8a2 2 0 002 2h8a2 2 0 002-2v-3a1 1 0 10-2 0v3H5V7h3a1 1 0 000-2H5z"></path></svg>
+            </a>
+          </span>
         </div>
         <span className="font-medium">{numberFormatter(page.count)}</span>
       </div>
diff --git a/assets/js/dashboard/stats/sources/referrer-list.js b/assets/js/dashboard/stats/sources/referrer-list.js
index 99bab9b9a..9eee5044f 100644
--- a/assets/js/dashboard/stats/sources/referrer-list.js
+++ b/assets/js/dashboard/stats/sources/referrer-list.js
@@ -56,7 +56,7 @@ export default class Referrers extends React.Component {
   renderExternalLink(referrer) {
     if (this.props.query.filters.source && this.props.query.filters.source !== 'Google' && referrer.name !== 'Direct / None') {
       return (
-        <a target="_blank" href={'//' + referrer.name}>
+        <a target="_blank" href={'//' + referrer.name} className="hidden group-hover:block">
           <svg className="inline h-4 w-4 ml-1 -mt-1 text-gray-600" fill="currentColor" viewBox="0 0 20 20"><path d="M11 3a1 1 0 100 2h2.586l-6.293 6.293a1 1 0 101.414 1.414L15 6.414V9a1 1 0 102 0V4a1 1 0 00-1-1h-5z"></path><path d="M5 5a2 2 0 00-2 2v8a2 2 0 002 2h8a2 2 0 002-2v-3a1 1 0 10-2 0v3H5V7h3a1 1 0 000-2H5z"></path></svg>
         </a>
       )
@@ -72,7 +72,7 @@ export default class Referrers extends React.Component {
       <div className="flex items-center justify-between my-1 text-sm" key={referrer.name}>
         <div className="w-full h-8" style={{maxWidth: 'calc(100% - 4rem)'}}>
           <Bar count={referrer.count} all={this.state.referrers} bg="bg-blue-50" />
-          <span className="flex px-2" style={{marginTop: '-26px'}} >
+          <span className="flex px-2 group" style={{marginTop: '-26px'}} >
             <LinkOption className="block truncate" to={{search: query.toString()}} disabled={referrer.name === 'Direct / None'}>
               <img src={`https://icons.duckduckgo.com/ip3/${referrer.url}.ico`} className="inline h-4 w-4 mr-2 align-middle -mt-px" />
               { referrer.name }
diff --git a/docker_push b/docker_push
index 289a713f4..2d01a2967 100755
--- a/docker_push
+++ b/docker_push
@@ -1,4 +1,4 @@
 #!/bin/bash
 echo "$DOCKER_PASSWORD" | docker login -u "$DOCKER_USERNAME" --password-stdin
-docker build -t plausible/analytics .
-docker push plausible/analytics
+docker build -t plausible/analytics:dev .
+docker push plausible/analytics:dev