Merge pull request #173 from ajayyy/experimental-ajay

Made the popup work on non YouTube tabs

README.md

@@ -1,5 +1,5 @@
 <p align="center">
-  <img src="icons/LogoSponsorBlocker256px.png" alt="Logo"></img>
+  <a href="https://sponsor.ajay.app"><img src="icons/LogoSponsorBlocker256px.png" alt="Logo"></img></a>
   
   <br/>
   <sub>Logo by <a href="https://github.com/munadikieh">@munadikieh</a></sub>
@@ -8,12 +8,18 @@
 <h1 align="center">SponsorBlock</h1>
 
 <p align="center">
+  <b>Download:</b>
   <a href="https://chrome.google.com/webstore/detail/mnjggcdmjocbbbhaepdhchncahnbgone">Chrome/Chromium</a> |
   <a href="https://addons.mozilla.org/addon/sponsorblock/?src=external-github">Firefox</a> |
   <a href="https://sponsor.ajay.app">Website</a> |
   <a href="https://sponsor.ajay.app/stats">Stats</a>
 </p>
 
+<p align="center">
+  <b>Unofficial Ports:</b>
+  <a href="https://github.com/ajayyy/SponsorBlock/wiki/Unofficial-Ports#mpv-media-player">MPV</a>
+</p>
+
 <p align="center">
     <a href="https://addons.mozilla.org/addon/sponsorblock/?src=external-github"><img src="https://img.shields.io/amo/users/sponsorblock?label=Firefox%20Users" alt="Badge"></img></a>
     <a href="https://chrome.google.com/webstore/detail/mnjggcdmjocbbbhaepdhchncahnbgone"><img src="https://img.shields.io/chrome-web-store/users/mnjggcdmjocbbbhaepdhchncahnbgone?label=Chome%20Users" alt="Badge"></img></a>
