From 52100a9f4f2e5b5d3249934143fb9a7097f156f9 Mon Sep 17 00:00:00 2001 From: Kohei Yoshino Date: Thu, 21 Jun 2018 10:28:21 -0400 Subject: Bug 1469333 - Check attachment file size client-side and inform user of too large file before uploading it --- js/attachment.js | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'js') diff --git a/js/attachment.js b/js/attachment.js index f967f64d3..6d6dae58d 100644 --- a/js/attachment.js +++ b/js/attachment.js @@ -93,6 +93,21 @@ function DataFieldHandler() { } } } + + // Check the current file size (in KB) + const file_size = field_data.files[0].size / 1024; + const max_size = BUGZILLA.param.maxattachmentsize; + const invalid = file_size > max_size; + const message = invalid ? `This file (${(file_size / 1024).toFixed(1)} MB) is larger than the ` + + `maximum allowed size (${(max_size / 1024).toFixed(1)} MB).
Please consider uploading it ` + + `to an online file storage and sharing the link in a bug comment instead.` : ''; + const message_short = invalid ? 'File too large' : ''; + const $error = document.querySelector('#data-error'); + + // Show an error message if the file is too large + $error.innerHTML = message; + field_data.setCustomValidity(message_short); + field_data.setAttribute('aria-invalid', invalid); } function clearAttachmentFields() { -- cgit v1.2.3-24-g4f1b