Defer hiding and reloading the draft banner
Review Request #7924 — Created Jan. 30, 2016 and submitted
We need to do this because hideAndReload might be called from within a $.funcQueue, and on Firefox, a change to window.location will result in later functions in the $.funcQueue not running. Deferring until the next tick of the event loop allows the rest of the $.funcQueue to run. Fixes bug 3727.
Created a review request that I had edit rights on. After
publishing, I attempted to create a review on myself, and hit
"Publish" in the review dialog. Without this patch, I see the
"Are you sure you want to leave this page" popup. With this
patch, I don't see the popup, and the page reloads correctly.
| Description | From | Last Updated |
|---|---|---|
|
Can you change this to a /* */ comment? |
 david
|
-
I think there was a related fix about showing the confirmation prompt when you didn't have editing rights to the review request, for reviews/replies?
- Change Summary:
-
Switched to /* */ style comment block.
- Commit:
-
5a493070e23c744e67ff4810d6d663ac54d1cdb313cf704fb09a8b5440f0802056557edc1f4b6e3d
Tool: Pyflakes Ignored Files: reviewboard/static/rb/js/views/draftReviewBannerView.js Tool: PEP8 Style Checker Ignored Files: reviewboard/static/rb/js/views/draftReviewBannerView.js