From 400fd77d0afae6410e8c78053288ae9fcf6349d2 Mon Sep 17 00:00:00 2001
From: Antony O'Neill <antony@boclips.com>
Date: Thu, 4 Jul 2019 19:02:22 +0100
Subject: [PATCH] Prevent default on settings icon click

---
 src/js/listeners.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/js/listeners.js b/src/js/listeners.js
index c5076ff3..bcb3571e 100644
--- a/src/js/listeners.js
+++ b/src/js/listeners.js
@@ -603,12 +603,19 @@ class Listeners {
         this.bind(elements.buttons.airplay, 'click', player.airplay, 'airplay');
 
         // Settings menu - click toggle
-        this.bind(elements.buttons.settings, 'click', event => {
-            // Prevent the document click listener closing the menu
-            event.stopPropagation();
+        this.bind(
+            elements.buttons.settings,
+            'click',
+            event => {
+                // Prevent the document click listener closing the menu
+                event.stopPropagation();
+                event.preventDefault();
 
-            controls.toggleMenu.call(player, event);
-        });
+                controls.toggleMenu.call(player, event);
+            },
+            null,
+            false
+        ); // Can't be passive as we're preventing default
 
         // Settings menu - keyboard toggle
         // We have to bind to keyup otherwise Firefox triggers a click when a keydown event handler shifts focus