Merge pull request #53 from hankxdev/fix-context-menu

fix context menu

Author: hankxdev

package-lock.json

@@ -67,6 +67,7 @@
 		"dot-json": "^1.2.0",
 		"eslint": "^6.8.0",
 		"eslint-plugin-svelte3": "^2.7.3",
+		"husky": "^4.2.5",
 		"npm-run-all": "^4.1.5",
 		"rollup": "^2.6.1",
 		"rollup-plugin-copy-glob": "^0.3.1",
@@ -77,5 +78,10 @@
 		"svelte": "^3.20.1",
 		"web-ext": "^4.1.0",
 		"xo": "^0.29.1"
+	},
+	"husky":{
+		"hooks":{
+			"pre-commit": "npm run fix"
+		}
 	}
 }

package.json

@@ -71,12 +71,12 @@
 		});
 	});
 
-	// Show extra buttons on right click on the name
+	// Toggle extra buttons on right click on the name
 	let onContextMenu;
-	$: onContextMenu = showExtras ? () => {} : (event => {
-		showExtras = true;
+	$: onContextMenu = event => {
+		showExtras = !showExtras;
 		event.preventDefault();
-	});
+	};
 </script>
 
 <svelte:window on:keydown={keyboardNavigationHandler}/>

source/App.svelte

@@ -49,9 +49,7 @@
 	<button type="button" class="ext-name" on:click={toggleExtension} on:contextmenu>
 		<img alt="" src={getIcon(icons, 16)}>{name}
 	</button>
-
-	{#if enabled}
-		{#if optionsUrl}
+		{#if  showExtras || (optionsUrl && enabled)}
 			<a href='chrome://extensions/?options={id}' title={getI18N('gotoOpt')} on:click={openInTab}>
 				<img src="icons/options.svg" alt="">
 			</a>
@@ -69,5 +67,4 @@
 				<img src="icons/bin.svg" alt="">
 			</button>
 		{/if}
-	{/if}
 </li>

source/Extension.svelte

@@ -1,5 +1,5 @@
 /** Required for chrome:// links */
 export default function openInTab(event) {
-	browser.tabs.create({url: event.target.href});
+	browser.tabs.create({url: event.currentTarget.href});
 	event.preventDefault();
 }