@@ -48,6 +54,8 @@ You can load this project as an unpacked extension. Make sure to rename the `con
 
 The awesome [Invidious API](https://github.com/omarroth/invidious/wiki/API) is used to grab the time the video was published.
 
+Original code from [YTSponsorSkip](https://github.com/OfficialNoob/YTSponsorSkip), but not much of the code is left.
+
 Some icons made by <a href="https://www.flaticon.com/authors/gregor-cresnar" title="Gregor Cresnar">Gregor Cresnar</a> from <a href="https://www.flaticon.com/" title="Flaticon">www.flaticon.com</a> and are licensed by <a href="http://creativecommons.org/licenses/by/3.0/" title="Creative Commons BY 3.0" target="_blank">CC 3.0 BY</a>
 
 Some icons made by <a href="https://www.flaticon.com/authors/freepik" title="Freepik">Freepik</a> from <a href="https://www.flaticon.com/" title="Flaticon">www.flaticon.com</a> are licensed by <a href="http://creativecommons.org/licenses/by/3.0/" title="Creative Commons BY 3.0" target="_blank">CC 3.0 BY</a>

_locales/en/messages.json

@@ -100,7 +100,7 @@
         "message": "Sponsorship Ends Now"
     },
     "noVideoID": {
-        "message": "This probably isn't a YouTube tab, or you clicked too early. \n If you know this is a YouTube tab,\n close this popup and open it again.\n\nYou must refresh your YouTube tab after installing or updating (even auto-update) the extension."
+        "message": "No YouTube video found at this tab. If you know this is a YouTube tab, close this popup and open it again. If that does not work, try reloading the tab."
     },
     "success": {
         "message": "Success!"
@@ -141,9 +141,6 @@
     "removeFromWhitelist": {
          "message": "Remove Channel From Whitelist"
     },
-    "whitelistDescription": {
-         "message": "Whitelist the channels who do sponsorships ethically to encourage good behavior, or maybe if they are just entertaining and funny. Or don't, that's your call."
-    },
     "voteOnTime": {
          "message": "Vote On A Sponsor Time"
     },
@@ -193,10 +190,10 @@
         "message": "Options"
     },
     "showButtons": {
-        "message": "Hide Buttons On YouTube Player"
+        "message": "Show Buttons On YouTube Player"
     },
     "hideButtons": {
-        "message": "Show Buttons On YouTube Player"
+        "message": "Hide Buttons On YouTube Player"
     },
     "hideButtonsDescription": {
         "message": "This hides the buttons that appear on the YouTube player to submit sponsors. I can see this being annoying for some\n people. Instead of using the button there, this popup can be used to submit sponsors. To hide the notice that appears, \n use the button that appears on the notice saying \"Don't show this again\". You can always enable these settings again later."
@@ -262,5 +259,18 @@
     },
     "keybindDescriptionComplete": {
         "message": "The keybind has been set to: "
+    },
+    "0": {
+        "message": "Connection Timeout. Check your internet connection. If your internet is working, the server is probably overloaded or down."
+    },
+    "disableSkipping": {
+        "message": "Disable SponsorBlock"
+    },
+    "enableSkipping": {
+        "message": "Enable SponsorBlock"
+    },
+    "yourWork": {
+        "message": "Your Work",
+        "description": "Used to describe the section that will show you the statistics from your submissions."
     }
 }

_locales/fr/messages.json

@@ -193,10 +193,10 @@
         "message": "Options"
     },
     "showButtons": {
-        "message": "Cacher les boutons sur le lecteur YouTube"
+        "message": "Montrer les boutons sur le lecteur YouTube"
     },
     "hideButtons": {
-        "message": "Montrer les boutons sur le lecteur YouTube"
+        "message": "Cacher les boutons sur le lecteur YouTube"
     },
     "hideButtonsDescription": {
         "message": "Cela permet de cacher du lecteur YouTube les boutons utilisés pour soumettre des segments commerciaux. Je peux \ncomprendre que certaines personnes les trouvent perturbants. Au lieu d'utiliser ces boutons, cette fenêtre peut être utilisée \npour soumettre des segments commerciaux. Pour cacher la notification, utilisez le bouton \"Ne plus montrer\" sur la notification. Vous pouvez toujours réactiver ces paramètres plus tard."

_locales/it/messages.json

@@ -194,10 +194,10 @@
         "message": "Opzioni"
     },
     "showButtons": {
-        "message": "Nascondi i Pulsanti nel Lettore di YouTube"
+        "message": "Mostra i Pulsanti nel Lettore di YouTube"
     },
     "hideButtons": {
-        "message": "Mostra i Pulsanti nel Lettore di YouTube"
+        "message": "Nascondi i Pulsanti nel Lettore di YouTube"
     },
     "hideButtonsDescription": {
 		"message": "Nasconde i pulsanti che appaiono nel lettore di YouTube per inviare spezzoni sponsorizzati. Capisco che può essere fastidioso per alcune\n persone. Invece di utilizzare quei pulsanti, è possibile utilizzare questo popup per inviare gli spezzoni sponsorizzati. Per nascondere l'avviso che appare, \nusa il bottone \"Non mostrare più\" nell'avviso. Potrai sempre abilitare nuovamente queste impostazioni in futuro."

_locales/pt_BR/messages.json

@@ -194,10 +194,10 @@
         "message": "Opções"
     },
     "showButtons": {
-        "message": "Esconder botões no player do Youtube"
+        "message": "Mostrar botões no player do Youtube"
     },
     "hideButtons": {
-        "message": "Mostrar botões no player do Youtube"
+        "message": "Esconder botões no player do Youtube"
     },
     "hideButtonsDescription": {
         "message": "Isto esconde os botões que aparecem no player do Youtube para submeter patrocínios. Entendemos que possa ser\n incómodo a algumas pessoas. Em vez de usar esses botões pode usar os do popup. Para esconder a mensagem que aparece, \n ususe o botão na mesma que diz \"Don't show this again\". Pode sempre reactivar estas definições novamente."

_locales/pt_pt/messages.json

@@ -193,10 +193,10 @@
         "message": "Opções"
     },
     "showButtons": {
-        "message": "Esconder botões no player do Youtube"
+        "message": "Mostrar botões no player do Youtube"
     },
     "hideButtons": {
-        "message": "Mostrar botões no player do Youtube"
+        "message": "Esconder botões no player do Youtube"
     },
     "hideButtonsDescription": {
         "message": "Isto esconde os botões que aparecem no player do Youtube para submeter patrocínios. Entendemos que possa ser\n incómodo a algumas pessoas. Em vez de usar esses botões pode usar os do popup. Para esconder a mensagem que aparece, \n ususe o botão na mesma que diz \"Don't show this again\". Pode sempre reactivar estas definições novamente."

background.js

@@ -42,13 +42,14 @@ chrome.runtime.onMessage.addListener(function (request, sender, callback) {
 
 //add help page on install
 chrome.runtime.onInstalled.addListener(function (object) {
-    chrome.storage.sync.get(["userID", "shownInstallPage"], function(result) {
+    setTimeout(function() {
+        chrome.storage.sync.get(["userID"], function(result) {
             const userID = result.userID;
 
             // If there is no userID, then it is the first install.
             if (!userID){
                 //open up the install page
-            chrome.tabs.create({url: chrome.extension.getURL("/help/"+chrome.i18n.getMessage("helpPage"))});
+                chrome.tabs.create({url: chrome.extension.getURL("/help/index_en.html")});
 
                 //generate a userID
                 const newUserID = generateUserID();
@@ -58,6 +59,7 @@ chrome.runtime.onInstalled.addListener(function (object) {
                 });
             }
         });
+    }, 1500);
 });
 
 //gets the sponsor times from memory
@@ -157,6 +159,9 @@ function submitTimes(videoID, callback) {
 
             //submit these times
             for (let i = 0; i < sponsorTimes.length; i++) {
+                    //to prevent it from happeneing twice
+                    let increasedContributionAmount = false;
+
                     //submit the sponsorTime
                     sendRequestToServer("GET", "/api/postVideoSponsorTimes?videoID=" + videoID + "&startTime=" + sponsorTimes[i][0] + "&endTime=" + sponsorTimes[i][1]
                     + "&userID=" + userID, function(xmlhttp, error) {
@@ -175,7 +180,11 @@ function submitTimes(videoID, callback) {
                                     }
 
                                     //save the amount contributed
+                                    if (!increasedContributionAmount) {
+                                        increasedContributionAmount = true;
+                                        
                                         chrome.storage.sync.set({"sponsorTimesContributed": currentContributionAmount + sponsorTimes.length});
+                                    }
                                 });
                             }
                         } else if (error) {

content.js

@@ -67,6 +67,15 @@ var sponsorTimesSubmitting = [];
 //this is used to close the popup on YouTube when the other popup opens
 var popupInitialised = false;
 
+//should skips happen at all
+var disableSkipping = false;
+chrome.storage.sync.get(["disableSkipping"], function(result) {
+    let disableSkippingStorage = result.disableSkipping;
+    if (disableSkippingStorage != undefined) {
+        disableSkipping = disableSkippingStorage;
+    }
+});
+
 //should view counts be tracked
 var trackViewCount = false;
 chrome.storage.sync.get(["trackViewCount"], function(result) {
@@ -378,6 +387,18 @@ function sponsorsLookup(id, channelIDPromise) {
         v.addEventListener('durationchange', updatePreviewBar);
     }
 
+    if (channelIDPromise != null) {
+        if (channelIDPromise.isFulfilled) {
+            whitelistCheck();
+        } else if (channelIDPromise.isRejected) {
+            //try again
+            wait(getChannelID).then(whitelistCheck).catch();
+        } else {
+            //add it as a then statement
+            channelIDPromise.then(whitelistCheck);
+        }
+    }
+
     //check database for sponsor times
     //made true once a setTimeout has been created to try again after a server error
     let recheckStarted = false;
@@ -396,18 +417,6 @@ function sponsorsLookup(id, channelIDPromise) {
                 updatePreviewBar();
             }
 
-            if (channelIDPromise != null) {
-                if (channelIDPromise.isFulfilled) {
-                    whitelistCheck();
-                } else if (channelIDPromise.isRejected) {
-                    //try again
-                    wait(getChannelID).then(whitelistCheck).catch();
-                } else {
-                    //add it as a then statement
-                    channelIDPromise.then(whitelistCheck);
-                }
-            }
-
             sponsorLookupRetries = 0;
         } else if (xmlhttp.readyState == 4 && xmlhttp.status == 404) {
             sponsorDataFound = false;
@@ -439,10 +448,12 @@ function sponsorsLookup(id, channelIDPromise) {
     });
 
     //add the event to run on the videos "ontimeupdate"
+    if (!disableSkipping) {
         v.ontimeupdate = function () { 
             sponsorCheck();
         };
     }
+}
 
 function updatePreviewBar() {
     let localSponsorTimes = sponsorTimes;
@@ -525,29 +536,18 @@ function whitelistCheck() {
             UUIDs = [];
 
             channelWhitelisted = true;
-
-            //make sure the whitelistedChannels array isn't broken and full of null entries
-            //TODO: remove this at some point in the future as the bug that caused this should be patched
-            if (whitelistedChannels.some((el) => el === null)) {
-                //remove the entries that are null
-                let cleanWhitelistedChannelsArray = [];
-                for (let i = 0; i < whitelistedChannels.length; i++) {
-                    let channelURL = whitelistedChannels[i];
-                    if (channelURL !== null) {
-                        //add it
-                        cleanWhitelistedChannelsArray.push(channelURL);
-                    }
-                }
-
-                //save this value
-                chrome.storage.sync.set({"whitelistedChannels": cleanWhitelistedChannelsArray});
-            }
         }
     });
 }
 
 //video skipping
 function sponsorCheck() {
+    if (disableSkipping) {
+        // Make sure this isn't called again
+        v.ontimeupdate = null;
+        return;
+    }
+
     let skipHappened = false;
 
     if (sponsorTimes != null) {
@@ -1009,6 +1009,8 @@ function sendSubmitMessage(){
 
     let currentVideoID = sponsorVideoID;
 
+    let currentSponsorTimes = submitSponsorTimes;
+
     chrome.runtime.sendMessage({
         message: "submitTimes",
         videoID: currentVideoID
@@ -1032,17 +1034,21 @@ function sendSubmitMessage(){
 
                 //clear the sponsor times
                 let sponsorTimeKey = "sponsorTimes" + currentVideoID;
-                chrome.storage.sync.set({[sponsorTimeKey]: []});
+                chrome.storage.sync.set({[sponsorTimeKey]: []}, () => void updatePreviewBar());
 
-                //request the sponsors from the server again
+                //add submissions to current sponsors list
-                sponsorsLookup(currentVideoID);
+                sponsorTimes = sponsorTimes.concat(sponsorTimesSubmitting);
+                sponsorTimesSubmitting = [];
             } else {
                 //show that the upload failed
                 document.getElementById("submitButton").style.animation = "unset";
                 document.getElementById("submitImage").src = chrome.extension.getURL("icons/PlayerUploadFailedIconSponsorBlocker256px.png");
 
-                if([400,429,409,502].includes(response.statusCode)) {
+                if([400, 429, 409, 502, 0].includes(response.statusCode)) {
-                    alert(chrome.i18n.getMessage(response.statusCode));
+                    //treat them the same
+                    if (response.statusCode == 503) response.statusCode = 502;
+
+                    alert(chrome.i18n.getMessage(response.statusCode + ""));
                 } else {
                     alert(chrome.i18n.getMessage("connectionError") + response.statusCode);
                 }

manifest.json

@@ -1,7 +1,7 @@
 {
   "name": "__MSG_fullName__",
   "short_name": "__MSG_Name__",
-  "version": "1.1.9.3",
+  "version": "1.1.9.6",
   "default_locale": "en",
   "description": "__MSG_Description__",
   "content_scripts": [

popup.css

@@ -35,6 +35,10 @@ sub.popupElement {
 }
 /* end reset */
 
+#sponsorBlockPopupLogo {
+	vertical-align: text-bottom;
+}
+
 .popupElement {
     font-family: 'Source Sans Pro', sans-serif;
 
@@ -43,12 +47,13 @@ sub.popupElement {
 
 h1.popupElement {
 	margin-top: 0px;
+	margin-bottom: 10px;
 }
 
 .popupBody {
 	font-size: 14px;
 	background-color: #ffd9d9;
-	padding: 5px;
+	padding: 0px 5px;
 }
 
 .discreteLink.popupElement {

popup.html

@@ -8,12 +8,13 @@
   <body class="popupBody">
     <center>
       <div id="app" class="popupBody">
-          <img src="icons/LogoSponsorBlocker256px.png" height="64px" id="sponsorBlockPopupLogo"/>
+          <h1 class="popupElement">
-
+            <img src="icons/IconSponsorBlocker256px.png" height="32px" id="sponsorBlockPopupLogo"/>
-          <h1 class="popupElement">__MSG_Name__</h1>
+            __MSG_Name__
+          </h1>
 
           <!-- Loading text -->
-          <p id="loadingIndicator" class="popupElement">__MSG_Loading__</p>
+          <p id="loadingIndicator" class="popupElement">__MSG_noVideoID__</p>
           
           <!-- Hidden until loading complete -->
           <div id="mainControls" class="main popupElement" style="display: none">
@@ -26,17 +27,66 @@
   
             </div>
 
-            <br/>
+            <p class="popupElement"> 
+              __MSG_recordTimesDescription__
+            </p>
   
             <div>
+              <button id="sponsorStart" class="greenButton popupElement">__MSG_sponsorStart__</button>
+            </div>
+  
+            <sub class="popupElement">__MSG_popupHint__</sub>
+
+            <div id="submissionSection"  class="popupElement" style="display: none">
+              <h3 class="popupElement">__MSG_lastTimes__</h3>
+              <b>
+                <div id="sponsorMessageTimes" class="popupElement">
+        
+                </div>
+              </b>
+
+              <br/>
+      
+              <button id="clearTimes" class="smallButton popupElement">__MSG_clearTimesButton__</button>
+      
+              <div id="submitTimesContainer" class="popupElement" style="display: none">
+
+                <br/>
+                <br/>
+
+                <button id="submitTimes" class="smallButton popupElement">__MSG_submitTimesButton__</button>
+
+                <div id="submitTimesInfoMessageContainer" class="popupElement" style="display: none">
+                  <h3 id="submitTimesInfoMessage" class="popupElement">
+
+                  </h3>
+                </div>
+
+              </div>
+
+            </div>
+
+            <div id="discordButtonContainer" class="popupElement" style="display: none">
+              <br/>
+
+              <a href="https://discord.gg/QnmVMpU" class="popupElement" target="_blank"><img src="https://www.logolynx.com/images/logolynx/1b/1bcc0f0aefe71b2c8ce66ffe8645d365.png" height="32px"/></a>
+
+              <br/>
+
+                __MSG_discordAdvert__
+
+              <br/>
+
+              <span id="hideDiscordButton" class="smallLink popupElement">__MSG_hideThis__</span>
+            </div>
+
+            <div>
+              <br/>
+
               <button id="whitelistChannel" class="whitelistButton popupElement">__MSG_whitelistChannel__</button>
               <button id="unwhitelistChannel" class="whitelistButton popupElement" style="display: none">__MSG_removeFromWhitelist__</button>
             </div>
-            <sub class="popupElement">
-              __MSG_whitelistDescription__
-            </sub>
 
-            <br/>
             <br/>
 
             <button id="reportAnIssue" class="dangerButton popupElement">__MSG_voteOnTime__</button>
@@ -51,7 +101,17 @@
 
             </div>
 
-            <h2 class="recordingSubtitle popupElement">__MSG_recordTimes__</h2>
+            <br/>
+            <br/>
+
+          </div>
+
+          <div>
+            <button id="disableSkipping" class="greenButton popupElement">__MSG_disableSkipping__</button>
+            <button id="enableSkipping" class="whitelistButton popupElement" style="display: none">__MSG_enableSkipping__</button>
+          </div>
+
+          <h2 class="recordingSubtitle popupElement">__MSG_yourWork__</h2>
 
           <p class="popupElement">
             <span id=sponsorTimesContributionsContainer  class="popupElement" style="display: none">
@@ -80,47 +140,7 @@
             </div>
           </p>
 
-            <p class="popupElement"> 
-              __MSG_recordTimesDescription__
-            </p>
-    
-            <div>
-              <button id="sponsorStart" class="greenButton popupElement">__MSG_sponsorStart__</button>
-            </div>
-
-            <sub class="popupElement">__MSG_popupHint__</sub>
-    
-            <div id="submissionSection"  class="popupElement" style="display: none">
-              <h3 class="popupElement">__MSG_lastTimes__</h3>
-              <b>
-                <div id="sponsorMessageTimes" class="popupElement">
-        
-                </div>
-              </b>
-
-              <br/>
-      
-              <button id="clearTimes" class="smallButton popupElement">__MSG_clearTimesButton__</button>
-      
-              <br/>
-              <br/>
-
-              <div id="submitTimesContainer" class="popupElement" style="display: none">
-                <button id="submitTimes" class="smallButton popupElement">__MSG_submitTimesButton__</button>
-
-                <div id="submitTimesInfoMessageContainer" class="popupElement" style="display: none">
-                  <h3 id="submitTimesInfoMessage" class="popupElement">
-
-                  </h3>
-                </div>
-
-              </div>
-      
-            </div>
-
           <div id="setUsernameContainer" class="popupElement">
-              <br/>
-              <br/>
   
             <button id="setUsernameButton" class="warningButton popupElement">__MSG_setUsername__</button>
             <br/>
@@ -130,35 +150,18 @@
           </div>
 
           <div id="setUsername" class="popupElement" style="display: none">
-                <br/>
-
               <h3>__MSG_setUsername__</h3>
 
               <div id="setUsernameStatusContainer" style="display: none">
                   <h2 id="setUsernameStatus"></h2>
               </div>
 
-
               <input id="usernameInput" hint="Username"></input>
 
               <br/>
               <br/>
 
-                <button id="submitUsername" class="warningButton popupElement">Submit Username</button>
+              <button id="submitUsername" class="warningButton popupElement">__MSG_setUsername__</button>
-            </div>
-
-            <div id="discordButtonContainer" class="popupElement" style="display: none">
-              <br/>
-
-              <a href="https://discord.gg/QnmVMpU" class="popupElement" target="_blank"><img src="https://www.logolynx.com/images/logolynx/1b/1bcc0f0aefe71b2c8ce66ffe8645d365.png" height="32px"/></a>
-
-              <br/>
-
-                __MSG_discordAdvert__
-
-              <br/>
-
-              <span id="hideDiscordButton" class="smallLink popupElement">__MSG_hideThis__</span>
           </div>
 
           <div id="optionsButtonContainer" class="popupElement">
@@ -229,7 +232,6 @@
               <button id="showNoticeAgain" style="display: none" class="dangerButton popupElement">__MSG_showNotice__</button>
           </div>
         </div>
-        </div>
     </center>
   </body>
 

popup.js

@@ -26,8 +26,12 @@ function runThePopup() {
     var SB = {};
 
     ["sponsorStart",
+    // Top toggles
     "whitelistChannel",
     "unwhitelistChannel",
+    "disableSkipping",
+    "enableSkipping",
+    // More controls
     "clearTimes",
     "submitTimes",
     "showNoticeAgain",
@@ -80,6 +84,8 @@ function runThePopup() {
     SB.sponsorStart.addEventListener("click", sendSponsorStartMessage);
     SB.whitelistChannel.addEventListener("click", whitelistChannel);
     SB.unwhitelistChannel.addEventListener("click", unwhitelistChannel);
+    SB.disableSkipping.addEventListener("click", () => toggleSkipping(true));
+    SB.enableSkipping.addEventListener("click", () => toggleSkipping(false));
     SB.clearTimes.addEventListener("click", clearTimes);
     SB.submitTimes.addEventListener("click", submitTimes);
     SB.showNoticeAgain.addEventListener("click", showNoticeAgain);
@@ -120,21 +126,30 @@ function runThePopup() {
         if (hideDiscordLink == undefined || !hideDiscordLink) {
             chrome.storage.sync.get(["hideDiscordLaunches"], function(result) {
                 let hideDiscordLaunches = result.hideDiscordLaunches;
-                //only if less than 5 launches
+                //only if less than 10 launches
                 if (hideDiscordLaunches == undefined || hideDiscordLaunches < 10) {
                     SB.discordButtonContainer.style.display = null;
           
                     if (hideDiscordLaunches == undefined) {
-                        hideDiscordButton = 1;
+                        hideDiscordLaunches = 1;
                     }
   
-                    chrome.storage.sync.set({"hideDiscordLaunches": hideDiscordButton + 1});
+                    chrome.storage.sync.set({"hideDiscordLaunches": hideDiscordLaunches + 1});
                 }
             });
         }
     });
 
-    //if the don't show notice again letiable is true, an option to 
+    //show proper disable skipping button
+    chrome.storage.sync.get(["disableSkipping"], function(result) {
+        let disableSkipping = result.disableSkipping;
+        if (disableSkipping != undefined && disableSkipping) {
+            SB.disableSkipping.style.display = "none";
+            SB.enableSkipping.style.display = "unset";
+        }
+    });
+
+    //if the don't show notice again variable is true, an option to 
     //  disable should be available
     chrome.storage.sync.get(["dontShowNotice"], function(result) {
         let dontShowNotice = result.dontShowNotice;
@@ -280,7 +295,7 @@ function runThePopup() {
   
             //remove loading text
             SB.mainControls.style.display = "unset"
-            SB.loadingIndicator.innerHTML = "";
+            SB.loadingIndicator.style.display = "none";
 
             if (request.found) {
                 SB.videoFound.innerHTML = chrome.i18n.getMessage("sponsorFound");
@@ -811,8 +826,11 @@ function runThePopup() {
                     } else {
                         let errorMessage = "";
                         
-                        if([400,429,409,502].includes(response.statusCode)) {
+                        if([400, 429, 409, 502, 0].includes(response.statusCode)) {
-                            errorMessage = chrome.i18n.getMessage(response.statusCode);
+                            //treat them the same
+                            if (response.statusCode == 503) response.statusCode = 502;
+        
+                            errorMessage = chrome.i18n.getMessage(response.statusCode + "");
                         } else {
                             errorMessage = chrome.i18n.getMessage("connectionError") + response.statusCode;
                         }
@@ -1109,8 +1127,10 @@ function runThePopup() {
             type: type,
             UUID: UUID
         }, function(response) {
+            console.log(response)
             if (response != undefined) {
                 //see if it was a success or failure
+                console.log(response)
                 if (response.successType == 1 || (response.successType == -1 && response.statusCode == 429)) {
                     //success (treat rate limits as a success)
                     addVoteMessage(chrome.i18n.getMessage("voted"), UUID)
@@ -1130,7 +1150,7 @@ function runThePopup() {
     }
   
     function hideDiscordButton() {
-        chrome.storage.sync.set({"hideDiscordLink": false});
+        chrome.storage.sync.set({"hideDiscordLink": true});
   
         SB.discordButtonContainer.style.display = "none";
     }
@@ -1246,6 +1266,24 @@ function runThePopup() {
         });
     }
 
+    /**
+     * Should skipping be disabled (visuals stay)
+     */
+    function toggleSkipping(disabled) {
+        chrome.storage.sync.set({"disableSkipping": disabled});
+
+        let hiddenButton = SB.disableSkipping;
+        let shownButton = SB.enableSkipping;
+
+        if (!disabled) {
+            hiddenButton = SB.enableSkipping;
+            shownButton = SB.disableSkipping;
+        }
+
+        shownButton.style.display = "unset";
+        hiddenButton.style.display = "none";
+    }
+
     function setKeybind(startSponsorKeybind) {
         document.getElementById("keybindButtons").style.display = "none";