Fix warning for unsaved work on beforeUnload
Review Request #485 — Created March 17, 2017 and updated
Information | |
---|---|
guest822 | |
Review Board | |
176916d... | |
Reviewers | |
demo | |
Previously, when editing a review or comments as a user without edit rights, warning users about losing unsaved work did not get displayed. Now the ReviewRequestEditorView's beforeUnload event handler does not check a user's edit rights and only checks to see if any fields are being edited at the moment. New unit tests testing the onBeforeUnload event handler has been added. Testing Done: - Ran JS unit tests. - Check that ReviewDialogView for both edit and non-edit privileged users are warned before unloading. Bugs closed: 4084 Reviewed at https://reviews.reviewboard.org/r/8635/
Description | From | Last Updated |
---|---|---|
444 |
|
|
add general comment |
|
|
Where is this? |
|
|
ERROR################ |
|
|
ffffff |
|
|
ijnjnj |
|
|
asdasd asdasdkjkn |
|
|
asdf |
|
|
expand |
|
|
123 |
|
|
// is code rendered? if ((this.model.get('editable)) { return msg; } header |
|
|
how about message |
|
|
Test1 |
|
|
keep die return statement |
|
|
This is not this |
|
|
comment AAA BBB CCC |
|
|
Test2 |
|
|
This sounds strange.... |
|
|
Test3 |
|
|
Test4 |
|
-
-
reviewboard/static/rb/js/views/reviewRequestEditorView.js (Diff revision 1) // is code rendered?
if ((this.model.get('editable)) {
return msg;
}header
-
-
reviewboard/static/rb/js/views/tests/reviewRequestEditorViewTests.js (Diff revision 1) This sounds strange....
-
-
-
-
-
-
Please fix everything!!!!
-
-
-
-
ffaaa
-
-
reviewboard/static/rb/js/views/reviewRequestEditorView.js (Diff revision 1) second comment on same line
-
Here my 2€
-
reviewboard/static/rb/js/views/reviewRequestEditorView.js (Diff revision 1) keep die return statement
-
reviewboard/static/rb/js/views/tests/reviewRequestEditorViewTests.js (Diff revision 1) i don't like this comment
-
Ende