From b068a4c05504c4b5104a97c99f23f899e7dfb301 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=A0imon=20Brandner?= <simon.bra.ag@gmail.com>
Date: Wed, 24 Feb 2021 16:28:12 +0100
Subject: [PATCH] Make download into a button
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
---
 src/components/views/elements/ImageView.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/src/components/views/elements/ImageView.js b/src/components/views/elements/ImageView.js
index 28a087f77d..8251af1a5d 100644
--- a/src/components/views/elements/ImageView.js
+++ b/src/components/views/elements/ImageView.js
@@ -169,6 +169,15 @@ export default class ImageView extends React.Component {
         });
     }
 
+    onDownloadClick = (ev) => {
+        ev.preventDefault();
+        ev.stopPropagation();
+        const a = document.createElement("a");
+        a.href = this.props.src;
+        a.download = this.props.name;
+        a.click();
+    }
+
     onPanelClick = (ev) => {
         this.props.onFinished();
     }
@@ -281,13 +290,11 @@ export default class ImageView extends React.Component {
                                     title={_t("Zoom in")}
                                     onClick={ this.onZoomInClick }>
                                 </AccessibleTooltipButton>
-                                <a
+                                <AccessibleTooltipButton
                                     className="mx_ImageView_button mx_ImageView_button_download"
-                                    href={ this.props.src }
-                                    download={ this.props.name }
                                     title={_t("Download")}
-                                    target="_blank" rel="noopener">
-                                </a>
+                                    onClick={ this.onDownloadClick }>
+                                </AccessibleTooltipButton>
                                 <AccessibleTooltipButton
                                     className="mx_ImageView_button mx_ImageView_button_close"
                                     title={_t("Close")}