Summary:

Disable drag & drop attachment upload when modal dialog is shown

Review Request #10108 — Created July 29, 2018 and updated Aug. 4, 2018, 12:02 p.m.

Information
Owner:	easyb
Repository:	Review Board
Branch:	release-3.0.x
Bugs:
Depends On:
Commit:	c6a4dba...
Reviewers
Groups:	reviewboard
People:

Description

The previous behavior could be very annoying when trying to update a
diff using drag & drop. Also, when using the "Add File" dialog the
behavior was confusing.

Testing Done

Chose "Update Diff", dragged file onto browser window
-> drag & drop overlay did not appear
Chose "Add File", dragged file onto browser window
-> drag & drop overlay did not appear
Dragged file onto review request view (Reviews/Diff)
-> drag & drop overlay appeared

Issues

Description	From	Last Updated
There's no guarantee that this class name will be used in the future. I think an option that would help …	chipx86	July 30, 2018, 10:06 p.m.

flake8 passed.

JSHint passed.

Description:

~		The behavior can be very annoying when trying to update a diff using
~		drag & drop. Also, when using the "Add File" dialog the behavior is
~		confusing.
	~	The previous behavior could be very annoying when trying to update a
	~	diff using drag & drop. Also, when using the "Add File" dialog the
	~	behavior was confusing.

Fix it!

Show all issues

There's no guarantee that this class name will be used in the future.
I think an option that would help future-proof would be to introduce signals that RB.DialogView subclasses would automatically emit when showing/hiding to indicate to callers that a dialog is or is not on screen. The file upload area could then listen to this, disabling/re-enabling the upload support as needed.

easyb Aug. 4, 2018, 12:02 p.m.

Makes sense. But, I am a little confused here. To me it seems that RB.UploadAttachmentView and RB.UpdateDiffView do not inherit from RB.DialogView. Further, I do not get how I can trigger an event when $.modalBox is being hidden.