diff options
author | Stan Hu <stanhu@gmail.com> | 2015-07-18 09:06:59 +0300 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2015-07-18 09:06:59 +0300 |
commit | 895d988d55f3eacac45fdc53f1699cae3af6c06d (patch) | |
tree | bb349d767f968698c8942072462c8136a3e7d685 /app/assets | |
parent | 6615428679b28e9d6c6cf5dd887280b092fbdded (diff) | |
parent | 782500158608eba36e27f53e1d8822d25710fe80 (diff) |
Merge branch 'fix-note-edit-upload-link' into 'master'
Fix file upload dialog for comment editing
This fixes the file upload dialog which currently works for new comments
but not for comment editing. The cause for this is a missing DOM class
which causes traversal to the dropzone to fail.
This is fixed by adding the expected `gfm-form` class to the edit form.
It has to be added dynamically since adding it to the form template
would break the form's layout and comment saving.
The following screenshot illustrates the issue:
![gitlab-dropzone-upload-dialog-bug](https://gitlab.com/gitlab-org/gitlab-ce/uploads/3a9885c8ba7c07338148ec9e605e5771/gitlab-dropzone-upload-dialog-bug.png)
See merge request !956
Diffstat (limited to 'app/assets')
-rw-r--r-- | app/assets/javascripts/notes.js.coffee | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/assets/javascripts/notes.js.coffee b/app/assets/javascripts/notes.js.coffee index 1c05a2b9fe8..bcff7bcc49e 100644 --- a/app/assets/javascripts/notes.js.coffee +++ b/app/assets/javascripts/notes.js.coffee @@ -298,7 +298,7 @@ class @Notes note.find(".note-header").hide() base_form = note.find(".note-edit-form") form = base_form.clone().insertAfter(base_form) - form.addClass('current-note-edit-form') + form.addClass('current-note-edit-form gfm-form') form.find('.div-dropzone').remove() # Show the attachment delete link |