From b59c4702b245412961326d14e1384847ba083678 Mon Sep 17 00:00:00 2001
From: Robbie Antenesse <dev@alamantus.com>
Date: Wed, 29 May 2019 00:01:59 -0600
Subject: [PATCH] Use domain + pathname for public links instead of href

---
 src/js/account/render.js | 2 +-
 src/js/account/sync.js   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/js/account/render.js b/src/js/account/render.js
index c74b118..b5d7046 100644
--- a/src/js/account/render.js
+++ b/src/js/account/render.js
@@ -63,7 +63,7 @@ export function renderMakePublic() {
   </label>
   <p id="publicLinkDisplay" style="${!isPublic ? 'display:none;': ''}margin-left:20px;">
     <strong>Public Link:</strong><br>
-    <input readonly id="publicLink" value="${window.location.href + (externalID ? externalID.toString() : '')}">
+    <input readonly id="publicLink" value="${document.domain + window.location.pathname + (externalID ? externalID.toString() : '')}">
     <a class="small button" id="publicLinkCopy">Copy</a>
   </p>
   `;
diff --git a/src/js/account/sync.js b/src/js/account/sync.js
index 37bda23..5b4df0b 100644
--- a/src/js/account/sync.js
+++ b/src/js/account/sync.js
@@ -100,7 +100,7 @@ export function uploadWholeDictionary(asNew = false) {
       dictionary,
     }, remoteId => {
       window.currentDictionary.externalID = remoteId;
-      dictionary.getElementById('publicLink').value = window.location.href + remoteId.toString();
+      dictionary.getElementById('publicLink').value = document.domain + window.location.pathname + remoteId.toString();
       saveDictionary(false);
       addMessage('Dictionary Uploaded Successfully');
       renderChangeDictionaryOptions();