Update UI strings, remove autofillOnPageLoad enum

src/_locales/en/messages.json

@@ -892,25 +892,19 @@
     "message": "This is currently an experimental feature. Use at your own risk."
   },
   "defaultAutoFillOnPageLoad": {
-    "message": "Default auto-fill on page load setting"
-  },
-  "globalAutoFillOnPageLoadAlways": {
-    "message": "Always auto-fill"
-  },
-  "globalAutoFillOnPageLoadNever": {
-    "message": "Never auto-fill"
+    "message": "Default login setting"
   },
   "itemAutoFillOnPageLoad": {
-    "message": "Auto-fill On Page Load (if enabled)"
+    "message": "Auto-fill On Page Load (if enabled in Options)"
   },
-  "itemAutoFillOnPageLoadUseGlobal": {
+  "autoFillOnPageLoadUseDefault": {
     "message": "Use default setting"
   },
-  "itemAutoFillOnPageLoadAlways": {
-    "message": "Always auto-fill this login"
+  "autoFillOnPageLoadYes": {
+    "message": "Auto-fill"
   },
-  "itemAutoFillOnPageLoadNever": {
-    "message": "Never auto-fill this login"
+  "autoFillOnPageLoadNo": {
+    "message": "Do not auto-fill"
   },
   "commandOpenPopup": {
     "message": "Open vault popup"

src/content/autofiller.ts

@@ -3,8 +3,12 @@ document.addEventListener('DOMContentLoaded', event => {
     let filledThisHref = false;
     let delayFillTimeout: number;
 
-    setInterval(() => doFillIfNeeded(), 500);
-
+    const enabledKey = 'enableAutoFillOnPageLoad';
+    chrome.storage.local.get(enabledKey, (obj: any) => {
+        if (obj != null && obj[enabledKey] === true) {
+            setInterval(() => doFillIfNeeded(), 500);
+        }
+    });
     chrome.runtime.onMessage.addListener((msg: any, sender: any, sendResponse: Function) => {
         if (msg.command === 'fillForm' && pageHref === msg.url) {
             filledThisHref = true;

src/popup/settings/options.component.html

@@ -25,7 +25,7 @@
             <div class="sub-option">
                 <label for="defaultAutofill">{{'defaultAutoFillOnPageLoad' | i18n}}</label>
                 <select id="defaultAutofill" name="DefaultAutofill" [(ngModel)]="autoFillOnPageLoadDefault"
-                    [disabled]="!enableAutoFillOnPageLoad">
+                    (change)="updateAutoFillOnPageLoadDefault()" [disabled]="!enableAutoFillOnPageLoad">
                     <option *ngFor="let o of autoFillOnPageLoadOptions" [ngValue]="o.value">{{o.name}}</option>
                 </select>
             </div>

src/popup/settings/options.component.ts

@@ -67,8 +67,8 @@ export class OptionsComponent implements OnInit {
             { name: i18nService.t('fiveMinutes'), value: 300 },
         ];
         this.autoFillOnPageLoadOptions = [
-            { name: i18nService.t('globalAutoFillOnPageLoadAlways'), value: true },
-            { name: i18nService.t('globalAutoFillOnPageLoadNever'), value: false },
+            { name: i18nService.t('autoFillOnPageLoadYes'), value: true },
+            { name: i18nService.t('autoFillOnPageLoadNo'), value: false },
         ]
     }
 
@@ -76,6 +76,9 @@ export class OptionsComponent implements OnInit {
         this.enableAutoFillOnPageLoad = await this.storageService.get<boolean>(
             ConstantsService.enableAutoFillOnPageLoadKey);
         
+        this.autoFillOnPageLoadDefault = await this.storageService.get<boolean>(
+            ConstantsService.autoFillOnPageLoadDefaultKey) ?? false;
+
         this.disableAddLoginNotification = await this.storageService.get<boolean>(
             ConstantsService.disableAddLoginNotificationKey);
 
@@ -129,6 +132,10 @@ export class OptionsComponent implements OnInit {
         this.callAnalytics('Auto-fill Page Load', this.enableAutoFillOnPageLoad);
     }
 
+    async updateAutoFillOnPageLoadDefault() {
+        await this.storageService.save(ConstantsService.autoFillOnPageLoadDefaultKey, this.autoFillOnPageLoadDefault);
+    }
+
     async updateDisableFavicon() {
         await this.storageService.save(ConstantsService.disableFaviconKey, this.disableFavicon);
         await this.stateService.save(ConstantsService.disableFaviconKey, this.disableFavicon);