Commit 0c49b5eb authored by Jose Ivan Vargas's avatar Jose Ivan Vargas

Fix cancel button not working when uploading a file on the new issue page

parent 0b8b631a
...@@ -119,11 +119,9 @@ export default function dropzoneInput(form) { ...@@ -119,11 +119,9 @@ export default function dropzoneInput(form) {
// removeAllFiles(true) stops uploading files (if any) // removeAllFiles(true) stops uploading files (if any)
// and remove them from dropzone files queue. // and remove them from dropzone files queue.
$cancelButton.on('click', (e) => { $cancelButton.on('click', (e) => {
const target = e.target.closest('.js-main-target-form').querySelector('.div-dropzone');
e.preventDefault(); e.preventDefault();
e.stopPropagation(); e.stopPropagation();
Dropzone.forElement(target).removeAllFiles(true); Dropzone.forElement($formDropzone.get(0)).removeAllFiles(true);
}); });
// If 'error' event is fired, we store a failed files, // If 'error' event is fired, we store a failed files,
......
---
title: Fix cancel button not working while uploading on the new issue page
merge_request: 15137
author:
type: fixed
...@@ -583,6 +583,16 @@ describe 'Issues' do ...@@ -583,6 +583,16 @@ describe 'Issues' do
expect(page.find_field("issue_description").value).not_to match /\n\n$/ expect(page.find_field("issue_description").value).not_to match /\n\n$/
end end
it "cancels a file upload correctly" do
dropzone_file([Rails.root.join('spec', 'fixtures', 'dk.png')], 0, false)
click_button 'Cancel'
expect(page).to have_button('Attach a file')
expect(page).not_to have_button('Cancel')
expect(page).not_to have_selector('.uploading-progress-container', visible: true)
end
end end
context 'form filled by URL parameters' do context 'form filled by URL parameters' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment