Allow for prev/next navigation out of diffbase
If a user in diff view changes the diff range to one that no longer
includes the file that they are on, the Prev/Next links are grayed out
and they are forced to return to the change view to navigate the diff.
This change sets Next to direct to the first file in the list and Prev
to point to the last file in the list.
Bug: Issue 4932
Change-Id: Ifb460c9721bfafbc19afa68253402b9dcd2f2c3e
diff --git a/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js b/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
index 5dc1d95..600415a 100644
--- a/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
+++ b/polygerrit-ui/app/elements/diff/gr-diff-view/gr-diff-view.js
@@ -353,7 +353,10 @@
if (!path || fileList.length === 0) { return null; }
var idx = fileList.indexOf(path);
- if (idx === -1) { return null; }
+ if (idx === -1) {
+ var file = direction > 0 ? fileList[0] : fileList[fileList.length - 1];
+ return this._getDiffURL(this._changeNum, this._patchRange, file);
+ }
idx += direction;
// Redirect to the change view if opt_noUp isn’t truthy and idx falls
