From d1a07dcd715ea504ab1ac07f9fdd0f229c6ed885 Mon Sep 17 00:00:00 2001
From: Yonatan Offek <yonatan.offek@gmail.com>
Date: Tue, 17 Sep 2019 11:43:15 +0300
Subject: [PATCH 1/2] Fix save button

---
 index.html | 2 +-
 paint.js   | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/index.html b/index.html
index 8374af1..00e3707 100644
--- a/index.html
+++ b/index.html
@@ -13,7 +13,7 @@
     <body>
         <script type="text/template" id="main">
             <div id="ui">
-                <div id="clear-button" class="button">Clear</div><div id="save-button" class="button">Save</div>
+                <div id="clear-button" class="button">Clear</div><a id="save-button" class="button">Save</a>
 
                 <div class="control-label">Quality</div>
                 <div class="buttons" id="qualities"></div>
diff --git a/paint.js b/paint.js
index 7c0d3ea..638a1e3 100644
--- a/paint.js
+++ b/paint.js
@@ -913,6 +913,10 @@ var Paint = (function () {
     };
 
     Paint.prototype.save = function () {
+        //reset attributes so nothing gets saved if we hit an error somewhere
+        this.saveButton.removeAttribute('download');
+        this.saveButton.setAttribute('href', '#');
+
         //we first render the painting to a WebGL texture
 
         var wgl = this.wgl;
@@ -971,7 +975,8 @@ var Paint = (function () {
         imageData.data.set(savePixels);
         saveContext.putImageData(imageData, 0, 0);
 
-        window.open(saveCanvas.toDataURL());
+        this.saveButton.setAttribute('download', 'painting.png');
+        this.saveButton.setAttribute('href', saveCanvas.toDataURL());
     };
 
     Paint.prototype.onMouseMove = function (event) {

From 320add4baf814ffb988710f282c01fce838ac74d Mon Sep 17 00:00:00 2001
From: Yonatan Offek <yonatan.offek@gmail.com>
Date: Tue, 17 Sep 2019 12:34:44 +0300
Subject: [PATCH 2/2] Remove underline from button text

---
 paint.css | 1 +
 1 file changed, 1 insertion(+)

diff --git a/paint.css b/paint.css
index 0a4c157..acee479 100644
--- a/paint.css
+++ b/paint.css
@@ -41,6 +41,7 @@ body {
     cursor: pointer;
 
     pointer-events: auto;
+    text-decoration: none;
 }
 
 #clear-button {
