[WIP] Drag 'n Drop Inline Images
Review Request #6351 — Created Sept. 21, 2014 and discarded
Adding support for dragging and dropping images into the markdown editor. Instead of having upload their image first and then link to that image, users can simply drag-and-drop their image file into the markdown editor. This will upload the image to reviewboard and insert a link to that image in the editor.
To accomplish this, a new resource for uesr file attachments was added (see https://reviews.reviewboard.org/r/6220/ for initial work) and some modifications to the markdown editor JS were made to support drag-and-drop (see https://reviews.reviewboard.org/r/5954/ for initial work).
Still a work-in-progress.
Ignore all the JavaScript files for now please
Description | From | Last Updated |
---|---|---|
Make sure to keep the includes in alphabetical order. so put this under fileAttachmentCommentReplyModel.js |
ML mloyzer | |
Alphabetical order again. |
ML mloyzer | |
'FILE_ALREADY_EXISTS' imported but unused |
reviewbot | |
Col: 9 E265 block comment should start with '# ' |
reviewbot | |
Col: 9 E265 block comment should start with '# ' |
reviewbot | |
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot | |
Col: 18 E251 unexpected spaces around keyword / parameter equals |
reviewbot | |
Col: 19 E502 the backslash is redundant between brackets |
reviewbot | |
Col: 18 E251 unexpected spaces around keyword / parameter equals |
reviewbot |
-
Hey David.
I haven't looked at the change yet, but can you change the description to first summarize what this feature provides and how it's implemented, and then later in the description, refer to the other review requests?
Every review request filed should strive to convey enough information for the reviewers to know exactly what the change is about without having to click on any other links. These descriptions are also used for commit messages, which we try to keep detailed and individually descriptive.
- Change Summary:
-
Updating description with some more detailed information.
- Description:
-
~ Adding new resource for user file attachments and initial work for supporting drag-and-drop images into markdown editor. New RR combining https://reviews.reviewboard.org/r/6220/ and https://reviews.reviewboard.org/r/5954/ (with some modifications right now).
~ Adding support for dragging and dropping images into the markdown editor. Instead of having upload their image first and then link to that image, users can simply drag-and-drop their image file into the markdown editor. This will upload the image to reviewboard and insert a link to that image in the editor.
+ + To accomplish this, a new resource for uesr file attachments was added (see https://reviews.reviewboard.org/r/6220/ for initial work) and some modifications to the markdown editor JS were made to support drag-and-drop (see https://reviews.reviewboard.org/r/5954/ for initial work).
+ + Still a work-in-progress.
- Change Summary:
-
Some work on the Create & Update endpoints for user file attachments
- Commit:
-
4107dc8020ba48d71810686abb6910f9b2da3640b87f78958262925f140bf5c5804b4999d72c9a66
- Diff:
-
Revision 2 (+640 -67)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js
-
-
-
- Commit:
-
b87f78958262925f140bf5c5804b4999d72c9a666869a30d83d04c5884d714d915a68c49787b0df0
- Diff:
-
Revision 3 (+643 -67)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js
- Description:
-
Adding support for dragging and dropping images into the markdown editor. Instead of having upload their image first and then link to that image, users can simply drag-and-drop their image file into the markdown editor. This will upload the image to reviewboard and insert a link to that image in the editor.
To accomplish this, a new resource for uesr file attachments was added (see https://reviews.reviewboard.org/r/6220/ for initial work) and some modifications to the markdown editor JS were made to support drag-and-drop (see https://reviews.reviewboard.org/r/5954/ for initial work).
~ Still a work-in-progress.
~ Still a work-in-progress.
+ Ignore all the JavaScript files for now please - Commit:
-
6869a30d83d04c5884d714d915a68c49787b0df00a6436ae7cf97442fcc55e6b1270daaeb92b346e
- Diff:
-
Revision 4 (+904 -322)
- Commit:
-
0a6436ae7cf97442fcc55e6b1270daaeb92b346edb7dd3205cf80e641c304d60f79cc839ebfa5680
- Diff:
-
Revision 5 (+897 -323)
- Commit:
-
db7dd3205cf80e641c304d60f79cc839ebfa568037cf33b673807e79a97fab587b92c9eef3e92aed
- Diff:
-
Revision 6 (+933 -323)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js
-
- Commit:
-
37cf33b673807e79a97fab587b92c9eef3e92aed77995d530c7e6b2cf964eec0b525a56f6b4af10c
- Diff:
-
Revision 7 (+934 -323)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js
-
-
- Commit:
-
77995d530c7e6b2cf964eec0b525a56f6b4af10c5d8f8cb6c100c28783ff928cf0fd5b36de694c9a
- Diff:
-
Revision 8 (+934 -323)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js
-
- Commit:
-
5d8f8cb6c100c28783ff928cf0fd5b36de694c9a990a6b14a32e0114c04e5d2e0a629f4f3a817f0c
- Diff:
-
Revision 9 (+934 -323)
-
Tool: Pyflakes Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js Tool: PEP8 Style Checker Processed Files: reviewboard/webapi/resources/root.py reviewboard/webapi/resources/base_file_attachment.py reviewboard/webapi/resources/draft_file_attachment.py reviewboard/webapi/resources/diff_file_attachment.py reviewboard/attachments/models.py reviewboard/webapi/errors.py reviewboard/attachments/evolutions/__init__.py reviewboard/attachments/mimetypes.py reviewboard/attachments/evolutions/file_attachment_ownership.py reviewboard/webapi/resources/file_attachment.py reviewboard/webapi/resources/file_attachment_user.py reviewboard/webapi/resources/base_review_request_file_attachment.py reviewboard/attachments/forms.py reviewboard/staticbundles.py Ignored Files: reviewboard/static/rb/js/views/dndUploaderView.js reviewboard/static/rb/js/resources/models/fileAttachmentUserModel.js reviewboard/static/rb/js/views/reviewRequestEditorView.js reviewboard/static/rb/js/resources/models/apiTokenModel.js reviewboard/static/rb/js/views/commentDialogView.js reviewboard/static/rb/js/views/markdownEditorView.js