Fixed the Update -> Add File review request action not responding to clicks.
Review Request #9207 — Created Sept. 22, 2017 and submitted — Latest diff uploaded
In the Review Request screen, clicking Update -> Add File would not do
anything.The Update element's click event is registered before the Add File
element's click event. As a result, the Update element's click event is
firing before, eating up the Add File's click event and thus causing the
upload file dialog to never appear.The
onUploadFileClicked()
function was moved fromReviewablePageView
to
ReviewRequestEditorView
, and the element's handler registered in the
setupActions()
function. The.has-menu
handler was moved to
ReviewablePageView
, better respecting the ownership structure of the
page. This changes the order in which the click events are fired, allowing
theonUploadFileClicked()
function to actually fire in response to the
Add File element being clicked.Furthermore, the
has-menu
handler was moved from
ReviewRequestEditorView
toReviewablePageView
After the fix, the upload file dialog appears as intended.
Ran JSTests.
Tested manually by clicking Update -> Add File, where the upload file
dialog appears. This was done on both computer and mobile browsers.