Merge pull request #266 from ajayyy/experimental

Options page improvements + Preview sponsors

manifest/manifest.json

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

public/_locales/en/messages.json

@@ -415,7 +415,7 @@
         "message": "Reset"
     },
     "customAddressError": {
-        "message": "This address is not in the right form. Make sure you have http:// or https:// at the begining and no slashes or sub-folders at the end."
+        "message": "This address is not in the right form. Make sure you have http:// or https:// at the begining and no trailing slashes."
     },
     "areYouSureReset": {
         "message": "Are you sure you would like to reset this?"

src/content.ts

@@ -379,8 +379,20 @@ function sponsorsLookup(id: string, channelIDPromise?) {
         if (xmlhttp.readyState == 4 && xmlhttp.status == 200) {
             sponsorDataFound = true;
 
-            sponsorTimes = JSON.parse(xmlhttp.responseText).sponsorTimes;
+            let recievedSponsorTimes = JSON.parse(xmlhttp.responseText).sponsorTimes;
-            UUIDs = JSON.parse(xmlhttp.responseText).UUIDs;
+            let recievedUUIDs = JSON.parse(xmlhttp.responseText).UUIDs;
+
+            // Check if any old submissions should be kept
+            for (let i = 0; i < UUIDs.length; i++) {
+                if (UUIDs[i] === null)  {
+                    // This is a user submission, keep it
+                    recievedSponsorTimes.push(sponsorTimes[i]);
+                    recievedUUIDs.push(UUIDs[i]);
+                }
+            }
+
+            sponsorTimes = recievedSponsorTimes;
+            UUIDs = recievedUUIDs;
 
             // Remove all submissions smaller than the minimum duration
             if (Config.config.minDuration !== 0) {
@@ -1091,10 +1103,12 @@ function sendSubmitMessage(){
                 Config.config.sponsorTimes.delete(currentVideoID);
 
                 //add submissions to current sponsors list
+                if (sponsorTimes === null) sponsorTimes = [];
+                
                 sponsorTimes = sponsorTimes.concat(sponsorTimesSubmitting);
                 for (let i = 0; i < sponsorTimesSubmitting.length; i++) {
-                    // Add some random IDs
+                    // Add placeholder IDs
-                    UUIDs.push(utils.generateUserID());
+                    UUIDs.push(null);
                 }
 
                 // Empty the submitting times

src/options.ts

@@ -50,6 +50,16 @@ async function init() {
                     switch (option) {
                         case "supportInvidious":
                             invidiousOnClick(checkbox, option);
+                            break;
+                        case "disableAutoSkip":
+                            if (!checkbox.checked) {
+                                // Enable the notice
+                                Config.config["dontShowNotice"] = false;
+                                
+                                let showNoticeSwitch = <HTMLInputElement> document.querySelector("[sync-option='dontShowNotice'] > label > label > input");
+                                showNoticeSwitch.checked = true;
+                            }
+
                             break;
                     }
                 });
@@ -357,14 +367,13 @@ function activatePrivateTextChange(element: HTMLElement) {
  * @param input Input server address
  */
 function validateServerAddress(input: string): string {
-    // Trim the last slash if needed
+    input = input.trim();
-    if (input.endsWith("/")) {
-        input = input.substring(0, input.length - 1);
-    }
 
-    // Isn't HTTP protocol or has extra slashes
+    // Trim the trailing slashes
-    if ((!input.startsWith("https://") && !input.startsWith("http://")) 
+    input = input.replace(/\/+$/, "");
-        || input.replace("://", "").includes("/")) {
+
+    // If it isn't HTTP protocol
+    if ((!input.startsWith("https://") && !input.startsWith("http://"))) {
 
         alert(chrome.i18n.getMessage("customAddressError"));