Options weren't working correctly, needed to prevent default even upo…

…n form submit.
Inirit · Nov 23, 2017 · 709a8a3 · 709a8a3
1 parent 45f0483
commit 709a8a3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/ts/options/ExtensionOptions.ts b/ts/options/ExtensionOptions.ts
@@ -1,7 +1,5 @@
 import { HandlerType } from '../handlers/HandlerType';
 
-
-
 export interface IExtensionOptions
 {
 	HandlerType: HandlerType;
@@ -12,7 +10,7 @@ export interface IExtensionOptions
 export class ExtensionOptions implements IExtensionOptions
 {
 	private static readonly _defaultOptions: browser.storage.StorageObject = {
-		"HandlerType": HandlerType.Imgur
+		"HandlerType": `${HandlerType.Imgur}`
 	};
 
 	private readonly _currentOptions: browser.storage.StorageObject;
@@ -50,7 +48,7 @@ export class ExtensionOptions implements IExtensionOptions
 
 	set HandlerType(value: HandlerType)
 	{
-		this._currentOptions["HandlerType"] = value;
+		this._currentOptions["HandlerType"] = `${value}`;
 	}
 
 	get RawStorage(): browser.storage.StorageObject

diff --git a/ts/options/OptionsMain.ts b/ts/options/OptionsMain.ts
@@ -4,9 +4,11 @@ import * as $ from 'jquery';
 
 async function SaveOptions(e)
 {
+	e.preventDefault();
+
 	const currentOptions = await ExtensionOptions.GetCurrentOptions();
 
-	currentOptions.HandlerType = $("#handlers").val() as number;
+	currentOptions.HandlerType = $("#handlers").val() as HandlerType;
 
 	await ExtensionOptions.UpdateCurrentOptions(currentOptions);
 }