From ef8eef75195205ed0db4f7582bbdbc045587f042 Mon Sep 17 00:00:00 2001 From: Andrew Patton Date: Fri, 25 Jul 2014 17:51:18 -0400 Subject: [PATCH] Trivial cleanup using `event.preventDefault` instead of `return false` for being more explicit --- public/js/app.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/public/js/app.js b/public/js/app.js index 031c58245a..dba83f384c 100644 --- a/public/js/app.js +++ b/public/js/app.js @@ -726,7 +726,7 @@ function initIssue() { return false; }); - fileInput.addEventListener("change", function(event) { + fileInput.addEventListener("change", function() { for (var index = 0; index < fileInput.files.length; index++) { var file = fileInput.files[index]; @@ -752,9 +752,9 @@ function initIssue() { this.value = ""; }); - $addButton.on("click", function() { + $addButton.on("click", function(evt) { fileInput.click(); - return false; + evt.preventDefault(); }); }());