From eebeec509cb6a857d6e833d8a4a3c5354b9b164f Mon Sep 17 00:00:00 2001
From: David Lechner <david@pybricks.com>
Date: Tue, 26 Jan 2021 17:41:44 -0600
Subject: [PATCH] don't focus buttons on click

This prevents the navbar buttons from gaining focus when clicked.
This way it prevents losing focus from the editor or terminal.
It also prevents the settings button from being highlighted after
the settings menu is closed.
---
 src/components/ActionButton.tsx   | 1 +
 src/components/OpenFileButton.tsx | 1 +
 2 files changed, 2 insertions(+)

diff --git a/src/components/ActionButton.tsx b/src/components/ActionButton.tsx
index 7377dcb6..4b21ecde 100644
--- a/src/components/ActionButton.tsx
+++ b/src/components/ActionButton.tsx
@@ -51,6 +51,7 @@ class ActionButton extends React.Component<Props> {
                 <Button
                     ref={this.buttonRef}
                     intent={Intent.PRIMARY}
+                    onMouseDown={(e) => e.preventDefault()} // prevent focus
                     onClick={(): void => this.props.onAction()}
                     disabled={this.props.enabled === false}
                     className="no-box-shadow"
diff --git a/src/components/OpenFileButton.tsx b/src/components/OpenFileButton.tsx
index 28402120..bf8ac93c 100644
--- a/src/components/OpenFileButton.tsx
+++ b/src/components/OpenFileButton.tsx
@@ -114,6 +114,7 @@ class OpenFileButton extends React.Component<Props> {
                                         ? { pointerEvents: 'none' }
                                         : undefined
                                 }
+                                onMouseDown={(e) => e.preventDefault()} // prevent focus
                                 // onClick={this.props.onClick}
                                 // breaks Dropzone when this.props.onClick is undefined
                                 // so we have to do it the long way