Restructure files, update dependencies, fix extension icon and avoid …

…unnecessary page scrolling Resolves #13
felladrin · Dec 19, 2020 · 9e67e08 · 9e67e08
1 parent 2929809
commit 9e67e08
Show file tree

Hide file tree

Showing 9 changed files with 2,220 additions and 905 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -13,9 +13,10 @@
     "url": "https://github.com/felladrin/linkedin-autoconnect-chrome-extension/issues"
   },
   "scripts": {
-    "build": "run-s remove-dist-folder build-popup build-tab copy-static-files-to-dist zip-dist-folder",
-    "build-popup": "npx parcel build src/popup/popup.html --no-source-maps",
-    "build-tab": "npx parcel build src/tab/tab.ts --no-source-maps",
+    "watch": "run-s build parcel-watch",
+    "build": "run-s remove-dist-folder parcel-build copy-static-files-to-dist zip-dist-folder",
+    "parcel-watch": "parcel watch src/popup/popup.html src/tab/tab.ts",
+    "parcel-build": "parcel build src/popup/popup.html src/tab/tab.ts --no-source-maps",
     "copy-static-files-to-dist": "shx cp -r \"static/*\" \"dist\"",
     "zip-dist-folder": "cd dist && bestzip linkedin-autoconnect-chrome-extension.zip *",
     "remove-dist-folder": "shx rm -rf dist"
@@ -24,7 +25,7 @@
     "@types/chrome": "^0.0.126",
     "bestzip": "^2.1.7",
     "npm-run-all": "^4.1.5",
-    "parcel": "^2.0.0-beta.1",
+    "parcel": "^2.0.0-nightly.485",
     "shx": "^0.3.3"
   }
 }
diff --git a/src/popup/popup.html b/src/popup/popup.html
@@ -2,7 +2,7 @@
 <html>
   <head>
     <title>LinkedIn AutoConnect</title>
-    <link rel="stylesheet" href="popup.css" />
+    <link rel="stylesheet" href="./popup.css" />
   </head>
 
   <body>
@@ -31,6 +31,6 @@
     </div>
     <button id="startButton"><span>START</span></button>
     <button id="stopButton"><span>STOP</span></button>
-    <script src="popup.ts"></script>
+    <script src="./popup.ts"></script>
   </body>
 </html>
diff --git a/src/popup/popup.ts b/src/popup/popup.ts
@@ -16,7 +16,7 @@ state.startButton.addEventListener("click", () => {
       ExtensionMessage.IsAutoConnectAvailable,
       (response) => {
         if (!response) {
-          chrome.tabs.executeScript({ file: "tab.js" }, () => {
+          chrome.tabs.executeScript({ file: "tab/tab.js" }, () => {
             chrome.tabs.sendMessage(
               activeTab.id,
               ExtensionMessage.StartAutoConnect

diff --git a/src/tab/functions/addPeopleFromRecommendedForYouPage.ts b/src/tab/functions/addPeopleFromRecommendedForYouPage.ts
@@ -22,7 +22,6 @@ export function addPeopleFromRecommendedForYouPage() {
           item.focus();
           item.click();
           item.disabled = true;
-          window.scrollBy(0, 200);
         }
       }, alreadyInvited++ * delayBetweenClicks);
     }

diff --git a/src/tab/functions/addPeopleFromSearchPage.ts b/src/tab/functions/addPeopleFromSearchPage.ts
@@ -25,7 +25,6 @@ export function addPeopleFromSearchPage() {
           button.focus();
           button.click();
           button.disabled = true;
-          window.scrollBy(0, 200);
           clickSendNowButtonIfAvailable();
           dismissEmailRequiredDialog();
         }

diff --git a/static/icon_128x128.png → static/images/icon128.png b/static/icon_128x128.png → static/images/icon128.png
diff --git a/static/icon.png → static/images/icon16.png b/static/icon.png → static/images/icon16.png
diff --git a/static/manifest.json b/static/manifest.json
@@ -2,10 +2,13 @@
   "manifest_version": 2,
   "name": "LinkedIn AutoConnect",
   "description": "Automatically clicks all 'Connect' buttons available on LinkedIn 'Search People' and 'Recommended For You' pages.",
-  "version": "1.2.6",
+  "version": "1.2.7",
   "browser_action": {
-    "default_icon": "icon.png",
-    "default_popup": "popup.html"
+    "default_popup": "popup/popup.html"
+  },
+  "icons": {
+    "16": "images/icon16.png",
+    "128": "images/icon128.png"
   },
   "permissions": ["activeTab", "tabs", "*://*.linkedin.com/*"]
 }