Merge pull request #165 from NIAEFEUP/refactor/webpack

Use webpack for the build step

.gitignore

@@ -48,4 +48,13 @@ bundle/safari
 .vscode
 
 # Xcode project
-xcuserdata/
+xcuserdata/
+
+# Yarn
+.pnp.*
+.yarn/*
+!.yarn/patches
+!.yarn/plugins
+!.yarn/releases
+# !.yarn/sdks
+!.yarn/versions

README.md

@@ -1,34 +1,33 @@
-## Development / Building / Bundling the Extension
+## Development
 
 First, you must have [yarn](https://classic.yarnpkg.com/lang/en/docs/install) installed.
 
-Run
+Install dependencies:
+
 ```sh
-yarn && yarn bundle [all | firefox | chrome | safari]
+yarn install
 ```
-at the root directory and you'll get a bundle directory that looks like this:
 
+To develop choose your favorite browser.
+```sh
+yarn run dev:firefox
 ```
-📂 bundle
-└ 📁 chrome
-└ 📁 firefox
-└ 📁 safari
-└ 🗄️ chrome.zip
-└ 🗄️ firefox.zip
-└ 🗄️ safari.zip
+or
+```sh
+yarn run dev:chrome
 ```
 
-## Watch for changes
+This command will initialize a browser window and load the extension, watching for changes.
 
-```sh
-sudo yarn watch [firefox | chrome | safari]
-```
+You read to start!
 
-The extension polls the watch webserver for changes, if the page doesn't reload automatically, reload the extension and if that doesn't work, see the service worker/background script console for errors.
+## Building a bundle
 
-## Load Extension
+```sh
+yarn build
+```
 
-You can bundle the extension for `Chrome`, `Firefox`, and `Safari`:
+## Load a bundle
 
 <table>
 	<tr>
@@ -60,6 +59,4 @@ You can bundle the extension for `Chrome`, `Firefox`, and `Safari`:
 			</ol>
 		</td>
 	</tr>
-</table>
-
-#### This respository was based on the [typefully/minimal-twitter](https://github.com/typefully/minimal-twitter) extension repository
+</table>

background.js

@@ -8,13 +8,17 @@ const popupOptions = {
   font: "on",
 };
 
+const reloadFEUPSigarraPages = () => {
+  chrome.tabs.query({ url: "*://sigarra.up.pt/feup/*" }, (tabs) => {
+    tabs.forEach((tab) => {
+      chrome.tabs.reload(tab.id);
+    });
+  });
+}
+
 chrome.runtime.onInstalled.addListener((object) => {
   if (object.reason === "install") {
-    chrome.tabs.query({ url: "*://sigarra.up.pt/feup/*" }, (tabs) => {
-      tabs.forEach((tab) => {
-        chrome.tabs.reload(tab.id);
-      });
-    });
+    reloadFEUPSigarraPages()
 
     if (navigator.userAgent.toLowerCase().indexOf('firefox') > -1) {
       chrome.tabs.create({
@@ -30,6 +34,7 @@ chrome.runtime.onInstalled.addListener((object) => {
   }
 
   if (object.reason === "update") {
+    reloadFEUPSigarraPages();
     for (const opt in popupOptions) {
       if (chrome.storage.local.get(opt) == null)
         chrome.storage.local.set({[opt]: popupOptions[opt]});

content-scripts/src/pages/teacher_page.js

@@ -1,4 +1,4 @@
-import {groupSectionTitleAndContent, groupChildrenBySelector, moveChildrenToAncestor, moveChildrenToAncestor, removeTwoColumnTable} from "../modules/utilities/pageUtils"
+import {groupSectionTitleAndContent, groupChildrenBySelector, moveChildrenToAncestor, removeTwoColumnTable} from "../modules/utilities/pageUtils"
 
 
 const publicationWebsites = {